[FFmpeg-devel] [PATCH 1/1] Add null check for the frame pointer

2018-06-25 Thread Baozeng
0001-Add-null-check-for-the-frame-pointer.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/1] Add null check for the frame pointer

2018-06-25 Thread Nicolas George
Baozeng (2018-06-25): > The argument `frame` for avcodec_receive_frame function is user > controlled. If a null pointer is passed from the user, it will cause a > segmentation fault. Nack. Passing NULL there is a bug by the user, it needs to be fixed by the user. This patch would make it harder to

Re: [FFmpeg-devel] [PATCH] lavfi: add sobel, prewitt, roberts filters

2018-06-25 Thread Marton Balint
On Mon, 25 Jun 2018, Danil Iashchenko wrote: Add opencl version of sobel, prewitt, roberts filters. --- configure | 3 + libavfilter/Makefile| 8 +- libavfilter/allfilters.c| 3 + libavfilter/opencl/convolution.cl | 82 ++ liba

[FFmpeg-devel] [PATCH 1/1] Add null check for the frame pointer

2018-06-25 Thread Baozeng Ding
Signed-off-by: Baozeng Ding The argument `frame` for avcodec_receive_frame function is user controlled. If a null pointer is passed from the user, it will cause a segmentation fault. --- libavcodec/decode.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/decode.c b/libavcodec/deco

[FFmpeg-devel] [PATCH 1/1] Add null check for the frame pointer

2018-06-25 Thread Baozeng Ding
Signed-off-by: Baozeng Ding The argument `frame` for avcodec_receive_frame function is user controlled. If a null pointer is passed from the user, it will cause a segmentation fault. --- libavcodec/decode.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/decode.c b/libavcodec/deco

Re: [FFmpeg-devel] [PATCH 5/5] fate: Add test for namespace prefixes in libraries

2018-06-25 Thread Michael Niedermayer
On Sun, Jun 24, 2018 at 07:25:39PM +0100, Mark Thompson wrote: > Ensures that there are no external symbols in the built static libraries > which do not have suitable namespace prefixes. > --- > Not sure whether this will actually work on all platforms? (Only tested with > Linux/binutils.) fails

Re: [FFmpeg-devel] [PATCH 4/5] lavf/mpjpegdec: Mark local variable static

2018-06-25 Thread Michael Niedermayer
On Sun, Jun 24, 2018 at 07:25:38PM +0100, Mark Thompson wrote: > --- > libavformat/mpjpegdec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Old school: Use the lowest level language in which y

Re: [FFmpeg-devel] [PATCH 3/3] avformat/mov: Expose encryption info to the app.

2018-06-25 Thread Jacob Trimble
Rebased and updated to handle multiple PSSH atoms. PTAL. On Tue, Jan 9, 2018 at 10:28 AM Jacob Trimble wrote: > On Mon, Jan 8, 2018 at 5:23 PM, Carl Eugen Hoyos > wrote: > > 2018-01-09 0:22 GMT+01:00 Jacob Trimble < > modmaker-at-google@ffmpeg.org>: > > > >> Updated with the new design for

Re: [FFmpeg-devel] [PATCH] libavutil/encryption_info: Add unit tests.

2018-06-25 Thread Jacob Trimble
Now that the patches have been merged this doesn't depend on anything. On Fri, Jun 1, 2018 at 12:54 PM Jacob Trimble wrote: > On Fri, Jun 1, 2018 at 12:52 PM Jacob Trimble wrote: > > > > Signed-off-by: Jacob Trimble > > --- > > libavutil/Makefile| 1 + > > libavutil/encrypti

[FFmpeg-devel] [PATCH] lavd/dshow: Add namespace prefix to global functions

2018-06-25 Thread Mark Thompson
--- Not tested at all. libavdevice/dshow.c | 6 +++--- libavdevice/dshow_capture.h | 4 ++-- libavdevice/dshow_crossbar.c | 8 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a/libavdevice/dshow.c b/libavdevice/dshow.c index de910c0994..6f613f31b0 100644 --- a/liba

Re: [FFmpeg-devel] [PATCH 5/5] fate: Add test for namespace prefixes in libraries

2018-06-25 Thread Mark Thompson
On 25/06/18 18:41, Michael Niedermayer wrote: > On Sun, Jun 24, 2018 at 07:25:39PM +0100, Mark Thompson wrote: >> Ensures that there are no external symbols in the built static libraries >> which do not have suitable namespace prefixes. >> --- >> Not sure whether this will actually work on all plat

Re: [FFmpeg-devel] [PATCH] libavformat/yuv4mpeg: Add color range support for Y4M Add color_range support in Y4M. Also set pixel format and color_range for YUVJ pixel formats.

2018-06-25 Thread Sasi Inguva
Friendly ping! Thx. On Fri, Jun 22, 2018 at 6:25 AM Carl Eugen Hoyos wrote: > 2018-06-22 9:29 GMT+02:00, Wang Cao : > >> > >> My question was which authority defined this metadata for > >> y4m? Or which (non-FFmpeg-based) other software > >> understands the metadata your patch adds to the files.

Re: [FFmpeg-devel] [PATCH 4/5] lavf/mpjpegdec: Mark local variable static

2018-06-25 Thread Mark Thompson
On 25/06/18 18:57, Michael Niedermayer wrote: > On Sun, Jun 24, 2018 at 07:25:38PM +0100, Mark Thompson wrote: >> --- >> libavformat/mpjpegdec.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > > LGTM Applied. Thanks, - Mark ___ ffmpeg-deve

Re: [FFmpeg-devel] [PATCH] avcodec/cscd: Check output buffer size for lzo.

2018-06-25 Thread Michael Niedermayer
On Fri, Jun 22, 2018 at 02:38:02AM +0200, Michael Niedermayer wrote: > Fixes: Timeout > Fixes: > 8665/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_CSCD_fuzzer-5768442610188288 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > --- >

Re: [FFmpeg-devel] [PATCH 2/5] avcodec/ffv1dec: Check state transition table

2018-06-25 Thread Michael Niedermayer
On Fri, Jun 22, 2018 at 12:15:14AM +0200, Michael Niedermayer wrote: > Fixes: Timeout > Fixes: > 8646/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_FFV1_fuzzer-5649968353247232 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed-

Re: [FFmpeg-devel] [PATCH] avcodec/escape124: Check buf_size against num_superblocks

2018-06-25 Thread Michael Niedermayer
On Mon, Jun 25, 2018 at 02:33:12AM +0200, Michael Niedermayer wrote: > Fixes: Timeout > Fixes: > 8722/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ESCAPE124_fuzzer-4843268402577408 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Si

[FFmpeg-devel] [PATCH] lavfi/colorspace: Add namespace prefix to global functions

2018-06-25 Thread Mark Thompson
--- On 25/06/18 02:34, Song, Ruiling wrote: >> -Original Message- >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of >> Mark Thompson >> Sent: Monday, June 25, 2018 2:26 AM >> To: ffmpeg-devel@ffmpeg.org >> Subject: [FFmpeg-devel] [PATCH 1/5] lavfi/colorspace: Add n

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavf/amr: Make the heuristic for auto-detection even stricter.

2018-06-25 Thread Carl Eugen Hoyos
2018-06-23 14:43 GMT+02:00, Michael Niedermayer : > On Thu, Jun 21, 2018 at 10:09:14PM +, Carl Eugen Hoyos wrote: >> ffmpeg | branch: master | Carl Eugen Hoyos | Fri Jun >> 22 00:08:13 2018 +0200| [40b7e6071815fc416a4efc5dc1616f5460a3aacb] | >> committer: Carl Eugen Hoyos >> >> lavf/amr: Make

Re: [FFmpeg-devel] [PATCH v2 16/36] vaapi_encode: Clean up rate control configuration

2018-06-25 Thread Mark Thompson
On 21/06/18 18:03, Michael Niedermayer wrote: > On Thu, Jun 21, 2018 at 12:10:04AM +0100, Mark Thompson wrote: >> On 20/06/18 10:44, Li, Zhong wrote: -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of Mark Thompson Sent: Sunday, J

Re: [FFmpeg-devel] [PATCH]lavc/libopenjpeg: Support GRAY10, GRAY12 and GRAY14

2018-06-25 Thread Carl Eugen Hoyos
2018-06-22 15:34 GMT+02:00, Michael Bradshaw : > Hey, Carl Eugen! Thanks for the patch. > > On Thu, Jun 21, 2018 at 3:12 AM, Carl Eugen Hoyos > wrote: >> >> Attached patch allows to create (and read) gray1x samples with >> libopenjpeg. > > > Looks good to me. I have a few very small nits below. I

[FFmpeg-devel] [PATCH] avformat/avformat.h: Add av_stream_remove_side_data.

2018-06-25 Thread Jacob Trimble
Signed-off-by: Jacob Trimble --- libavformat/avformat.h | 8 libavformat/utils.c| 11 +++ 2 files changed, 19 insertions(+) diff --git a/libavformat/avformat.h b/libavformat/avformat.h index fdaffa5bf4..434c88837e 100644 --- a/libavformat/avformat.h +++ b/libavformat/avform

Re: [FFmpeg-devel] [PATCH] libavformat/yuv4mpeg: Add color range support for Y4M Add color_range support in Y4M. Also set pixel format and color_range for YUVJ pixel formats.

2018-06-25 Thread Michael Niedermayer
On Thu, Jun 14, 2018 at 12:03:13AM +0800, Wang Cao wrote: > --- > libavformat/yuv4mpegdec.c | 8 > libavformat/yuv4mpegenc.c | 37 +++-- > 2 files changed, 43 insertions(+), 2 deletions(-) > > diff --git a/libavformat/yuv4mpegdec.c b/libavformat/yuv4mpegd

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavf/amr: Make the heuristic for auto-detection even stricter.

2018-06-25 Thread Michael Niedermayer
On Mon, Jun 25, 2018 at 11:22:31PM +0200, Carl Eugen Hoyos wrote: > 2018-06-23 14:43 GMT+02:00, Michael Niedermayer : > > On Thu, Jun 21, 2018 at 10:09:14PM +, Carl Eugen Hoyos wrote: > >> ffmpeg | branch: master | Carl Eugen Hoyos | Fri Jun > >> 22 00:08:13 2018 +0200| [40b7e6071815fc416a4efc

Re: [FFmpeg-devel] [PATCH] libavutil/encryption_info: Add unit tests.

2018-06-25 Thread Michael Niedermayer
On Fri, Jun 01, 2018 at 12:51:48PM -0700, Jacob Trimble wrote: > Signed-off-by: Jacob Trimble > --- > libavutil/Makefile| 1 + > libavutil/encryption_info.h | 2 +- > libavutil/tests/.gitignore| 2 + > libavutil/tests/encryption_info.c | 176 +++

Re: [FFmpeg-devel] [PATCH] lavfi/colorspace: Add namespace prefix to global functions

2018-06-25 Thread Song, Ruiling
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > Mark Thompson > Sent: Tuesday, June 26, 2018 5:02 AM > To: ffmpeg-devel@ffmpeg.org > Subject: [FFmpeg-devel] [PATCH] lavfi/colorspace: Add namespace prefix to > global functions > > --- > On

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: check sseof value and clash with ss

2018-06-25 Thread Gyan Doshi
On 25-06-2018 10:07 AM, Gyan Doshi wrote: Revised. Will push soon. Thanks, Gyan Pushed in a0ac49e38ee1d1011c394d7be67d0f08b2281526 ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavfi: add opencl tonemap filter

2018-06-25 Thread Song, Ruiling
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > Michael Niedermayer > Sent: Friday, June 22, 2018 2:32 AM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [FFmpeg-cvslog] lavfi: add opencl tonemap filter > > On Thu, Jun 21, 201