Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Offset index timestamps by the minimum pts to make first pts zero.

2017-06-03 Thread Sasi Inguva
> -// Offset the DTS by ctts[0] to make the PTS of the first frame 0 > -if (ctts_data_old && ctts_count_old > 0) { > -edit_list_dts_entry_end -= ctts_data_old[0].duration; > -av_log(mov->fc, AV_LOG_DEBUG, "Offset DTS by ctts[%d].duration: %d\n", 0,

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/utils: Fix several integer overflows.

2017-06-03 Thread James Almer
On 6/3/2017 9:55 PM, Michael Niedermayer wrote: > On Sat, Jun 03, 2017 at 09:47:32PM -0300, James Almer wrote: >> On 6/3/2017 9:25 PM, Michael Niedermayer wrote: >>> Signed-off-by: Michael Niedermayer >>> --- >>> libavcodec/utils.c | 7 +++ >>> 1 file changed, 7

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/utils: Fix several integer overflows.

2017-06-03 Thread Michael Niedermayer
On Sat, Jun 03, 2017 at 09:47:32PM -0300, James Almer wrote: > On 6/3/2017 9:25 PM, Michael Niedermayer wrote: > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/utils.c | 7 +++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/libavcodec/utils.c

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/utils: Fix several integer overflows.

2017-06-03 Thread James Almer
On 6/3/2017 9:25 PM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavcodec/utils.c | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/libavcodec/utils.c b/libavcodec/utils.c > index cde5849a41..feee7556ac 100644 > ---

[FFmpeg-devel] [PATCH 3/3] avformat/utils: Slightly un-clutter code in determinable_frame_size() by using a switch

2017-06-03 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/utils.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index c5f1eac185..bbc7a7b547 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@

[FFmpeg-devel] [PATCH 2/3] avcodec/utils: Fix several integer overflows.

2017-06-03 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/utils.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index cde5849a41..feee7556ac 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -2278,6 +2278,9 @@ void

[FFmpeg-devel] [PATCH 1/3] avcodec/utils: Avoid hardcoding duplicated types in sizeof()

2017-06-03 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/utils.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index 1336e921c9..cde5849a41 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -655,7 +655,7

[FFmpeg-devel] [PATCH 2/2] avformat/options: Simplify loglevel initialization in io_open_default()

2017-06-03 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/options.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/libavformat/options.c b/libavformat/options.c index d6d589a0b0..16e3886e4b 100644 --- a/libavformat/options.c +++

[FFmpeg-devel] [PATCH 1/2] avformat/options: Remove one special case in io_open_default()

2017-06-03 Thread Michael Niedermayer
This simplifies the code and reduces clutter. Signed-off-by: Michael Niedermayer --- libavformat/options.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libavformat/options.c b/libavformat/options.c index 9371c72667..d6d589a0b0 100644 ---

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/mlpdec: Check quant_step_size against huff_lsbs

2017-06-03 Thread Michael Niedermayer
On Mon, May 22, 2017 at 05:36:10PM +0200, Michael Niedermayer wrote: > On Sun, May 21, 2017 at 01:42:18PM +0200, wm4 wrote: > > On Sat, 20 May 2017 23:01:04 +0200 > > Michael Niedermayer wrote: > > > > > This reorders the operations so as to avoid computations with the

Re: [FFmpeg-devel] [PATCH] libavformat/aviobuf.c: don't treat 0 from read_packet as EOF

2017-06-03 Thread James Almer
On 6/3/2017 6:11 PM, Daniel Kučera wrote: > 2017-06-03 23:04 GMT+02:00 James Almer : >> On 6/3/2017 5:31 PM, Daniel Kučera wrote: >>> 2017-06-03 21:54 GMT+02:00 Michael Niedermayer : On Sat, Jun 03, 2017 at 10:16:59AM +0200, Daniel Kucera wrote:

Re: [FFmpeg-devel] [PATCH] avformat/options: log filename on open

2017-06-03 Thread Michael Niedermayer
On Fri, Jun 02, 2017 at 03:45:58PM +0200, wm4 wrote: > On Fri, 2 Jun 2017 15:05:27 +0200 > Michael Niedermayer wrote: > > > The loglevel is choosen so that the main filename and any images of > > multi image sequences are shown only at debug level to avoid > > clutter. >

Re: [FFmpeg-devel] [PATCH] avformat/aacdec: add a custom read_packet function

2017-06-03 Thread James Almer
On 6/3/2017 5:16 PM, Michael Niedermayer wrote: > On Sat, Jun 03, 2017 at 12:33:33AM -0300, James Almer wrote: >> Atempt to read and propagate only full ADTS frames and not other data, >> like id3v1 or APETags at the end of the file. >> >> Fixes ticket #6439. >> >> Signed-off-by: James Almer

Re: [FFmpeg-devel] [PATCH] libavformat/aviobuf.c: don't treat 0 from read_packet as EOF

2017-06-03 Thread Daniel Kučera
2017-06-03 23:04 GMT+02:00 James Almer : > On 6/3/2017 5:31 PM, Daniel Kučera wrote: >> 2017-06-03 21:54 GMT+02:00 Michael Niedermayer : >>> On Sat, Jun 03, 2017 at 10:16:59AM +0200, Daniel Kucera wrote: Signed-off-by: Daniel Kucera

Re: [FFmpeg-devel] [PATCH] libavformat/aviobuf.c: don't treat 0 from read_packet as EOF

2017-06-03 Thread James Almer
On 6/3/2017 5:31 PM, Daniel Kučera wrote: > 2017-06-03 21:54 GMT+02:00 Michael Niedermayer : >> On Sat, Jun 03, 2017 at 10:16:59AM +0200, Daniel Kucera wrote: >>> Signed-off-by: Daniel Kucera >>> --- >>> libavformat/avio.c| 2 +- >>>

[FFmpeg-devel] [PATCH] avfilter/af_surround: add support for some upmixing of 3.0 channel layout

2017-06-03 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- doc/filters.texi | 8 +- libavfilter/af_surround.c | 283 -- 2 files changed, 231 insertions(+), 60 deletions(-) diff --git a/doc/filters.texi b/doc/filters.texi index d4fbb5a..65eef89

[FFmpeg-devel] [PATCH] libavformat/aviobuf.c: don't treat 0 from read_packet as EOF

2017-06-03 Thread Daniel Kucera
Signed-off-by: Daniel Kucera --- libavformat/avio.c| 2 +- libavformat/aviobuf.c | 20 libavformat/cache.c | 2 +- libavformat/file.c| 2 ++ libavformat/subfile.c | 2 +- libavformat/wtvdec.c | 4 ++-- 6 files changed, 19

Re: [FFmpeg-devel] [PATCH] libavformat/aviobuf.c: don't treat 0 from read_packet as EOF

2017-06-03 Thread Daniel Kučera
2017-06-03 21:54 GMT+02:00 Michael Niedermayer : > On Sat, Jun 03, 2017 at 10:16:59AM +0200, Daniel Kucera wrote: >> Signed-off-by: Daniel Kucera >> --- >> libavformat/avio.c| 2 +- >> libavformat/aviobuf.c | 18 ++ >>

Re: [FFmpeg-devel] [PATCH] libavfilter/scale2ref: Fix out-of-bounds array access

2017-06-03 Thread Michael Niedermayer
On Sat, Jun 03, 2017 at 05:04:19AM -0400, Kevin Mark wrote: > ff_scale_eval_dimensions blindly assumes that two inputs are always > available as of 3385989b98be7940044e4f0a6b431a0a00abf2fa. This is > notably not the case when the function is called for the scale > filter. With the scale filter

Re: [FFmpeg-devel] [PATCH] avformat/aacdec: add a custom read_packet function

2017-06-03 Thread Michael Niedermayer
On Sat, Jun 03, 2017 at 12:33:33AM -0300, James Almer wrote: > Atempt to read and propagate only full ADTS frames and not other data, > like id3v1 or APETags at the end of the file. > > Fixes ticket #6439. > > Signed-off-by: James Almer > --- > libavformat/aacdec.c | 42

Re: [FFmpeg-devel] [PATCH] libavformat/aviobuf.c: don't treat 0 from read_packet as EOF

2017-06-03 Thread Michael Niedermayer
On Sat, Jun 03, 2017 at 10:16:59AM +0200, Daniel Kucera wrote: > Signed-off-by: Daniel Kucera > --- > libavformat/avio.c| 2 +- > libavformat/aviobuf.c | 18 ++ > libavformat/cache.c | 2 +- > libavformat/file.c| 2 ++ > libavformat/subfile.c

Re: [FFmpeg-devel] libavformat: deprecated AVStream member suggests usage of private field

2017-06-03 Thread wm4
On Sat, 3 Jun 2017 21:15:01 +0200 "Tom M." wrote: > This is pretty confusing. Those "All fields below this line are not part of > the public API." comments are completely ignored by doxygen. And this is > where most people look at, I doubt they parse the raw header files

Re: [FFmpeg-devel] libavformat: deprecated AVStream member suggests usage of private field

2017-06-03 Thread Hendrik Leppkes
On Sat, Jun 3, 2017 at 9:15 PM, Tom M. wrote: > This is pretty confusing. Those "All fields below this line are not part of > the public API." comments are completely ignored by doxygen. And this is > where most people look at, I doubt they parse the raw header files >

[FFmpeg-devel] [PATCH] avformat/hls: Check local file extensions

2017-06-03 Thread Michael Niedermayer
This reduces the attack surface of local file-system information leaking. It prevents the existing exploit leading to an information leak. As well as similar hypothetical attacks. Leaks of information from files and symlinks ending in common multimedia extensions are still possible. But files

Re: [FFmpeg-devel] libavformat: deprecated AVStream member suggests usage of private field

2017-06-03 Thread Tom M.
This is pretty confusing. Those "All fields below this line are not part of the public API." comments are completely ignored by doxygen. And this is where most people look at, I doubt they parse the raw header files themselves. Wouldnt it be possible to move these private fields into the internal

Re: [FFmpeg-devel] [PATCH 1/2] avformat/matroskaenc: also write chapters when output is WebM

2017-06-03 Thread James Almer
On 6/2/2017 12:11 AM, James Zern wrote: > On Thu, Jun 1, 2017 at 7:51 PM, James Almer wrote: >> WebM supports a subset of elements from the Chapters master. >> See https://www.webmproject.org/docs/container/#chapters >> >> Addresses ticket #6425 >> >> Signed-off-by: James Almer

Re: [FFmpeg-devel] libavformat: deprecated AVStream member suggests usage of private field

2017-06-03 Thread Hendrik Leppkes
On Sat, Jun 3, 2017 at 6:51 PM, Tom M. wrote: > I noticed that > > AVCodecContext *codec; > > within AVStream > has been > marked deprecated and suggests the usage of > > AVCodecParameters *codecpar; > >

Re: [FFmpeg-devel] [PATCH] libavformat/aviobuf.c: don't treat 0 from read_packet as EOF

2017-06-03 Thread Michael Niedermayer
On Sat, Jun 03, 2017 at 10:16:59AM +0200, Daniel Kucera wrote: > Signed-off-by: Daniel Kucera > --- > libavformat/avio.c| 2 +- > libavformat/aviobuf.c | 18 ++ > libavformat/cache.c | 2 +- > libavformat/file.c| 2 ++ > libavformat/subfile.c

Re: [FFmpeg-devel] [PATCH] compat: LoadLibrary isn't available on UWP/WinRT

2017-06-03 Thread Aaron Levinson
On 6/2/2017 7:11 AM, wm4 wrote: On Fri, 2 Jun 2017 15:29:07 +0200 Hugo Beauzée-Luyssen wrote: --- compat/w32dlfcn.h | 4 1 file changed, 4 insertions(+) diff --git a/compat/w32dlfcn.h b/compat/w32dlfcn.h index bc9bb8c9f5..308763be53 100644 --- a/compat/w32dlfcn.h

Re: [FFmpeg-devel] [PATCH] avformat/mov: add support for reading VP Codec Configuration Box

2017-06-03 Thread James Almer
On 5/27/2017 7:00 PM, James Almer wrote: > As defined in "VP Codec ISO Media File Format Binding v1.0" > https://github.com/webmproject/vp9-dash/blob/master/VPCodecISOMediaFileFormatBinding.md > > Signed-off-by: James Almer > --- > libavformat/mov.c | 45

Re: [FFmpeg-devel] [PATCH] avformat/oggenc: add ogg_init() and ogg_free()

2017-06-03 Thread James Almer
On 5/28/2017 5:09 PM, James Almer wrote: > Signed-off-by: James Almer > --- > libavformat/oggenc.c | 32 > 1 file changed, 28 insertions(+), 4 deletions(-) > Ping for this and the aiff one. I have a set of about 50 patches to do the same

Re: [FFmpeg-devel] [PATCH 1/6] initial alignment corrections for xmm registers

2017-06-03 Thread Michael Niedermayer
On Sat, Jun 03, 2017 at 02:18:04AM +0200, James Darnley wrote: > --- > libavcodec/x86/simple_idct.asm | 47 > ++ > 1 file changed, 34 insertions(+), 13 deletions(-) should be ok minor cosmetic misalignment below: [...] > @@ -582,7 +603,7 @@ SECTION

Re: [FFmpeg-devel] [PATCH 2/2] avformat/utils: change bitrate to int64_t in av_find_best_stream

2017-06-03 Thread Michael Niedermayer
On Fri, Jun 02, 2017 at 11:10:14PM +0200, Marton Balint wrote: > Signed-off-by: Marton Balint > --- > libavformat/utils.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/libavformat/utils.c b/libavformat/utils.c > index b544b6f9c5..aa2a00552e

Re: [FFmpeg-devel] [PATCH 1/2] avformat/utils: return impaired streams in av_find_best_stream if only those exist

2017-06-03 Thread Michael Niedermayer
On Fri, Jun 02, 2017 at 11:10:13PM +0200, Marton Balint wrote: > Fixes ticket #6397. > > Signed-off-by: Marton Balint > --- > libavformat/utils.c | 17 + > 1 file changed, 9 insertions(+), 8 deletions(-) LGTM can you add a fate test for this ? thx [...] --

[FFmpeg-devel] [PATCH] avcodec/aacps: move checks for valid length outside the stereo_interpolate dsp function

2017-06-03 Thread James Almer
Signed-off-by: James Almer --- libavcodec/aacps.c | 7 --- libavcodec/x86/aacpsdsp.asm | 6 -- 2 files changed, 4 insertions(+), 9 deletions(-) diff --git a/libavcodec/aacps.c b/libavcodec/aacps.c index 29de8a5ea7..473da7bd43 100644 --- a/libavcodec/aacps.c

[FFmpeg-devel] libavformat: deprecated AVStream member suggests usage of private field

2017-06-03 Thread Tom M.
I noticed that AVCodecContext *codec; within AVStream has been marked deprecated and suggests the usage of AVCodecParameters *codecpar; However, due to avformat.h:1001

[FFmpeg-devel] [PATCH] libavformat/mov.c Fixed 'Error while filtering: Operation not permitted' when using -stream_loop, a short file with negativ timestamps and only one keyframe Fixes Ticket #6139

2017-06-03 Thread Jonas Licht
--- libavformat/mov.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 3845e63b53..afc2efe063 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -6547,9 +6547,9 @@ static int mov_seek_stream(AVFormatContext *s, AVStream

Re: [FFmpeg-devel] [PATCH] avfilter/af_sofalizer: switch to libmysofa

2017-06-03 Thread Paul B Mahol
On 6/3/17, wm4 wrote: > On Sat, 3 Jun 2017 14:42:59 +0200 > Paul B Mahol wrote: > >> Signed-off-by: Paul B Mahol >> --- > > And libmysofa can't be NIH'ed? Why? ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] ABI break between 3.0 and 3.3 without symvers/SONAME changes

2017-06-03 Thread Jan Engelhardt
On Saturday 2017-06-03 14:27, Timo Rothenpieler wrote: > > It is made sure that the public ABI does not break between major releases, > this > the major version is not bumped. > The problem in practice is that _a lot_ of downstream library users misuse the > API, using non-public fields and what

Re: [FFmpeg-devel] [PATCH] avformat/hls: Check file extensions

2017-06-03 Thread Hendrik Leppkes
On Sat, Jun 3, 2017 at 2:58 PM, Michael Niedermayer wrote: > On Sat, Jun 03, 2017 at 11:18:46AM +0200, Hendrik Leppkes wrote: >> On Sat, Jun 3, 2017 at 2:31 AM, Michael Niedermayer >> wrote: >> > On Fri, Jun 02, 2017 at 09:27:16PM +0200, Hendrik

Re: [FFmpeg-devel] [PATCH] avformat/hls: Check file extensions

2017-06-03 Thread wm4
On Sat, 3 Jun 2017 14:58:19 +0200 Michael Niedermayer wrote: > Do you object to fixing this security issue that has a working exploit? > Can you provide a testcase the fix breaks ? So i can look into what > can be done about it ? (multiple testcases would be even better

[FFmpeg-devel] [PATCH] avformat/hls: Check file extensions

2017-06-03 Thread Michael Niedermayer
This reduces the attack surface of local file-system and local network information leaking. It prevents the existing exploit leading to an information leak. As well as similar hypothetical attacks. Leaks of information from files and symlinks ending in common multimedia extensions are still

Re: [FFmpeg-devel] [PATCH] avformat/hls: Check file extensions

2017-06-03 Thread Michael Niedermayer
On Sat, Jun 03, 2017 at 11:18:46AM +0200, Hendrik Leppkes wrote: > On Sat, Jun 3, 2017 at 2:31 AM, Michael Niedermayer > wrote: > > On Fri, Jun 02, 2017 at 09:27:16PM +0200, Hendrik Leppkes wrote: > >> On Fri, Jun 2, 2017 at 9:19 PM, Michael Niedermayer > >>

Re: [FFmpeg-devel] [PATCH] avfilter/af_sofalizer: switch to libmysofa

2017-06-03 Thread wm4
On Sat, 3 Jun 2017 14:42:59 +0200 Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- And libmysofa can't be NIH'ed? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

[FFmpeg-devel] [PATCH] avfilter/af_sofalizer: switch to libmysofa

2017-06-03 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- configure | 8 +- doc/filters.texi | 2 +- libavfilter/af_sofalizer.c | 700 - 3 files changed, 194 insertions(+), 516 deletions(-) diff --git a/configure b/configure

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Offset index timestamps by the minimum pts to make first pts zero.

2017-06-03 Thread wm4
On Fri, 2 Jun 2017 18:17:22 -0700 Sasi Inguva wrote: > Fixes t/6421 > > Signed-off-by: Sasi Inguva > --- > libavformat/mov.c| 57 -- > tests/ref/fate/h264-twofields-packet | 60 >

Re: [FFmpeg-devel] ABI break between 3.0 and 3.3 without symvers/SONAME changes

2017-06-03 Thread wm4
On Sat, 3 Jun 2017 14:31:31 +0200 (CEST) Jan Engelhardt wrote: > On Saturday 2017-06-03 14:27, Timo Rothenpieler wrote: > > > >> could ffmpeg then ensure that the major is in fact bumped on major > >> releases? In distro context, a bump is easier to swallow than a > >>

Re: [FFmpeg-devel] ABI break between 3.0 and 3.3 without symvers/SONAME changes

2017-06-03 Thread Jan Engelhardt
On Saturday 2017-06-03 14:27, Timo Rothenpieler wrote: > >> could ffmpeg then ensure that the major is in fact bumped on major >> releases? In distro context, a bump is easier to swallow than a >> non-bump.. > > It is made sure that the public ABI does not break between major > releases, this the

Re: [FFmpeg-devel] ABI break between 3.0 and 3.3 without symvers/SONAME changes

2017-06-03 Thread wm4
On Sat, 3 Jun 2017 14:19:25 +0200 (CEST) Jan Engelhardt wrote: > >Anyway, in general, I recommend not pretending that FFmpeg has ABI > >compatibility. Especially not across major releases > > That would be fine with me - could ffmpeg then ensure that the > major is in fact

Re: [FFmpeg-devel] ABI break between 3.0 and 3.3 without symvers/SONAME changes

2017-06-03 Thread Timo Rothenpieler
Am 03.06.2017 um 14:19 schrieb Jan Engelhardt: On Saturday 2017-06-03 14:11, wm4 wrote: The layout of struct AVCodec changed. Four function pointers were inserted in the middle, shifting void (*flush) to a new address. The size of AVCodec also changed, which would have required a full MAJOR

Re: [FFmpeg-devel] ABI break between 3.0 and 3.3 without symvers/SONAME changes

2017-06-03 Thread Jan Engelhardt
On Saturday 2017-06-03 14:11, wm4 wrote: >> >> The layout of struct AVCodec changed. Four function pointers were >> inserted in the middle, shifting void (*flush) to a new address. The >> size of AVCodec also changed, which would have required a full MAJOR >> bump. > >Those aren't even public

Re: [FFmpeg-devel] ABI break between 3.0 and 3.3 without symvers/SONAME changes

2017-06-03 Thread Hendrik Leppkes
On Sat, Jun 3, 2017 at 2:07 PM, Jan Engelhardt wrote: > > On Saturday 2017-06-03 14:02, Hendrik Leppkes wrote: > >>On Sat, Jun 3, 2017 at 1:58 PM, Jan Engelhardt wrote: >>> >>> On Saturday 2017-06-03 13:34, Hendrik Leppkes wrote: > > Fixing the .v file

Re: [FFmpeg-devel] ABI break between 3.0 and 3.3 without symvers/SONAME changes

2017-06-03 Thread Timo Rothenpieler
Size changes are breakers in both directions. The size of the structs are not part of the public API/ABI. Only the location of public fields in them is. You claim your ABI is stable in one direction, which means that a program built with 3.0 headers should work with a 3.3 library on a

Re: [FFmpeg-devel] ABI break between 3.0 and 3.3 without symvers/SONAME changes

2017-06-03 Thread Hendrik Leppkes
On Sat, Jun 3, 2017 at 2:07 PM, Jan Engelhardt wrote: > > On Saturday 2017-06-03 13:34, Hendrik Leppkes wrote: >>On Sat, Jun 3, 2017 at 1:18 PM, Jan Engelhardt wrote: >>> >>>The layout of struct AVCodec changed. Four function pointers were >>>inserted in the

Re: [FFmpeg-devel] ABI break between 3.0 and 3.3 without symvers/SONAME changes

2017-06-03 Thread wm4
On Sat, 3 Jun 2017 14:07:30 +0200 (CEST) Jan Engelhardt wrote: > On Saturday 2017-06-03 13:34, Hendrik Leppkes wrote: > >On Sat, Jun 3, 2017 at 1:18 PM, Jan Engelhardt wrote: > >> > >>The layout of struct AVCodec changed. Four function pointers were >

Re: [FFmpeg-devel] ABI break between 3.0 and 3.3 without symvers/SONAME changes

2017-06-03 Thread wm4
On Sat, 3 Jun 2017 13:18:30 +0200 (CEST) Jan Engelhardt wrote: > > The layout of struct AVCodec changed. Four function pointers were > inserted in the middle, shifting void (*flush) to a new address. The > size of AVCodec also changed, which would have required a full MAJOR

Re: [FFmpeg-devel] ABI break between 3.0 and 3.3 without symvers/SONAME changes

2017-06-03 Thread Jan Engelhardt
On Saturday 2017-06-03 13:34, Hendrik Leppkes wrote: >On Sat, Jun 3, 2017 at 1:18 PM, Jan Engelhardt wrote: >> >>The layout of struct AVCodec changed. Four function pointers were >>inserted in the middle, shifting void (*flush) to a new address. >>The size of AVCodec also

Re: [FFmpeg-devel] ABI break between 3.0 and 3.3 without symvers/SONAME changes

2017-06-03 Thread Jan Engelhardt
On Saturday 2017-06-03 14:02, Hendrik Leppkes wrote: >On Sat, Jun 3, 2017 at 1:58 PM, Jan Engelhardt wrote: >> >> On Saturday 2017-06-03 13:34, Hendrik Leppkes wrote: Fixing the .v file with a patch now won't help remedy the situation on existing systems;

Re: [FFmpeg-devel] ABI break between 3.0 and 3.3 without symvers/SONAME changes

2017-06-03 Thread Hendrik Leppkes
On Sat, Jun 3, 2017 at 1:58 PM, Jan Engelhardt wrote: > > On Saturday 2017-06-03 13:34, Hendrik Leppkes wrote: >>> >>> Fixing the .v file with a patch now won't help remedy the situation on >>> existing systems; however, on future patch submissions, this action of >>> editing the

Re: [FFmpeg-devel] ABI break between 3.0 and 3.3 without symvers/SONAME changes

2017-06-03 Thread Jan Engelhardt
On Saturday 2017-06-03 13:34, Hendrik Leppkes wrote: >> >> Fixing the .v file with a patch now won't help remedy the situation on >> existing systems; however, on future patch submissions, this action of >> editing the .v file needs to be remembered when functions are >> added or removed. > >We

Re: [FFmpeg-devel] Backport HLS Cookie Authentication Fixes to 3.3

2017-06-03 Thread Micah Galizia
On 2017-06-01 09:04 PM, Michael Niedermayer wrote: On Tue, May 30, 2017 at 09:29:53PM -0400, Micah Galizia wrote: Hello, I'd like to backport the HLS cookie authentication fixes to 3.3 so that Kodi 18 picks them up. Originally, this was a three patch set but the first fix (to

Re: [FFmpeg-devel] ABI break between 3.0 and 3.3 without symvers/SONAME changes

2017-06-03 Thread Hendrik Leppkes
On Sat, Jun 3, 2017 at 1:18 PM, Jan Engelhardt wrote: > > I received a user report about ABI breakage between ffmpeg 3.0 and 3.3 > through the distribution commenting system > https://build.opensuse.org/package/show/multimedia:libs/ffmpeg : > > """I'm having a library

[FFmpeg-devel] ABI break between 3.0 and 3.3 without symvers/SONAME changes

2017-06-03 Thread Jan Engelhardt
I received a user report about ABI breakage between ffmpeg 3.0 and 3.3 through the distribution commenting system https://build.opensuse.org/package/show/multimedia:libs/ffmpeg : """I'm having a library compiled against ffmpeg 3.3 on OBS. It uses the

[FFmpeg-devel] [PATCH] web - src/contact - added link to SuperUser forum

2017-06-03 Thread Gyan
The support forum at http://ffmpeg.gusari.org/index.php which is linked to at http://www.ffmpeg.org/contact.html#Forums now sports a banner announcing its likely closure. The banner redirects visitors to Stack Exchange which already receives most ffmpeg support queries. The patch below adds a

[FFmpeg-devel] [PATCH] libavfilter/scale2ref: Fix out-of-bounds array access

2017-06-03 Thread Kevin Mark
ff_scale_eval_dimensions blindly assumes that two inputs are always available as of 3385989b98be7940044e4f0a6b431a0a00abf2fa. This is notably not the case when the function is called for the scale filter. With the scale filter inputs[1] does not exist. ff_scale_eval_dimensions now has an updated

Re: [FFmpeg-devel] [PATCH] libavfilter/scale2ref: Add constants for the primary input

2017-06-03 Thread Kevin Mark
On Fri, Jun 2, 2017 at 9:37 PM, Kevin Mark wrote: > On Fri, Jun 2, 2017 at 9:16 PM, James Almer wrote: >> This broke 820 tests with Valgrind. Patch has been submitted. Thanks again for the report! ___

Re: [FFmpeg-devel] [PATCH] avformat/hls: Check file extensions

2017-06-03 Thread Hendrik Leppkes
On Sat, Jun 3, 2017 at 2:31 AM, Michael Niedermayer wrote: > On Fri, Jun 02, 2017 at 09:27:16PM +0200, Hendrik Leppkes wrote: >> On Fri, Jun 2, 2017 at 9:19 PM, Michael Niedermayer >> wrote: >> > This reduces the attack surface of local file-system

[FFmpeg-devel] [PATCH] libavformat/aviobuf.c: don't treat 0 from read_packet as EOF

2017-06-03 Thread Daniel Kucera
Signed-off-by: Daniel Kucera --- libavformat/avio.c| 2 +- libavformat/aviobuf.c | 18 ++ libavformat/cache.c | 2 +- libavformat/file.c| 2 ++ libavformat/subfile.c | 2 +- libavformat/wtvdec.c | 4 ++-- 6 files changed, 17 insertions(+),

Re: [FFmpeg-devel] [PATCH] libavformat/aviobuf.c: don't treat 0 from read_packet as EOF

2017-06-03 Thread Daniel Kučera
2017-06-03 8:58 GMT+02:00 Daniel Kučera : >> >> breaks: >> ffmpeg -i subfile,,start,1,end,40,,:matrixbench_mpeg2.mpg >> subfile.avi >> >> (infnite loop) >> >> > > Where can I get that matrixbench_mpeg2.mpg test file? If I run it > against

Re: [FFmpeg-devel] [PATCH] libavformat/aviobuf.c: don't treat 0 from read_packet as EOF

2017-06-03 Thread Daniel Kučera
> > breaks: > ffmpeg -i subfile,,start,1,end,40,,:matrixbench_mpeg2.mpg subfile.avi > > (infnite loop) > > Where can I get that matrixbench_mpeg2.mpg test file? If I run it against fate-suite/mpeg2/matrixbench_mpeg2.lq1.mpg it works: ./ffmpeg -i

Re: [FFmpeg-devel] [PATCH] cmdutils: add log time info into report log file

2017-06-03 Thread Steven Liu
2017-06-03 0:30 GMT+08:00 Nicolas George : > > Le quintidi 15 prairial, an CCXXV, Steven Liu a écrit : > > add time info into every line of log report > > the time info can be used to find out error message occur time. > > > > Signed-off-by: Steven Liu >