[FFmpeg-devel] [PATCH] avformat/dashenc: Support HTTP Persistent for master.mu8 as well

2018-10-17 Thread Karthick J
--- libavformat/dashenc.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index 3f5f290e25..15b84a0f3b 100644 --- a/libavformat/dashenc.c +++ b/libavformat/dashenc.c @@ -878,14 +878,14 @@ static int write_manifest(AVForm

[FFmpeg-devel] [PATCH v2] avformat/dashenc: Support HTTP Persistent for master.m3u8 as well

2018-10-17 Thread Karthick J
--- libavformat/dashenc.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index 3f5f290e25..15b84a0f3b 100644 --- a/libavformat/dashenc.c +++ b/libavformat/dashenc.c @@ -878,14 +878,14 @@ static int write_manifest(AVForm

[FFmpeg-devel] [PATCH] avformat/dashenc: Support HTTP Persistent for master.mu8 as well

2018-10-17 Thread Karthick J
--- libavformat/dashenc.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index 3f5f290e25..15b84a0f3b 100644 --- a/libavformat/dashenc.c +++ b/libavformat/dashenc.c @@ -878,14 +878,14 @@ static int write_manifest(AVForm

Re: [FFmpeg-devel] [PATCH] avformat/dashenc: Dont ignore the codec tag from codec parameters

2018-10-17 Thread Jeyapal, Karthick
>On 10/10/18, 4:55 PM, "Jeyapal, Karthick" wrote: > >My git send-email is not working. Hence attaching the patch. Patch Pushed. > >Regards, >Karthick ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec/h264_cavlc: Check mb_skip_run

2018-10-17 Thread Michael Niedermayer
On Fri, Oct 05, 2018 at 03:31:44AM +0200, Michael Niedermayer wrote: > Fixes: > 10300/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_H264_fuzzer-6292205497483264 > Fixes: signed integer overflow: -2147483648 - 1 cannot be represented in type > 'int' > > Found-by: continuous fuzzing process >

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/ra144: Fix integer overflow in add_wav()

2018-10-17 Thread Michael Niedermayer
On Thu, Oct 11, 2018 at 01:45:29AM +0200, Michael Niedermayer wrote: > Fixes: signed integer overflow: -2144033225 + -5208934 cannot be represented > in type 'int' > Fixes: > 10633/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_RA_144_fuzzer-5679133791617024 > > Found-by: continuous fuzzing p

Re: [FFmpeg-devel] [PATCH] avformat/utils: Never store negative values in last_IP_duration

2018-10-17 Thread Michael Niedermayer
On Sat, Oct 13, 2018 at 03:13:10AM +0200, Michael Niedermayer wrote: > Fixes: integer overflow compute_pkt_fields() > Fixes: compute_pkt_usan > > Reported-by: Thomas Guilbert > Signed-off-by: Michael Niedermayer > --- > libavformat/utils.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletio

Re: [FFmpeg-devel] [PATCH 1/2] avformat/utils: Fix integer overflow in discontinuity check

2018-10-17 Thread Michael Niedermayer
On Fri, Oct 12, 2018 at 03:40:20AM +0200, Michael Niedermayer wrote: > Fixes: signed integer overflow: 7738135736989908991 - -7954308516317364223 > cannot be represented in type 'long' > Fixes: find_stream_info_usan > > Reported-by: Thomas Guilbert > Signed-off-by: Michael Niedermayer > --- >

[FFmpeg-devel] [PATCH] avcodec/mpeg4videodec: Fix typo in sprite delta check

2018-10-17 Thread Michael Niedermayer
Fixes: Integer overflow Fixes: 10890/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_MPEG4_fuzzer-5636062181851136 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/mpeg4videodec.c | 2 +- 1 f

[FFmpeg-devel] [PATCH v2] web/documentation: add new book about FFmpeg in China

2018-10-17 Thread Steven Liu
Signed-off-by: Steven Liu --- src/documentation | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/documentation b/src/documentation index debca83..69b02f2 100644 --- a/src/documentation +++ b/src/documentation @@ -130,6 +130,8 @@ http://ffmpeg.tv";>FFmpeg Basics by Franti

Re: [FFmpeg-devel] [PATCH]lavc/mjpegdec: Support 2:3 subsampling

2018-10-17 Thread Carl Eugen Hoyos
2018-10-17 22:12 GMT+02:00, Michael Niedermayer : > On Wed, Oct 17, 2018 at 09:01:30PM +0200, Carl Eugen Hoyos wrote: >> 2018-10-17 17:51 GMT+02:00, Michael Niedermayer : >> > On Wed, Oct 17, 2018 at 12:22:51AM +0200, Carl Eugen Hoyos wrote: >> >> Hi! >> >> >> >> Attached patch allows decoding the

Re: [FFmpeg-devel] [PATCH V2] Add a filter implementing HDR image reconstruction from a single exposure using deep CNNs

2018-10-17 Thread Carl Eugen Hoyos
2018-10-17 18:41 GMT+02:00, Guo, Yejun : > +short* outg = (short*)out->data[0]; > +short* outb = (short*)out->data[1]; > +short* outr = (short*)out->data[2]; I believe this should use "int16_t", there is no guarantee that short is smaller than 128 bit. Carl Eugen

[FFmpeg-devel] [PATCH]lavf/dump: Fix a typo: comentary -> commentary

2018-10-17 Thread Carl Eugen Hoyos
Hi! Is there a reason to keep the wrong spelling? Please comment, Carl Eugen From 80a6b2f5d4a61e8a1fd198adfdd059c2d6bce1fa Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Thu, 18 Oct 2018 00:13:38 +0200 Subject: [PATCH] lavf/dump: Fix a typo: comentary -> commentary. Fixes ticket #7499. -

Re: [FFmpeg-devel] [PATCH]lavc/mjpegdec: Support 2:3 subsampling

2018-10-17 Thread Michael Niedermayer
On Wed, Oct 17, 2018 at 09:01:30PM +0200, Carl Eugen Hoyos wrote: > 2018-10-17 17:51 GMT+02:00, Michael Niedermayer : > > On Wed, Oct 17, 2018 at 12:22:51AM +0200, Carl Eugen Hoyos wrote: > >> Hi! > >> > >> Attached patch allows decoding the sample from ticket #7495. > >> > >> Please review, Carl E

Re: [FFmpeg-devel] [PATCH 3.4 0/2] configure: fix LIBDRM handling for static linking

2018-10-17 Thread Giulio Benetti
Il 17/10/2018 21:48, Hendrik Leppkes ha scritto: On Wed, Oct 17, 2018 at 9:45 PM Moritz Barsnick wrote: On Wed, Oct 17, 2018 at 21:40:58 +0200, Giulio Benetti wrote: PS: do I need to remove your e-mail on TO field every time I reply to you? It's not a problem, only to know. This mailing lis

Re: [FFmpeg-devel] [PATCH 3.4 0/2] configure: fix LIBDRM handling for static linking

2018-10-17 Thread Giulio Benetti
Il 17/10/2018 21:45, Moritz Barsnick ha scritto: On Wed, Oct 17, 2018 at 21:40:58 +0200, Giulio Benetti wrote: PS: do I need to remove your e-mail on TO field every time I reply to you? It's not a problem, only to know. This mailing list uses a Reply-To: header, pointing at the list address. Y

Re: [FFmpeg-devel] [PATCH 3.4 0/2] configure: fix LIBDRM handling for static linking

2018-10-17 Thread Hendrik Leppkes
On Wed, Oct 17, 2018 at 9:45 PM Moritz Barsnick wrote: > > On Wed, Oct 17, 2018 at 21:40:58 +0200, Giulio Benetti wrote: > > PS: do I need to remove your e-mail on TO field every time I reply to > > you? It's not a problem, only to know. > > This mailing list uses a Reply-To: header, pointing at t

Re: [FFmpeg-devel] [PATCH 3.4 0/2] configure: fix LIBDRM handling for static linking

2018-10-17 Thread Moritz Barsnick
On Wed, Oct 17, 2018 at 21:40:58 +0200, Giulio Benetti wrote: > PS: do I need to remove your e-mail on TO field every time I reply to > you? It's not a problem, only to know. This mailing list uses a Reply-To: header, pointing at the list address. Your Thuderbird should honor that automatically (

Re: [FFmpeg-devel] [PATCH 3.4 0/2] configure: fix LIBDRM handling for static linking

2018-10-17 Thread Giulio Benetti
Il 17/10/2018 21:23, Carl Eugen Hoyos ha scritto: 2018-10-17 21:20 GMT+02:00, Giulio Benetti : Il 17/10/2018 18:51, Carl Eugen Hoyos ha scritto: I just meant that it was a mistake that above patch was backported although it neither fixes a security issue nor a regression. Ok, I understand, so

Re: [FFmpeg-devel] [PATCH 3.4 0/2] configure: fix LIBDRM handling for static linking

2018-10-17 Thread Carl Eugen Hoyos
2018-10-17 21:20 GMT+02:00, Giulio Benetti : > Il 17/10/2018 18:51, Carl Eugen Hoyos ha scritto: >> I just meant that it was a mistake that above patch was backported >> although it neither fixes a security issue nor a regression. > > Ok, I understand, so please discard this patch. I was under the

Re: [FFmpeg-devel] [PATCH 3.4 0/2] configure: fix LIBDRM handling for static linking

2018-10-17 Thread Giulio Benetti
Il 17/10/2018 18:51, Carl Eugen Hoyos ha scritto: I just meant that it was a mistake that above patch was backported although it neither fixes a security issue nor a regression. Ok, I understand, so please discard this patch. On Buildroot we will use a not upstreamed patch then. Please don't

Re: [FFmpeg-devel] [PATCH]lavc/mjpegdec: Support 2:3 subsampling

2018-10-17 Thread Carl Eugen Hoyos
2018-10-17 17:51 GMT+02:00, Michael Niedermayer : > On Wed, Oct 17, 2018 at 12:22:51AM +0200, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch allows decoding the sample from ticket #7495. >> >> Please review, Carl Eugen > >> mjpegdec.c | 17 ++--- >> 1 file changed, 14 insertions

Re: [FFmpeg-devel] [PATCH]lavf/mxfenc: Remove a write-only variable

2018-10-17 Thread Carl Eugen Hoyos
2018-10-17 20:28 GMT+02:00, Baptiste Coudurier : > Hi Carl, > > On Tue, Oct 16, 2018 at 3:00 PM Carl Eugen Hoyos wrote: > >> 2018-10-13 13:12 GMT+02:00, Carl Eugen Hoyos : >> >> > Attached patch fixes a warning, I wonder if the value is actually >> > supposed to use somewhere... >> >> Ping. >> > >

Re: [FFmpeg-devel] [PATCH] avformat/xwma: fix WMAv2 with incorrect bit rate

2018-10-17 Thread Paul B Mahol
On 10/4/18, Paul B Mahol wrote: > On 10/3/18, bananaman...@gmail.com wrote: >> From: bnnm >> >> Fixes trac issue #7215 >> >> Output for files created by xWMAEncode and various videogames is correct >> now. >> >> 1ch 32000hz files are still broken, would need fixes in WMA decoder. >> >> Signed-of

Re: [FFmpeg-devel] [PATCH]lavf/mxfenc: Remove a write-only variable

2018-10-17 Thread Baptiste Coudurier
Hi Carl, On Tue, Oct 16, 2018 at 3:00 PM Carl Eugen Hoyos wrote: > 2018-10-13 13:12 GMT+02:00, Carl Eugen Hoyos : > > > Attached patch fixes a warning, I wonder if the value is actually > > supposed to use somewhere... > > Ping. > LGTM. -- Baptiste Coudurier __

Re: [FFmpeg-devel] [PATCH V2] Add a filter implementing HDR image reconstruction from a single exposure using deep CNNs

2018-10-17 Thread Pedro Arthur
Hi, How hard is it to support the native backend? which operations are missing or any other limitations? Em qua, 17 de out de 2018 às 05:47, Guo, Yejun escreveu: > > see the algorithm's paper and code below. > > the filter's parameter looks like: > sdr2hdr=model_filename=/path_to_tensorflow_grap

Re: [FFmpeg-devel] [PATCH] web/documentation: add new book about FFmpeg in China

2018-10-17 Thread Kieran O Leary
On Wed, 17 Oct 2018, 04:28 Steven Liu, wrote: > Signed-off-by: Steven Liu > --- > src/documentation | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/src/documentation b/src/documentation > index e3bbf4c..7569e86 100644 > --- a/src/documentation > +++ b/src/documentation > @@ -131,6 +1

Re: [FFmpeg-devel] [PATCH] download: change OS X to macOS

2018-10-17 Thread Carl Eugen Hoyos
2018-10-16 4:11 GMT+02:00, Helmut K. C. Tessarek : > --- > src/download | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/download b/src/download > index 0bda5a0..40f4706 100644 > --- a/src/download > +++ b/src/download > @@ -95,7 +95,7 @@ > > >

Re: [FFmpeg-devel] [PATCH 3.4 0/2] configure: fix LIBDRM handling for static linking

2018-10-17 Thread Carl Eugen Hoyos
2018-10-17 15:54 GMT+02:00, Giulio Benetti : > Re-add everybody in Cc > > Il 17/10/2018 15:08, Carl Eugen Hoyos ha scritto: >> 2018-10-17 14:02 GMT+02:00, Giulio Benetti >> : >>> Commit c50dc77ac708e98d02da7c422a6b9cbf9f565aa5 has a pitfall, it >>> appends -ldrm to "Libs:" instead of "Libs.private:

Re: [FFmpeg-devel] [PATCH]lavc/mjpegdec: Support 2:3 subsampling

2018-10-17 Thread Michael Niedermayer
On Wed, Oct 17, 2018 at 12:22:51AM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch allows decoding the sample from ticket #7495. > > Please review, Carl Eugen > mjpegdec.c | 17 ++--- > 1 file changed, 14 insertions(+), 3 deletions(-) > 8f0d03a533b42c35f2bd8d5bdae4dabd24b1

Re: [FFmpeg-devel] [PATCH 3.4 0/2] configure: fix LIBDRM handling for static linking

2018-10-17 Thread Giulio Benetti
Re-add everybody in Cc Il 17/10/2018 15:08, Carl Eugen Hoyos ha scritto: 2018-10-17 14:02 GMT+02:00, Giulio Benetti : Commit c50dc77ac708e98d02da7c422a6b9cbf9f565aa5 has a pitfall, it appends -ldrm to "Libs:" instead of "Libs.private:" in avutil.pc. Assuming this was not a regression before:

[FFmpeg-devel] [PATCH 3.4 2/2] configure: add LIBDRM to Libs.private introducing privatelibs_avutil

2018-10-17 Thread Giulio Benetti
When static linking programs using ffmpeg libraries, if linking against libavutil, -ldrm is listed before -lavutil. This leads to linking failure due to undefined reference of drmGetVersion() and drmFreeVersion(). This is because when pkg-config creates libavutil.pc it doesn't append -ldrm to Libs.

[FFmpeg-devel] [PATCH 3.4 0/2] configure: fix LIBDRM handling for static linking

2018-10-17 Thread Giulio Benetti
Commit c50dc77ac708e98d02da7c422a6b9cbf9f565aa5 has a pitfall, it appends -ldrm to "Libs:" instead of "Libs.private:" in avutil.pc. This leads to append -ldrm to link library list tail in shared build too and this doesn't make sense. So revert the commit above and add a patch to handle explicit pri

[FFmpeg-devel] [PATCH 3.4 1/2] Revert "configure: add LIBDRM to extralibs_avutil"

2018-10-17 Thread Giulio Benetti
This reverts commit c50dc77ac708e98d02da7c422a6b9cbf9f565aa5. The offending commit appends -ldrm to "Libs:" instead of "Libs.private:" in libavutil.pc. This makes no sense, since -ldrm must be listed in linking libraries list only when static linking. Signed-off-by: Giulio Benetti --- configure

[FFmpeg-devel] [PATCH V2] libavcodec/vp8dec: fix the multi-thread HWAccel decode error

2018-10-17 Thread Jun Zhao
Fix the issue: https://github.com/intel/media-driver/issues/317 the root cause is update_dimensions call get_pixel_format will trigger the hwaccel_uninit/hwaccel_init , in current context, there are 3 situations in the update_dimensions(): 1. First time calling. No matter single thread or multithr

Re: [FFmpeg-devel] [PATCH] download: change OS X to macOS

2018-10-17 Thread Lou Logan
On Tue, Oct 16, 2018, at 4:33 PM, Helmut K. C. Tessarek wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA512 > > I've changed the commit message as James suggested. > > Do I have to do anything else to get someone to commit it? I can push this in about a week unless someone else does first.

Re: [FFmpeg-devel] [PATCH] ffmpeg: release the last_frame before the decoders are closed

2018-10-17 Thread Hendrik Leppkes
On Wed, Oct 17, 2018 at 10:14 AM Steve Lhomme wrote: > > On 17/10/2018 09:43, Steve Lhomme wrote: > > On 16/10/2018 18:04, James Almer wrote: > >> On 10/16/2018 12:34 PM, Steve Lhomme wrote: > >>> On 16/10/2018 16:59, Hendrik Leppkes wrote: > On Tue, Oct 16, 2018 at 4:02 PM James Almer wrote

[FFmpeg-devel] [PATCH V2] Add a filter implementing HDR image reconstruction from a single exposure using deep CNNs

2018-10-17 Thread Guo, Yejun
see the algorithm's paper and code below. the filter's parameter looks like: sdr2hdr=model_filename=/path_to_tensorflow_graph.pb:out_fmt=gbrp10le The input of the deep CNN model is RGB24 while the output is float for each color channel. This is the filter's default behavior to output format with

Re: [FFmpeg-devel] [PATCH] ffmpeg: release the last_frame before the decoders are closed

2018-10-17 Thread Steve Lhomme
On 17/10/2018 09:43, Steve Lhomme wrote: On 16/10/2018 18:04, James Almer wrote: On 10/16/2018 12:34 PM, Steve Lhomme wrote: On 16/10/2018 16:59, Hendrik Leppkes wrote: On Tue, Oct 16, 2018 at 4:02 PM James Almer wrote: On 10/16/2018 9:16 AM, Steve Lhomme wrote: If the decoder provides its

Re: [FFmpeg-devel] [PATCH] Add a filter implementing HDR image reconstruction from a single exposure using deep CNNs

2018-10-17 Thread Guo, Yejun
thanks for the review, see in line. > -Original Message- > From: myp...@gmail.com [mailto:myp...@gmail.com] > Sent: Wednesday, October 17, 2018 7:18 AM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Cc: Guo, Yejun ; g...@ffbox0-bg.ffmpeg.org > Subject: Re: [FFmpeg-de

Re: [FFmpeg-devel] [PATCH] ffmpeg: release the last_frame before the decoders are closed

2018-10-17 Thread Steve Lhomme
On 16/10/2018 18:04, James Almer wrote: On 10/16/2018 12:34 PM, Steve Lhomme wrote: On 16/10/2018 16:59, Hendrik Leppkes wrote: On Tue, Oct 16, 2018 at 4:02 PM James Almer wrote: On 10/16/2018 9:16 AM, Steve Lhomme wrote: If the decoder provides its own buffers it might not be able to releas

Re: [FFmpeg-devel] [PATCH] Add a filter implementing HDR image reconstruction from a single exposure using deep CNNs

2018-10-17 Thread Guo, Yejun
thanks for your comments, please see inline. > -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Paul B Mahol > Sent: Tuesday, October 16, 2018 5:00 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-d