On Mon, Aug 11, 2014 at 01:49:19AM +0200, Michael Niedermayer wrote:
> From: Luca Barbato
>
> TODO:bump
>
> Signed-off-by: Michael Niedermayer
> ---
> libswresample/Makefile |1 +
> libswresample/swresample.h | 61 +++
> libswresample/swresample_frame.c | 158
On 11 August 2014 00:49, Michael Niedermayer wrote:
> From: Luca Barbato
Clearly this is Luca's patch but rewritten by you for swr so you
should say "Based on patch from Luca for avresample" or similar.
IANAL but this is quite dubious from a copyright perspective.
___
On 2014-08-11 07:44, Clément Bœsch wrote:
> /**
> * @file
> * Walken IDCT
> * Alternative idct implementations for decoding compatibility
> *
> * @author Skal
> * @note this "C" version is not the original one, but is modified to
> * yield the same error profile than the MMX version.
Hi Guys,
The FFmpeg hls module can make m3u8 and ts, but it dosen't delete the
old ts segment file.
If always run this module, the disk will full, so this patch can fix
the problem.
When update the segment list m3u8 file, it will delete the ts segment
out range from the list file.
On Mon, Aug 11, 2014 at 07:44:25AM +0200, Clément Bœsch wrote:
> On Mon, Aug 11, 2014 at 05:18:32AM +0200, Michael Niedermayer wrote:
> > From: Pascal Massimino
> >
> > Thanks to Pascal Massimino and Michael Militzer for permission to use under
> > LGPL
> > Signed-off-by: Michael Niedermayer
>
On Mon, Aug 11, 2014 at 10:56:11AM +0200, James Darnley wrote:
> On 2014-08-11 07:44, Clément Bœsch wrote:
> > /**
> > * @file
> > * Walken IDCT
> > * Alternative idct implementations for decoding compatibility
> > *
> > * @author Skal
> > * @note this "C" version is not the original one, but
On 2014-08-11 12:39, Michael Niedermayer wrote:
> On Mon, Aug 11, 2014 at 10:56:11AM +0200, James Darnley wrote:
>> On 2014-08-11 07:44, Clément Bœsch wrote:
>>> /**
>>> * @file
>>> * Walken IDCT
>>> * Alternative idct implementations for decoding compatibility
>>> *
>>> * @author Skal
>>> *
On Mon, Aug 11, 2014 at 12:36:56AM -0300, James Almer wrote:
> On 11/08/14 12:18 AM, Michael Niedermayer wrote:
> > From: Pascal Massimino
> >
> > Thanks to Pascal Massimino and Michael Militzer for permission to use under
> > LGPL
> > Signed-off-by: Michael Niedermayer
> > ---
> > libavcodec/
On Sat, Aug 09, 2014 at 09:04:32PM -0300, James Almer wrote:
> Signed-off-by: James Almer
> ---
> libavcodec/libavcodec.v | 18 --
> 1 file changed, 18 deletions(-)
applied
thanks
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Asymptotically
On Sat, Aug 09, 2014 at 10:01:46PM -0300, James Almer wrote:
> Signed-off-by: James Almer
> ---
> libavcodec/raw.h | 4
> 1 file changed, 4 deletions(-)
applied
thanks
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
You can kill me, but you cannot change
On Sat, Aug 09, 2014 at 10:01:47PM -0300, James Almer wrote:
> Signed-off-by: James Almer
> ---
> libavformat/mpegts.c | 18 --
> libavformat/mpegts.h | 7 ---
> 2 files changed, 25 deletions(-)
applied
thanks
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF1
Decoders/parsers run ff_metadata_conv() unconditionally. This makes
it impossible to retrieve or keep original metadata field names.
If one wishes to skip ff_metadata_conv() calls at the decoding/parsing
stage. This patch implements AVFMT_FLAG_NO_META_CONV which is set by
the decoding fflag o
On Sat, Aug 09, 2014 at 09:04:33PM -0300, James Almer wrote:
> Except for those currently used by ffserver.
>
> Signed-off-by: James Almer
> ---
> libavformat/libavformat.v | 9 -
> 1 file changed, 9 deletions(-)
mostly applied
thanks
[...]
--
Michael GnuPG fingerprint: 9FF2128B1
From: Clément Bœsch
The reasoning behind this addition is that various third party
applications are interested in getting some motion information out of a
video "for free" when it is available.
It was considered to export other information as well (such as the intra
information about the block,
Am 11.08.2014 um 00:20 schrieb Michael Niedermayer :
> On Sun, Aug 10, 2014 at 08:36:37PM +0200, Daniel Oberhoff wrote:
>>
>>
>> Von meinem iPhone gesendet
>>
>>> Am 10.08.2014 um 17:50 schrieb Michael Niedermayer :
>>>
On Sun, Aug 10, 2014 at 03:36:04PM +0200, Nicolas George wrote:
On Mon, Aug 11, 2014 at 05:33:10PM +0200, Daniel Oberhoff wrote:
>
> Am 11.08.2014 um 00:20 schrieb Michael Niedermayer :
>
> > On Sun, Aug 10, 2014 at 08:36:37PM +0200, Daniel Oberhoff wrote:
> >>
> >>
> >> Von meinem iPhone gesendet
> >>
> >>> Am 10.08.2014 um 17:50 schrieb Michael Niedermay
On Sun, Aug 10, 2014 at 12:25:33AM -0700, Andrew Kelley wrote:
>
> High quality libraries must iterate on their API. Especially for a library
> trying to solve such a complex problem as audio and video encoding and
> decoding for every codec and format. It is unreasonable to expect no
> incompatib
On Sun, 2014-08-10 at 23:02 +0200, Andreas Cadhalpun wrote:
[...]
> * dvswitch: Still uses CodecID (and also avcodec_encode_video, but
> that is still present in FFmpeg.) [3]
[...]
dvswitch was also broken by the removal of support for downscaled
decoding of DV video. I don't know whether t
Hi,
On Montag, 11. August 2014, Ben Hutchings wrote:
> dvswitch was also broken by the removal of support for downscaled
> decoding of DV video. I don't know whether that change is specific to
> libav or was also made in FFmpeg.
dvswitch is still broken and there is no dvswitch in jessie...
We
This has become necessary since the new mime field was added.
Signed-off-by: Reimar Döffinger
---
libavformat/img2dec.c | 2 +-
tools/probetest.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavformat/img2dec.c b/libavformat/img2dec.c
index d70fc75..a82f50f 100644
Hello,
(sorry for being too lazy to send a patch)
With the major version bump AVProbeData was extended by a new field.
This so far has broken 3 places within FFmpeg and one within MPlayer,
where AVProbeData was only initialized field-by-field.
This will cause "random" crashes.
I'm at this point fai
Reimar Döffinger gmx.de> writes:
> This has become necessary since the new mime field was added.
The patch is definitely ok, sorry for missing this
when I fixed one occurrence.
Carl Eugen
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://
Hi Ivan,
On 11.08.2014 01:23, Ivan Kalvachev wrote:
The patch is inspired by something I read in the Debian discussion.
Libav and FFmpeg could be installed side by side without conflicts in
the libraries, thanks to using additional suffixes.
Thanks for your interest in the matter.
However de
On 10.08.2014 02:04, James Almer wrote:
Signed-off-by: James Almer
---
libavutil/libavutil.v | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavutil/libavutil.v b/libavutil/libavutil.v
index eb16ae1..e9f04cb 100644
--- a/libavutil/libavutil.v
+++ b/libavutil/libavutil.v
Hello,
Apologies for not being able to resist answering even if it is getting
off-topic.
On Sun, Aug 10, 2014 at 05:43:33PM -0400, Theodore Ts'o wrote:
> On Sun, Aug 10, 2014 at 12:25:33AM -0700, Andrew Kelley wrote:
> >
> > High quality libraries must iterate on their API. Especially for a libr
On Mon, Aug 11, 2014 at 06:09:54PM +, Carl Eugen Hoyos wrote:
> Reimar Döffinger gmx.de> writes:
>
> > This has become necessary since the new mime field was added.
>
> The patch is definitely ok, sorry for missing this
> when I fixed one occurrence.
I'm glad you fixed the other one first,
> ffmpeg | branch: master | James Almer | Sat Aug 9
> 21:04:33 2014 -0300| [7b3de03c8fa7c7a49522b45dc90f9970417554b4] | committer:
> Michael Niedermayer
>
> lavf: stop exporting internal functions
>
> Except for those currently used by ffserver.
>
> Signed-off-by: James Almer
>
> Also left som
On 11/08/14 3:39 PM, Andreas Cadhalpun wrote:
> On 10.08.2014 02:04, James Almer wrote:
>> Signed-off-by: James Almer
>> ---
>> libavutil/libavutil.v | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/libavutil/libavutil.v b/libavutil/libavutil.v
>> index eb16ae1..e9f04
This allows getting rid of the many, slightly differing, implementations
of basically the same thing.
Signed-off-by: Reimar Döffinger
---
doc/APIchanges | 3 +++
libavutil/dict.c| 8
libavutil/dict.h| 8
libavutil/version.h | 2 +-
4 files changed, 20 insertions(+)
Get rid of the many, slightly differing, implementations
of basically the same thing.
Signed-off-by: Reimar Döffinger
---
ffmpeg_opt.c | 12 +++-
ffplay.c | 2 +-
libavfilter/vf_bbox.c| 4 +---
libavfilter/vf_cropdetect.c |
Unfortunately this was not explicitly documented and thus
might be very risky.
But basically all uses I saw in FFmpeg had a memleak in these
cases.
Signed-off-by: Reimar Döffinger
---
libavutil/dict.c | 9 +++--
libavutil/dict.h | 2 ++
2 files changed, 9 insertions(+), 2 deletions(-)
diff
Ensure this is even the case if they are empty because
we failed adding the first entry.
Signed-off-by: Reimar Döffinger
---
libavutil/dict.c | 4
1 file changed, 4 insertions(+)
diff --git a/libavutil/dict.c b/libavutil/dict.c
index f23b768..bdb5690 100644
--- a/libavutil/dict.c
+++ b/lib
On Sun, Aug 10, 2014 at 11:05:13PM -0300, James Almer wrote:
> On 10/08/14 8:49 PM, Michael Niedermayer wrote:
> > From: Luca Barbato
> >
> > TODO:bump
>
> APIChanges entry as well
added TODO
[...]
> > +int swr_config_by_frames(SwrContext *swr, AVFrame *out, AVFrame *in);
>
> Maybe swr_config
It was wrongly being exported and used by libavfilter.
Signed-off-by: James Almer
---
libavfilter/deshake_opencl.c | 4 ++--
libavfilter/unsharp_opencl.c | 6 +++---
libavutil/opencl_internal.c | 2 +-
libavutil/opencl_internal.h | 2 +-
4 files changed, 7 insertions(+), 7 deletions(-)
diff -
On Mon, Aug 11, 2014 at 09:25:20AM +0100, Kieran Kunhya wrote:
> On 11 August 2014 00:49, Michael Niedermayer wrote:
> > From: Luca Barbato
>
> Clearly this is Luca's patch but rewritten by you for swr so you
> should say "Based on patch from Luca for avresample" or similar.
its copy and pasted
On Aug 10, 2014 4:49 PM, "Michael Niedermayer" wrote:
>
> From: Luca Barbato
>
> TODO:bump
>
> Signed-off-by: Michael Niedermayer
> ---
> libswresample/Makefile |1 +
> libswresample/swresample.h | 61 +++
> libswresample/swresample_frame.c | 158
+
On Mon, Aug 11, 2014 at 10:02:54AM +0200, Clément Bœsch wrote:
> On Mon, Aug 11, 2014 at 01:49:19AM +0200, Michael Niedermayer wrote:
> > From: Luca Barbato
> >
> > TODO:bump
> >
> > Signed-off-by: Michael Niedermayer
> > ---
> > libswresample/Makefile |1 +
> > libswresample/swr
On Mon, 11 Aug 2014 21:17:18 +0200
Reimar Döffinger wrote:
> Unfortunately this was not explicitly documented and thus
> might be very risky.
> But basically all uses I saw in FFmpeg had a memleak in these
> cases.
It's the more convenient behavior, although on the other hand it feels
wrong to c
On Mon, Aug 11, 2014 at 12:53:30PM -0700, Timothy Gu wrote:
> On Aug 10, 2014 4:49 PM, "Michael Niedermayer" wrote:
[...]
> > + * to an internal FIFO buffer, to be returned at the next call to this
> function
> > + * or to swr_convert() or to swr_convert_frame().
>
> at the next call to this func
On Mon, Aug 11, 2014 at 08:05:38PM +0200, Reimar Döffinger wrote:
> Hello,
> (sorry for being too lazy to send a patch)
> With the major version bump AVProbeData was extended by a new field.
> This so far has broken 3 places within FFmpeg and one within MPlayer,
> where AVProbeData was only initial
On Mon, Aug 11, 2014 at 04:35:39PM -0300, James Almer wrote:
> It was wrongly being exported and used by libavfilter.
>
> Signed-off-by: James Almer
> ---
> libavfilter/deshake_opencl.c | 4 ++--
> libavfilter/unsharp_opencl.c | 6 +++---
> libavutil/opencl_internal.c | 2 +-
> libavutil/opencl
On Mon, 11 Aug 2014 20:40:28 +0200
Reimar Döffinger wrote:
> Hello,
>
> Apologies for not being able to resist answering even if it is getting
> off-topic.
>
> On Sun, Aug 10, 2014 at 05:43:33PM -0400, Theodore Ts'o wrote:
> > On Sun, Aug 10, 2014 at 12:25:33AM -0700, Andrew Kelley wrote:
> > >
On 11/08/14 5:31 PM, Michael Niedermayer wrote:
> On Mon, Aug 11, 2014 at 04:35:39PM -0300, James Almer wrote:
>> It was wrongly being exported and used by libavfilter.
>>
>> Signed-off-by: James Almer
>> ---
>> libavfilter/deshake_opencl.c | 4 ++--
>> libavfilter/unsharp_opencl.c | 6 +++---
>>
There are mostly issues (in order), potentially causing crashes:
- no check on the buffer left
- no reallocation when the buffer will overflow
- incorrect buffer size when encoding alpha
Christophe Gisquet (4):
proresenc_kostya: remove unneeded parameters
proresenc_kostya: report buffer overfl
If the allocated size, despite best efforts, is too small, exit
with the appropriate error.
---
libavcodec/proresenc_kostya.c | 13 ++---
1 file changed, 10 insertions(+), 3 deletions(-)
diff --git a/libavcodec/proresenc_kostya.c b/libavcodec/proresenc_kostya.c
index 24cb333..a70ae3c 1006
The buffer allocation may be incorrect (e.g. with an alpha plane),
and currently causes the buffer to be set to NULL by init_put_bits,
later on causing crashing.
So, detect that situation, and if detected, reallocate the buffer
and ask a sample if it happens.
Fixes ticket #2760
---
libavcodec/pr
The packet buffer allocation considered as dct-coded, while it is
actually run-coded and thus requires a larger buffer.
---
libavcodec/proresenc_kostya.c | 13 ++---
1 file changed, 10 insertions(+), 3 deletions(-)
diff --git a/libavcodec/proresenc_kostya.c b/libavcodec/proresenc_kostya.c
---
libavcodec/proresenc_kostya.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/libavcodec/proresenc_kostya.c b/libavcodec/proresenc_kostya.c
index 93bcde7..24cb333 100644
--- a/libavcodec/proresenc_kostya.c
+++ b/libavcodec/proresenc_kostya.c
@@ -471,7 +471,6 @@ static voi
On Mon, Aug 11, 2014 at 09:17:16PM +0200, Reimar Döffinger wrote:
> This allows getting rid of the many, slightly differing, implementations
> of basically the same thing.
>
> Signed-off-by: Reimar Döffinger
> ---
> doc/APIchanges | 3 +++
> libavutil/dict.c| 8
> libavutil/dic
On 2014-08-11 21:42 +0200, Michael Niedermayer wrote:
> On Mon, Aug 11, 2014 at 09:25:20AM +0100, Kieran Kunhya wrote:
> > On 11 August 2014 00:49, Michael Niedermayer wrote:
> > > From: Luca Barbato
> >
> > Clearly this is Luca's patch but rewritten by you for swr so you
> > should say "Based o
On Mon, Aug 11, 2014 at 10:53:56PM +0200, wm4 wrote:
>
> To be fair, FFmpeg does its own "manual" symbol versioning by appending
> increasing numbers to function names. But the real problem are not
> these functions, but public structs. Imagine a new API user fighting to
> guess which fields in AV
On Mon, Aug 11, 2014 at 10:06:07PM +, Christophe Gisquet wrote:
> ---
> libavcodec/proresenc_kostya.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
applied
thanks
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
During times of universal deceit,
Am 11.08.2014 um 17:39 schrieb Clément Bœsch :
> On Mon, Aug 11, 2014 at 05:33:10PM +0200, Daniel Oberhoff wrote:
>>
>> Am 11.08.2014 um 00:20 schrieb Michael Niedermayer :
>>
>>> On Sun, Aug 10, 2014 at 08:36:37PM +0200, Daniel Oberhoff wrote:
Von meinem iPhone gesendet
On 11/08/14 at 17:39, Steven Liu wrote:
> Hi Guys,
Hello,
>
> The FFmpeg hls module can make m3u8 and ts, but it dosen't delete the
> old ts segment file.
> If always run this module, the disk will full, so this patch can fix
> the problem.
> When update the segment list m3u8 file, it will delete
On Mon, Aug 11, 2014 at 10:06:10PM +, Christophe Gisquet wrote:
> The packet buffer allocation considered as dct-coded, while it is
> actually run-coded and thus requires a larger buffer.
> ---
> libavcodec/proresenc_kostya.c | 13 ++---
> 1 file changed, 10 insertions(+), 3 deletions(
On Mon, Aug 11, 2014 at 10:06:08PM +, Christophe Gisquet wrote:
> If the allocated size, despite best efforts, is too small, exit
> with the appropriate error.
> ---
> libavcodec/proresenc_kostya.c | 13 ++---
> 1 file changed, 10 insertions(+), 3 deletions(-)
This is kind of an inter
On 8/11/14, Andreas Cadhalpun wrote:
> Hi Ivan,
>
> On 11.08.2014 01:23, Ivan Kalvachev wrote:
>> The patch is inspired by something I read in the Debian discussion.
>> Libav and FFmpeg could be installed side by side without conflicts in
>> the libraries, thanks to using additional suffixes.
>
>
On Mon, Aug 11, 2014 at 10:06:09PM +, Christophe Gisquet wrote:
> The buffer allocation may be incorrect (e.g. with an alpha plane),
> and currently causes the buffer to be set to NULL by init_put_bits,
> later on causing crashing.
>
> So, detect that situation, and if detected, reallocate the
Hi,
wm4:
> Build something on a newer glibc system, and try to run the binary on
> an older system. It most likely won't work - even if it could in
> theory. (At least it was this way some years ago. Probably still is.)
What would be the point of introducing new or updated interfaces
if you then
On Tue, Aug 12, 2014 at 12:52:45AM +0200, Daniel Oberhoff wrote:
>
> Am 11.08.2014 um 17:39 schrieb Clément Bœsch :
>
> > On Mon, Aug 11, 2014 at 05:33:10PM +0200, Daniel Oberhoff wrote:
> >>
> >> Am 11.08.2014 um 00:20 schrieb Michael Niedermayer :
> >>
> >>> On Sun, Aug 10, 2014 at 08:36:37PM
On Mon, Aug 11, 2014 at 05:56:50PM -0300, James Almer wrote:
> On 11/08/14 5:31 PM, Michael Niedermayer wrote:
> > On Mon, Aug 11, 2014 at 04:35:39PM -0300, James Almer wrote:
> >> It was wrongly being exported and used by libavfilter.
> >>
> >> Signed-off-by: James Almer
> >> ---
> >> libavfilte
2014-08-12 7:26 GMT+08:00 Simon Thelen :
> On 11/08/14 at 17:39, Steven Liu wrote:
> > Hi Guys,
> Hello,
> >
> > The FFmpeg hls module can make m3u8 and ts, but it dosen't delete the
> > old ts segment file.
> > If always run this module, the disk will full, so this patch can fix
> > the problem.
Its API is marked as experimental, we should not export experimental API
from shared libs
Signed-off-by: Michael Niedermayer
---
configure |4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/configure b/configure
index 0ac6132..84d308e 100755
--- a/configure
+++ b/configure
2014-08-12 7:26 GMT+08:00 Simon Thelen :
> On 11/08/14 at 17:39, Steven Liu wrote:
> > Hi Guys,
> Hello,
> >
> > The FFmpeg hls module can make m3u8 and ts, but it dosen't delete the
> > old ts segment file.
> > If always run this module, the disk will full, so this patch can fix
> > the problem.
On Mon, Aug 11, 2014 at 08:39:24PM +0200, Andreas Cadhalpun wrote:
> On 10.08.2014 02:04, James Almer wrote:
> >Signed-off-by: James Almer
> >---
> > libavutil/libavutil.v | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> >diff --git a/libavutil/libavutil.v b/libavutil/libavutil.v
>
>From 48e4da7d6476ac2a62cc462478b8ccf4d0c45361 Mon Sep 17 00:00:00 2001
From: Youka
Date: Tue, 12 Aug 2014 04:32:02 +0200
Subject: [PATCH] fix: 'make' with mingw32
Older mingw32 compilers (not mingw-64, but tdm [default by IDE C::B]) don't
auto-include pthread headers, so struct timespec & nanos
Hi,
2014-08-12 2:34 GMT+02:00 Michael Niedermayer :
>> +if (pkt_size <= buf - orig_buf) {
>
> this isnt sufficient, there could be 1 byte space left, then the
> reallocate wouldnt run and encode_slice() would run into the extra
> padding and fail
Yeah, you're right. I have no idea
Hi,
2014-08-12 1:56 GMT+02:00 Michael Niedermayer :
>> - (2 + 2 * ctx->num_planes +
>> + (2 + 2 * FFMIN(3, ctx->num_planes) +
>
> slice_hdr_size is 2 + 2 * (ctx->num_planes - 1);
>
> is this difference intended ?
I don't know why
68 matches
Mail list logo
