Re: [FFmpeg-devel] [PATCH] avcodec/omx: fix xFramerate calculation

2019-08-29 Thread Nicolas George
Aman Gupta (12019-08-29): > > > -in_port_params.format.video.xFramerate = (1 << 16) * > > avctx->framerate.num / avctx->framerate.den; > > > +in_port_params.format.video.xFramerate = (int64_t)(1 << 16) * > > avctx->framerate.num / avctx->framerate.den; > > > > change 1 > 1LL is

[FFmpeg-devel] [PATCH v1] lavf/hlsenc: refine the get_relative_url function to avoid extra malloc for relation path

2019-08-29 Thread lance . lmwang
From: Limin Wang Reviewed-by: Steven Liu Signed-off-by: Limin Wang --- libavformat/hlsenc.c | 50 ++- libavformat/hlsplaylist.c | 4 ++-- libavformat/hlsplaylist.h | 4 ++-- 3 files changed, 19 insertions(+), 39 deletions(-) diff --git

[FFmpeg-devel] [PATCH v1 1/2] avcodec/v210dec: removed the duplicated 'if' condition

2019-08-29 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/v210dec.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libavcodec/v210dec.c b/libavcodec/v210dec.c index 5a33d8c089..6ce18aab5a 100644 --- a/libavcodec/v210dec.c +++ b/libavcodec/v210dec.c @@ -98,8 +98,7 @@

[FFmpeg-devel] [PATCH v1 2/2] avcodec/v210dec: add the slice threading support

2019-08-29 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/v210dec.c | 136 +++ libavcodec/v210dec.h | 1 + 2 files changed, 87 insertions(+), 50 deletions(-) diff --git a/libavcodec/v210dec.c b/libavcodec/v210dec.c index 6ce18aab5a..d3add8b45b 100644

Re: [FFmpeg-devel] [PATCH V1] avutil/file: always set *size to zero if *bufptr is NULL

2019-08-29 Thread myp...@gmail.com
On Thu, Aug 29, 2019 at 9:07 PM Michael Niedermayer wrote: > > On Thu, Aug 29, 2019 at 01:04:43AM +0800, Jun Zhao wrote: > > From: Jun Zhao > > > > Always set *size to zero if *bufptr is NULL, it's more make sence. > > > > fix #8095 > > > > Signed-off-by: Jun Zhao > > --- > > libavutil/file.c

Re: [FFmpeg-devel] [PATCH] MAINTAINERS: add myself to V4L2

2019-08-29 Thread Aman Gupta
On Thu, Aug 29, 2019 at 2:47 PM Jorge Ramirez wrote: > On 8/29/19 22:23, Aman Gupta wrote: > > From: Aman Gupta > > > lets discuss this a bit further. I havent been very active in the past > due to personal reasons and there might be others willing to take on > full maintanership with perhaps

Re: [FFmpeg-devel] [PATCH] avcodec/omx: fix xFramerate calculation

2019-08-29 Thread Aman Gupta
On Thu, Aug 29, 2019 at 5:56 PM Limin Wang wrote: > On Thu, Aug 29, 2019 at 05:30:25PM -0700, Aman Gupta wrote: > > From: Aman Gupta > > > > Integer overflow in the Q16 framerate calculation was sending > > invalid values to the OMX encoder. > > > > On the RPI4, this manifested as bitrate

Re: [FFmpeg-devel] [PATCH] avcodec/omx: fix xFramerate calculation

2019-08-29 Thread Limin Wang
On Thu, Aug 29, 2019 at 05:30:25PM -0700, Aman Gupta wrote: > From: Aman Gupta > > Integer overflow in the Q16 framerate calculation was sending > invalid values to the OMX encoder. > > On the RPI4, this manifested as bitrate controls being ignored > on video streams with 6/1001 framerates.

[FFmpeg-devel] [PATCH] avcodec/omx: fix xFramerate calculation

2019-08-29 Thread Aman Gupta
From: Aman Gupta Integer overflow in the Q16 framerate calculation was sending invalid values to the OMX encoder. On the RPI4, this manifested as bitrate controls being ignored on video streams with 6/1001 framerates. Video streams with 3/1001 framerates were not affected.

[FFmpeg-devel] [PATCH] fix rtmp handshake for some streams [v2]

2019-08-29 Thread William Martin
From: Will Martin Some rtmp streamers (i.e. AWS Elemental Encoder, Wirecast) send C0 and C1 together and expect S0 and S1 returned together. When sent in different packets, this results in a C2 handshake. This patch fixes that error. Note that the patch is based off of a fix proposed by

[FFmpeg-devel] [PATCH v2 2/2] avformat/hlsenc: avformat/hlsenc: reopen new http session for http_persistent

2019-08-29 Thread Steven Liu
fix ticket: 7975 Signed-off-by: Steven Liu --- libavformat/hlsenc.c | 66 +++- 1 file changed, 55 insertions(+), 11 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index c6bd664b92..d9b48f0e7c 100644 ---

[FFmpeg-devel] [PATCH v2 1/2] avformat/http: add ff_http_get_shutdown_status api for check the status of shutdown

2019-08-29 Thread Steven Liu
this function is used to get the previous shutdown status when reusing the old connection in block mode. Signed-off-by: Steven Liu --- libavformat/http.c | 16 libavformat/http.h | 9 + 2 files changed, 25 insertions(+) diff --git a/libavformat/http.c

[FFmpeg-devel] [PATCH] avcodec/wcmv: use ff_reget_buffer()

2019-08-29 Thread James Almer
Signed-off-by: James Almer --- Untested. There doesn't seem to be a fate test, and i have no samples at hand. libavcodec/wcmv.c | 23 +-- 1 file changed, 9 insertions(+), 14 deletions(-) diff --git a/libavcodec/wcmv.c b/libavcodec/wcmv.c index 0d60b9fe1f..68142b45fd 100644

[FFmpeg-devel] [PATCH] MAINTAINERS: add myself to V4L2

2019-08-29 Thread Aman Gupta
From: Aman Gupta Signed-off-by: Aman Gupta Signed-off-by: Jorge Ramirez-Ortiz --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 7f60ef0021..f83d543321 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -243,7 +243,7 @@ Codecs:

Re: [FFmpeg-devel] [PATCH] avformat/utils: return pending IO error on EOF in av_read_frame()

2019-08-29 Thread Nicolas George
Marton Balint (12019-08-24): > avio_feof() returns true both in case of actual EOF and in case of IO errors. > Some demuxers (matroska) have special handling to be able to return the proper > error for this exact reason, e.g.: > > if (avio_feof(pb)) { > if (pb->error) { > return

Re: [FFmpeg-devel] [PATCH] avformat/utils: return pending IO error on EOF in av_read_frame()

2019-08-29 Thread Marton Balint
On Sat, 24 Aug 2019, Marton Balint wrote: avio_feof() returns true both in case of actual EOF and in case of IO errors. Some demuxers (matroska) have special handling to be able to return the proper error for this exact reason, e.g.: if (avio_feof(pb)) { if (pb->error) { return

Re: [FFmpeg-devel] [PATCH] avformat/avidec: add support for recognizing HEVC fourcc when demuxing

2019-08-29 Thread Marton Balint
On Sat, 24 Aug 2019, James Almer wrote: On Sat, Aug 24, 2019 at 5:48 AM Marton Balint wrote: On Sat, 24 Aug 2019, Carl Eugen Hoyos wrote: > Am Sa., 24. Aug. 2019 um 01:47 Uhr schrieb Marton Balint : >> >> Some security cams generate this, as well as some versions of VirtualDub so >>

[FFmpeg-devel] [PATCH 2/2] avcodec/qdm2: Check frame size

2019-08-29 Thread Michael Niedermayer
Fixes: index 2304 out of bounds for type 'float [2304]' Fixes: 16332/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_QDM2_fuzzer-5679142481166336 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer ---

[FFmpeg-devel] [PATCH 1/2] tools/target_dec_fuzzer: increase snows threshold

2019-08-29 Thread Michael Niedermayer
snow allows creating large output from tiny input, like other wavelet codecs Fixes: Timeout (82sec -> 1.5sec) Fixes: 9520/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_SNOW_fuzzer-6286612576075776 Found-by: continuous fuzzing process

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/arbc: Use ff_reget_buffer()

2019-08-29 Thread James Almer
On 8/29/2019 4:04 PM, Michael Niedermayer wrote: > Fixes: Timeout (20sec -> 0.5sec) > Fixes: > 16404/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ARBC_fuzzer-5660216355454976 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg >

[FFmpeg-devel] [PATCH 2/2] avcodec/arbc: Use ff_reget_buffer()

2019-08-29 Thread Michael Niedermayer
Fixes: Timeout (20sec -> 0.5sec) Fixes: 16404/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ARBC_fuzzer-5660216355454976 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/arbc.c | 21

[FFmpeg-devel] [PATCH 1/2] avcodec/cfhd: Check that cropped size is smaller than full

2019-08-29 Thread Michael Niedermayer
Fixes: signed integer overflow: 57342 * 120830 cannot be represented in type 'int' Fixes: 16426/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_CFHD_fuzzer-5758744817827840 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg ---

Re: [FFmpeg-devel] [PATCH v4] Add ZeroMQ as protocol option

2019-08-29 Thread Marton Balint
On Thu, 29 Aug 2019, Andriy Gelman wrote: Changes in v4: - Use polling instead of non-blocking option for socket read/write operations. - Added pkt_size, timeout_send, timeout_recv options. Updated documentation for new options. No, timeout_send and timeout_recv is not needed. The

[FFmpeg-devel] [PATCH v4] Add ZeroMQ as protocol option

2019-08-29 Thread Andriy Gelman
Changes in v4: - Use polling instead of non-blocking option for socket read/write operations. - Added pkt_size, timeout_send, timeout_recv options. Updated documentation for new options. - Removed redundant memcpy now that pkt_size is set. - Check that context is successfully

Re: [FFmpeg-devel] [PATCH] Change libvpx-vpx default to crf=32.

2019-08-29 Thread Jan Ekström
On Thu, Aug 29, 2019 at 7:17 PM Carl Eugen Hoyos wrote: > > Am Do., 29. Aug. 2019 um 18:12 Uhr schrieb Elliott Karpilovsky > : > > > > On Wed, Aug 28, 2019 at 3:11 PM Carl Eugen Hoyos wrote: > > > > > > Am Mi., 28. Aug. 2019 um 23:26 Uhr schrieb Elliott Karpilovsky > > > : > > > > > > > >

Re: [FFmpeg-devel] [PATCH] avcodec/qtrle: set pict type information in ouptut frames

2019-08-29 Thread Paul B Mahol
On 8/29/19, James Almer wrote: > On 8/29/2019 1:18 PM, Paul B Mahol wrote: >> On 8/29/19, James Almer wrote: >>> Signed-off-by: James Almer >>> --- >>> libavcodec/qtrle.c | 7 +++ >>> 1 file changed, 7 insertions(+) >>> >>> diff --git a/libavcodec/qtrle.c b/libavcodec/qtrle.c >>> index

Re: [FFmpeg-devel] [PATCH] avcodec/qtrle: set pict type information in ouptut frames

2019-08-29 Thread James Almer
On 8/29/2019 1:18 PM, Paul B Mahol wrote: > On 8/29/19, James Almer wrote: >> Signed-off-by: James Almer >> --- >> libavcodec/qtrle.c | 7 +++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/libavcodec/qtrle.c b/libavcodec/qtrle.c >> index f565e0e358..ba6740f5d4 100644 >> ---

Re: [FFmpeg-devel] [PATCH] avcodec/qtrle: set pict type information in ouptut frames

2019-08-29 Thread Paul B Mahol
On 8/29/19, James Almer wrote: > Signed-off-by: James Almer > --- > libavcodec/qtrle.c | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/libavcodec/qtrle.c b/libavcodec/qtrle.c > index f565e0e358..ba6740f5d4 100644 > --- a/libavcodec/qtrle.c > +++ b/libavcodec/qtrle.c > @@ -449,6

Re: [FFmpeg-devel] [PATCH] Change libvpx-vpx default to crf=32.

2019-08-29 Thread Carl Eugen Hoyos
Am Do., 29. Aug. 2019 um 18:12 Uhr schrieb Elliott Karpilovsky : > > On Wed, Aug 28, 2019 at 3:11 PM Carl Eugen Hoyos wrote: > > > > Am Mi., 28. Aug. 2019 um 23:26 Uhr schrieb Elliott Karpilovsky > > : > > > > > > Current default is 200kbps, which produces inconsistent > > > results (too high for

Re: [FFmpeg-devel] [PATCH 1/2] lavu/pixfmt: add AYUV pixel format

2019-08-29 Thread Carl Eugen Hoyos
Am Do., 29. Aug. 2019 um 16:16 Uhr schrieb Fu, Linjie : > Thus AYUV will make more sense. If your hardware decoding does not produce valid alpha data (0xFF for opaque, 0x00 for completely transparent), you cannot use an FFmpeg pix_fmt that defines alpha for decoding. If you define a pix_fmt that

Re: [FFmpeg-devel] [PATCH] Change libvpx-vpx default to crf=32.

2019-08-29 Thread Elliott Karpilovsky
On Wed, Aug 28, 2019 at 3:11 PM Carl Eugen Hoyos wrote: > > Am Mi., 28. Aug. 2019 um 23:26 Uhr schrieb Elliott Karpilovsky > : > > > > Current default is 200kbps, which produces inconsistent > > results (too high for low-res, too low for hi-res). Use > > CRF instead, which will adapt. Affects

Re: [FFmpeg-devel] [PATCH] libavformat/rtspdec: Don't send teardown if rtsp_hd_out is null

2019-08-29 Thread Ross Nicholson
Hey Jun, So I got kodi running with FFmpeg n4.2 and the issue persists. Here's the debugger output after trying to play the test link you provided. With the patch this does not occur so their must be some way to call function with rt->rtsp_hd_out as NULL; Thanks, Ross Process 92017 stopped *

[FFmpeg-devel] [PATCH] avcodec/qtrle: set pict type information in ouptut frames

2019-08-29 Thread James Almer
Signed-off-by: James Almer --- libavcodec/qtrle.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavcodec/qtrle.c b/libavcodec/qtrle.c index f565e0e358..ba6740f5d4 100644 --- a/libavcodec/qtrle.c +++ b/libavcodec/qtrle.c @@ -449,6 +449,7 @@ static int

Re: [FFmpeg-devel] [PATCH] hwcontext_d3d11va: support bgr0 pixel format

2019-08-29 Thread Thomas Volkert
On 24.08.2019 20:12, Carl Eugen Hoyos wrote: > Am Do., 8. Aug. 2019 um 02:35 Uhr schrieb Thomas Volkert : >> From: Thomas Volkert >> >> --- >> libavutil/hwcontext_d3d11va.c | 7 --- >> 1 file changed, 4 insertions(+), 3 deletions(-) >> >> diff --git a/libavutil/hwcontext_d3d11va.c

Re: [FFmpeg-devel] [PATCH 1/2] lavu/pixfmt: add AYUV pixel format

2019-08-29 Thread Fu, Linjie
> -Original Message- > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > Of Carl Eugen Hoyos > Sent: Thursday, August 29, 2019 21:31 > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH 1/2] lavu/pixfmt: add AYUV

Re: [FFmpeg-devel] [PATCH 1/2] lavu/pixfmt: add AYUV pixel format

2019-08-29 Thread Carl Eugen Hoyos
> Am 29.08.2019 um 14:30 schrieb Hendrik Leppkes : > >> On Thu, Aug 29, 2019 at 1:47 PM Carl Eugen Hoyos wrote: >> >> >> >> Am 29.08.2019 um 03:34 schrieb Fu, Linjie : >> -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of Carl

Re: [FFmpeg-devel] [PATCH 1/2] lavu/pixfmt: add AYUV pixel format

2019-08-29 Thread Carl Eugen Hoyos
> Am 29.08.2019 um 14:30 schrieb Hendrik Leppkes : > >> On Thu, Aug 29, 2019 at 1:47 PM Carl Eugen Hoyos wrote: >> >> >> >> Am 29.08.2019 um 03:34 schrieb Fu, Linjie : >> -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of Carl

Re: [FFmpeg-devel] [PATCH 1/3] tools/target_dec_fuzzer: Adjust GDV pixel threshold down by a factor of 2

2019-08-29 Thread Michael Niedermayer
On Wed, Aug 14, 2019 at 09:00:06AM +0200, Michael Niedermayer wrote: > Fixes: Timeout (7sec -> 1sec) > Fixes: > 14709/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_GDV_fuzzer-5704215281795072 > > Found-by: continuous fuzzing process >

Re: [FFmpeg-devel] [PATCH 3/8] tools/target_dec_fuzzer: adjust pixel threshold for SANM, as it allows coding gigantic images on tiny input

2019-08-29 Thread Michael Niedermayer
On Mon, Aug 12, 2019 at 09:17:03PM +0200, Michael Niedermayer wrote: > Fixes: Timeout (13sec ->1sec) > Fixes: > 16122/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_SANM_fuzzer-5724944247291904 > > Found-by: continuous fuzzing process >

Re: [FFmpeg-devel] [PATCH 1/6] avcodec/tta: Fix integer overflow in prediction

2019-08-29 Thread Michael Niedermayer
On Mon, Aug 12, 2019 at 02:17:14AM +0200, Michael Niedermayer wrote: > Fixes: signed integer overflow: -395281576 + -1827578048 cannot be > represented in type 'int' > Fixes: > 16038/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_TTA_fuzzer-5646109705240576 > > Found-by: continuous fuzzing

Re: [FFmpeg-devel] [PATCH 6/6] avcodec/vb: Check input packet size to be large enough to contain flags

2019-08-29 Thread Michael Niedermayer
On Mon, Aug 12, 2019 at 02:17:19AM +0200, Michael Niedermayer wrote: > Fixes: Timeout (->9sec) > Fixes: > 16292/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_VB_fuzzer-5747063496638464 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg

Re: [FFmpeg-devel] [PATCH 2/8] avcodec/cavsdec: Limit the number of access units per packet to 2

2019-08-29 Thread Michael Niedermayer
On Sat, Aug 10, 2019 at 11:09:43PM +0200, Michael Niedermayer wrote: > Fixes: Timeout (122sec -> 13ms) > Fixes: > 15978/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_CAVS_fuzzer-5148925004087296 > > Found-by: continuous fuzzing process >

Re: [FFmpeg-devel] [PATCH 7/8] avutil/mathematics: Fix 2 overflows in av_add_stable()

2019-08-29 Thread Michael Niedermayer
On Sat, Aug 10, 2019 at 11:09:48PM +0200, Michael Niedermayer wrote: > Fixes: signed integer overflow: 9223372036854775807 + 1 cannot be represented > in type 'long' > Fixes: > 16022/clusterfuzz-testcase-minimized-ffmpeg_DEMUXER_fuzzer-5759796759756800 > > Found-by: continuous fuzzing process

Re: [FFmpeg-devel] [PATCH] avformat/mpsubdec: Remove floating point usage

2019-08-29 Thread Michael Niedermayer
On Tue, Jul 30, 2019 at 08:18:47PM +0200, Michael Niedermayer wrote: > This makes the code bitexact between platforms. > Intermediate timestamps between frames are preserved. > The timebase is simplified. > Rounding differs from doubles in cases where timestamps/durations > are "funny" > >

[FFmpeg-devel] [PATCH v17 15/15] lavc/tiff: Enable decoding of LinearRaw images

2019-08-29 Thread Nick Renieris
From: Nick Renieris "LinearRaw" is a value that the PhotometricInterpretation tag can be set to on DNG images that contain color information for all channels instead of being bayer-encoded ("CFA" value). The DNG decoder is complete enough that we can enable this now. Sample: -

[FFmpeg-devel] [PATCH v17 14/15] lavc/tiff: Default-initialize WhiteLevel DNG tag value

2019-08-29 Thread Nick Renieris
From: Nick Renieris Initialized to `(2 ^ BitsPerSample) - 1` as per the DNG Specification. Also make sure that `BlackLevel < WhiteLevel`. This fixes decoding for "X7 CinemaDNG" samples here: - https://www.dji.com/gr/zenmuse-x7/info#downloads Signed-off-by: Nick Renieris ---

[FFmpeg-devel] [PATCH v17 08/15] lavc/tiff: Force DNG pixel data endianness on an edge case

2019-08-29 Thread Nick Renieris
From: Nick Renieris This fixes "X7 RAW" and "X7 CinemaDNG" samples here: - https://www.dji.com/gr/zenmuse-x7/info#downloads Signed-off-by: Nick Renieris --- libavcodec/tiff.c | 12 1 file changed, 12 insertions(+) diff --git a/libavcodec/tiff.c b/libavcodec/tiff.c index

[FFmpeg-devel] [PATCH v17 10/15] lavc/tiff: Support decoding of DNGs with single-component JPEGs

2019-08-29 Thread Nick Renieris
From: Nick Renieris This enables decoding of DNG images generated by the 'DJI Zenmuse X7' digital camera Samples: https://www.dji.com/gr/zenmuse-x7/info#downloads Signed-off-by: Nick Renieris --- libavcodec/tiff.c | 61 +++ 1 file changed, 51

[FFmpeg-devel] [PATCH v17 13/15] lavc/tiff: Support DNGs with striped (non-tiled) JPEGs images

2019-08-29 Thread Nick Renieris
From: Nick Renieris DNG samples here can now be decoded: - https://www.photographyblog.com/previews/pentax_k1_photos Signed-off-by: Nick Renieris --- libavcodec/tiff.c | 61 --- 1 file changed, 37 insertions(+), 24 deletions(-) diff --git

[FFmpeg-devel] [PATCH v17 11/15] lavc/tiff: Decode 10-bit and 14-bit DNG images

2019-08-29 Thread Nick Renieris
From: Nick Renieris 10-bit sample: http://www.rawsamples.ch/raws/phones/RAW_ONEPLUS_ONE-A0001.DNG 14-bit sample: https://drive.google.com/open?id=0B4JyRT3Lth5HVndyOTVOdWktM3J4TFEydTk1MnY3RWlpSzVB Signed-off-by: Nick Renieris --- libavcodec/tiff.c | 23 +++ 1 file changed,

[FFmpeg-devel] [PATCH v17 12/15] lavc/mjpegdec: Skip unknown APPx marker on bayer images

2019-08-29 Thread Nick Renieris
From: Nick Renieris Samples: - Embedded JPEG images in the DNG images here: https://www.photographyblog.com/previews/pentax_k1_photos Signed-off-by: Nick Renieris --- libavcodec/mjpegdec.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/libavcodec/mjpegdec.c

[FFmpeg-devel] [PATCH v17 07/15] lavc/tiff: Don't apply strips-related logic to tiled images

2019-08-29 Thread Nick Renieris
From: Nick Renieris Signed-off-by: Nick Renieris --- libavcodec/tiff.c | 42 ++ 1 file changed, 22 insertions(+), 20 deletions(-) diff --git a/libavcodec/tiff.c b/libavcodec/tiff.c index 3bf2b3e557..28b8f42edd 100644 --- a/libavcodec/tiff.c +++

[FFmpeg-devel] [PATCH v17 06/15] lavc/tiff: Fix edge case with full-length/width tiles

2019-08-29 Thread Nick Renieris
From: Nick Renieris When the height is equal to the tile length (full-height tile) after `height % tile_length` is applied with the current code, it results in the operating tile_length to be 0. This commit makes this leftover logic only applies if it's necessary. Signed-off-by: Nick Renieris

[FFmpeg-devel] [PATCH v17 04/15] lavc/tiff: Apply color scaling to uncompressed DNGs

2019-08-29 Thread Nick Renieris
From: Nick Renieris Signed-off-by: Nick Renieris --- libavcodec/tiff.c | 25 - 1 file changed, 24 insertions(+), 1 deletion(-) diff --git a/libavcodec/tiff.c b/libavcodec/tiff.c index eb064eec5c..78b4bd5301 100644 --- a/libavcodec/tiff.c +++ b/libavcodec/tiff.c @@

[FFmpeg-devel] [PATCH v17 09/15] lavc/mjpegdec: Enable decoding of single-component bayer images

2019-08-29 Thread Nick Renieris
From: Nick Renieris Also, ensure no false positives when determining DNG bayer images, by setting them in tiff.c instead of relying on a heuristic. There's no way to determine this just from the JPEG data, so we have to pass this information from outside the MJPEG decoder. Signed-off-by: Nick

[FFmpeg-devel] [PATCH v17 05/15] lavc/jpegtables: Handle multiple mappings to the same value

2019-08-29 Thread Nick Renieris
From: Nick Renieris Some JPEGs [1] have incorrect DHT entries that map 2 codes to the same value. The second (last) mapping does not ever actually appear in the code stream, therefore ignoring any mappings after the first one fixes this. Without this, an "mjpeg_decode_dc: bad vlc: 0:0" error

[FFmpeg-devel] [PATCH v17 03/15] lavc/tiff: Convert DNGs to sRGB color space

2019-08-29 Thread Nick Renieris
From: Nick Renieris Signed-off-by: Nick Renieris --- libavcodec/tiff.c | 34 +++--- 1 file changed, 23 insertions(+), 11 deletions(-) diff --git a/libavcodec/tiff.c b/libavcodec/tiff.c index 4c6b835afe..eb064eec5c 100644 --- a/libavcodec/tiff.c +++

[FFmpeg-devel] [PATCH v17 02/15] lavc/tiff: Decode embedded JPEGs in DNG images

2019-08-29 Thread Nick Renieris
From: Nick Renieris Used a technique similar to lavc/tdsc.c for invoking the MJPEG decoder. This commit adds support for: - DNG tiles - DNG tile huffman lossless JPEG decoding - DNG 8-bpp ("packed" as dcraw calls it) decoding - DNG color scaling [1] - LinearizationTable tag - BlackLevel tag

[FFmpeg-devel] [PATCH v17 01/15] lavc/mjpegdec: Decode Huffman-coded lossless JPEGs embedded in DNGs

2019-08-29 Thread Nick Renieris
From: Nick Renieris Main image data in DNGs is usually comprised of tiles, each of which is a Huffman-encoded lossless JPEG. Tested for ljpeg regressions with: `ffmpeg -f lavfi -i testsrc=d=1 -vcodec ljpeg test.avi` `ffmpeg test.avi out.avi` The modified code in ljpeg_decode_rgb_scan runs

Re: [FFmpeg-devel] [PATCH v16 05/15] lavc/jpegtables: Handle multiple mappings to the same value

2019-08-29 Thread Nick Renieris
Στις Πέμ, 29 Αυγ 2019 στις 4:00 μ.μ., ο/η Michael Niedermayer έγραψε: > This looks unrelated Messed up when rebasing, will fix in a bit. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To

Re: [FFmpeg-devel] [PATCH V1] avutil/file: always set *size to zero if *bufptr is NULL

2019-08-29 Thread Michael Niedermayer
On Thu, Aug 29, 2019 at 01:04:43AM +0800, Jun Zhao wrote: > From: Jun Zhao > > Always set *size to zero if *bufptr is NULL, it's more make sence. > > fix #8095 > > Signed-off-by: Jun Zhao > --- > libavutil/file.c |7 ++- > 1 files changed, 6 insertions(+), 1 deletions(-) probably ok

Re: [FFmpeg-devel] [PATCH v16 05/15] lavc/jpegtables: Handle multiple mappings to the same value

2019-08-29 Thread Michael Niedermayer
On Wed, Aug 28, 2019 at 03:27:13PM +0300, Nick Renieris wrote: > From: Nick Renieris > > Some JPEGs [1] have incorrect DHT entries that map 2 codes to > the same value. > > The second (last) mapping does not ever actually appear in the > code stream, therefore ignoring any mappings after the

Re: [FFmpeg-devel] [PATCH 1/2] lavu/pixfmt: add AYUV pixel format

2019-08-29 Thread Hendrik Leppkes
On Thu, Aug 29, 2019 at 1:47 PM Carl Eugen Hoyos wrote: > > > > Am 29.08.2019 um 03:34 schrieb Fu, Linjie : > > >> -Original Message- > >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > >> Of Carl Eugen Hoyos > >> Sent: Thursday, August 29, 2019 05:17 > >> To:

Re: [FFmpeg-devel] [PATCH 1/2] lavu/pixfmt: add AYUV pixel format

2019-08-29 Thread Paul B Mahol
On 8/29/19, Carl Eugen Hoyos wrote: > > > Am 29.08.2019 um 03:34 schrieb Fu, Linjie : > >>> -Original Message- >>> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf >>> Of Carl Eugen Hoyos >>> Sent: Thursday, August 29, 2019 05:17 >>> To: FFmpeg development discussions

Re: [FFmpeg-devel] [PATCH 1/2] lavu/pixfmt: add AYUV pixel format

2019-08-29 Thread Carl Eugen Hoyos
Am 29.08.2019 um 03:34 schrieb Fu, Linjie : >> -Original Message- >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf >> Of Carl Eugen Hoyos >> Sent: Thursday, August 29, 2019 05:17 >> To: FFmpeg development discussions and patches > de...@ffmpeg.org> >> Subject:

[FFmpeg-devel] [PATCH] avformat/dashdec: start from the root uri when baseURL is start with '/'

2019-08-29 Thread Steven Liu
fix ticket: 8097 Signed-off-by: Steven Liu --- libavformat/dashdec.c | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/libavformat/dashdec.c b/libavformat/dashdec.c index f82a750d6f..a6f75514f6 100644 --- a/libavformat/dashdec.c +++ b/libavformat/dashdec.c @@

Re: [FFmpeg-devel] [PATCH v1] avformat/mxfenc: fix warning: unused function 'klv_ber_length' [-Wunused-function]

2019-08-29 Thread Limin Wang
On Thu, Aug 29, 2019 at 10:12:23AM +0200, Marton Balint wrote: > > On Thu, 29 Aug 2019, lance.lmw...@gmail.com wrote: > > >From: Limin Wang > > See https://patchwork.ffmpeg.org/patch/14141/ Haven't notice it, however the patch isn't continue anymore. I submit the update patch by the Tomas

[FFmpeg-devel] [PATCH v2] avformat/mxfenc: fix warning: unused function 'klv_ber_length' [-Wunused-function]

2019-08-29 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavformat/mxfenc.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c index 8b3d599a6f..aa23ee3947 100644 --- a/libavformat/mxfenc.c +++ b/libavformat/mxfenc.c @@ -553,15

Re: [FFmpeg-devel] [PATCH v1] avformat/mxfenc: fix warning: unused function 'klv_ber_length' [-Wunused-function]

2019-08-29 Thread Marton Balint
On Thu, 29 Aug 2019, lance.lmw...@gmail.com wrote: From: Limin Wang See https://patchwork.ffmpeg.org/patch/14141/ Thanks, Marton Signed-off-by: Limin Wang --- libavformat/mxfenc.c | 8 1 file changed, 8 deletions(-) diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c

[FFmpeg-devel] [PATCH v1] avformat/mxfenc: fix warning: unused function 'klv_ber_length' [-Wunused-function]

2019-08-29 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavformat/mxfenc.c | 8 1 file changed, 8 deletions(-) diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c index 8b3d599a6f..b5d9bb5f46 100644 --- a/libavformat/mxfenc.c +++ b/libavformat/mxfenc.c @@ -542,14 +542,6 @@ static void

Re: [FFmpeg-devel] [PATCH] libavformat/rtspdec: Don't send teardown if rtsp_hd_out is null

2019-08-29 Thread Ross Nicholson
I did leave out one detail. Currently kodi is using v4.0.4. It's possible there was a fix since that prevents rt->rtsp_hd being NULL in the call to rtsp_send_cmd_with_content_async in the first place. Won't be able to test with 4.2 and kodi until the version gets bumped to see if it persists

[FFmpeg-devel] [PATCH v2 5/6] avcodec/v210enc: define DEFINE_V210_ENCODE_FUNC to remove duplicate code

2019-08-29 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/v210enc.c | 171 +-- 1 file changed, 69 insertions(+), 102 deletions(-) diff --git a/libavcodec/v210enc.c b/libavcodec/v210enc.c index 2400538f73..1b17d885cc 100644 --- a/libavcodec/v210enc.c +++

Re: [FFmpeg-devel] [PATCH V5] avcodec/libvpxenc: add ROI-based encoding support for VP8/VP9 support

2019-08-29 Thread Guo, Yejun
> -Original Message- > From: James Zern [mailto:jz...@google.com] > Sent: Thursday, August 29, 2019 2:14 PM > To: Guo, Yejun > Cc: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH V5] avcodec/libvpxenc: add ROI-based > encoding support for VP8/VP9 support

Re: [FFmpeg-devel] [PATCH V5] avcodec/libvpxenc: add ROI-based encoding support for VP8/VP9 support

2019-08-29 Thread James Zern
On Wed, Aug 28, 2019 at 10:20 PM Guo, Yejun wrote: > > > > > -Original Message- > > From: James Zern [mailto:jz...@google.com] > > Sent: Thursday, August 29, 2019 12:39 PM > > To: Guo, Yejun > > Cc: FFmpeg development discussions and patches > > Subject: Re: [FFmpeg-devel] [PATCH V5]