On Sun, Sep 29, 2019 at 01:53:43AM +0200, Michael Niedermayer wrote:
> Fixes: Timeout (110sec -> 10sec)
> Fixes:
> 17705/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_VP7_fuzzer-5765834135306240
>
> Found-by: continuous fuzzing process
>
> From: ffmpeg-devel On Behalf Of Linjie Fu
> Sent: Friday, September 27, 2019 1:47 PM
> To: ffmpeg-devel@ffmpeg.org
> Cc: ChaoX A Liu ; Fu, Linjie
> Subject: [FFmpeg-devel] [PATCH, v2] lavc/qsvdec: Add GPU-accelerated memory
> copy support
>
> GPU copy enables or disables GPU accelerated
James Almer:
> On 9/27/2019 11:52 PM, Andreas Rheinhardt wrote:
>> James Almer:
>>> On 9/20/2019 5:39 PM, Andreas Rheinhardt wrote:
Up until now, read_frame_internal in avformat/utils.c uses a spare
packet on the stack that serves no real purpose: At no point in this
function is
Replace a return ret with return 0 to more clearly indicate success.
Also don't use separate ret and err variables.
Signed-off-by: Andreas Rheinhardt
---
libavformat/utils.c | 15 +++
1 file changed, 7 insertions(+), 8 deletions(-)
diff --git a/libavformat/utils.c
header.data_max and header.data_min are not necessarely set on all decoding
scenarios.
Fixes a Valgrind reported regression since
cfa193779103c97bbfc28273a0ab12c114b6786d.
Signed-off-by: James Almer
---
libavcodec/fitsdec.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
On Sat, Sep 28, 2019 at 11:29 PM Michael Niedermayer
wrote:
>
> On Sat, Sep 28, 2019 at 08:23:57PM +0800, Jun Zhao wrote:
> > From: Jun Zhao
> >
> > add duration estimate method trace, it's can help to some duration
> > issue trace or debug.
>
> theres some english grammer issue here
>
Will fix,
> -Original Message-
> From: Li, Zhong
> Sent: Friday, September 13, 2019 00:05
> To: FFmpeg development discussions and patches de...@ffmpeg.org>
> Cc: Fu, Linjie
> Subject: RE: [FFmpeg-devel] [PATCH, v2] lavc/vaapi_encode: grow packet if
> vaMapBuffer returns multiple buffers
>
> >
On Sat, Sep 28, 2019 at 6:52 AM Lou Logan wrote:
>
> Signed-off-by: Lou Logan
> ---
> fftools/cmdutils.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c
> index 6f4031fbb9..84f98b7c04 100644
> --- a/fftools/cmdutils.c
> +++
Fixes: Timeout (110sec -> 10sec)
Fixes:
17705/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_VP7_fuzzer-5765834135306240
Found-by: continuous fuzzing process
https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer
---
tools/target_dec_fuzzer.c | 1
Fixes: TImeout (20sec -> 4sec)
Fixes:
17735/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_DXV_fuzzer-5723368317255680
Found-by: continuous fuzzing process
https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer
---
tools/target_dec_fuzzer.c | 1 +
Fixes: shift exponent -2 is negative
Fixes:
17736/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_PCM_F16LE_fuzzer-5742815929171968
Found-by: continuous fuzzing process
https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer
---
libavcodec/pcm.c | 3
Fixes: Timeout (147sec -> 2sec)
Fixes:
17704/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_VC1IMAGE_fuzzer-5723851098423296
Found-by: continuous fuzzing process
https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer
---
libavcodec/vc1_block.c | 8
Fixes: signed integer overflow: 62220 * 262144 cannot be represented in type
'int'
Fixes:
17145/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_VC1IMAGE_fuzzer-5667394743173120
Found-by: continuous fuzzing process
https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by:
James Almer:
> On 9/20/2019 5:39 PM, Andreas Rheinhardt wrote:
>> From: James Almer
>>
>> Demuxers may have allocated a packet before encountering an error and
>> aborting.
>
> Fun thing about this, the doxy states that packets should be freed on
> error before returning from the demuxer, so
Matthieu Bouron:
> On Fri, Sep 27, 2019 at 04:14:33PM +0100, Derek Buitenhuis wrote:
>> On 27/09/2019 15:37, Matthieu Bouron wrote:
>>> Allows the creation of the sdtp atom while remuxing MP4 to MP4. This
>>> atom is required by Apple devices (iPhone, Apple TV) in order to accept
>>> 2160p medias.
On 9/20/2019 5:39 PM, Andreas Rheinhardt wrote:
> From: James Almer
>
> Demuxers may have allocated a packet before encountering an error and
> aborting.
Fun thing about this, the doxy states that packets should be freed on
error before returning from the demuxer, so technically the apng
On 9/27/2019 11:52 PM, Andreas Rheinhardt wrote:
> James Almer:
>> On 9/20/2019 5:39 PM, Andreas Rheinhardt wrote:
>>> Up until now, read_frame_internal in avformat/utils.c uses a spare
>>> packet on the stack that serves no real purpose: At no point in this
>>> function is there a need for
On Fri, 27 Sep 2019, Limin Wang wrote:
ping, please help push it or comments for further changes.
Applied the patchset with some minor cosmetic changes.
Thanks,
Marton
On Tue, Sep 24, 2019 at 06:18:07PM +0800, lance.lmw...@gmail.com wrote:
From: Limin Wang
Reviewed-by: Paul B Mahol
Am Sa., 28. Sept. 2019 um 16:52 Uhr schrieb Tomas Härdin :
>
> tor 2019-09-26 klockan 23:10 +0200 skrev Michael Niedermayer:
> > Fixes: Timeout(103sec -> 3sec)
> > Fixes:
> > 17678/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_FLIC_fuzzer-5715436989054976
> >
> > Found-by: continuous fuzzing
The structure of a ProRes frame in mov/mp4 is that of a typical atom:
First a 32 bit BE size field, then a tag detailling the content. Said
size field includes the eight bytes of the atom header.
This header is actually redundant, as the size of the atom is already
known from the containing atom.
On Wed, Sep 18, 2019 at 01:43:54AM +0200, Michael Niedermayer wrote:
> Fixes: Timeout (30sec -> 25sec)
> Fixes:
> 17050/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_MOTIONPIXELS_fuzzer-5719149803732992
>
> Found-by: continuous fuzzing process
>
On Sat, Sep 14, 2019 at 11:39:49PM +0200, Michael Niedermayer wrote:
> Fixes: Timeout (89sec -> 7sec)
> Fixes:
> 17035/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_BINK_fuzzer-5737222422134784
>
> Found-by: continuous fuzzing process
>
On Sun, Sep 01, 2019 at 12:32:36AM +0200, Michael Niedermayer wrote:
> "9.1.1.43 P Reference Distance (REFDIST)"
> "The value of REFDIST shall be less than, or equal to, 16."
>
> Signed-off-by: Michael Niedermayer
> ---
> libavcodec/vc1.c | 4 +++-
> 1 file changed, 3 insertions(+), 1
On Wed, Sep 11, 2019 at 02:18:41AM +0200, Michael Niedermayer wrote:
> Fixes: left shift of negative value -7
> Fixes:
> 16959/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_WMV3_fuzzer-5200360825683968
>
> Found-by: continuous fuzzing process
>
On Tue, Sep 03, 2019 at 02:14:23AM +0200, Michael Niedermayer wrote:
> Fixes: negation of -2147483648 cannot be represented in type 'int'; cast to
> an unsigned type to negate this value to itself
> Fixes: signed integer overflow: -14527961 - 2147483425 cannot be represented
> in type 'int'
>
On Tue, Sep 10, 2019 at 01:36:49AM +0200, Michael Niedermayer wrote:
> Values larger would fail subsequent tests.
>
> Fixes: signed integer overflow: 5 + 2147483646 cannot be represented in type
> 'int'
> Fixes:
>
On Tue, Sep 03, 2019 at 02:39:27PM -0300, James Almer wrote:
> On 9/3/2019 8:13 AM, Michael Niedermayer wrote:
> > On Mon, Sep 02, 2019 at 09:40:47PM -0300, James Almer wrote:
> >> On 9/2/2019 9:14 PM, Michael Niedermayer wrote:
> >>> Fixes: OOM
> >>> Fixes:
> >>>
This ensures the default ycbcr_subsampling is 2 while also
ensuring the subsampling values are correct for all pixel formats.
This solution while it takes a few lines more code should be more
robust
In TIFF specification only CbCr subsampling is allowed. The field is explicitly named
On Thu, Sep 12, 2019 at 11:09:16PM +0200, Tomas Härdin wrote:
> tor 2019-09-12 klockan 00:21 +0200 skrev Michael Niedermayer:
> > On Wed, Sep 11, 2019 at 11:18:47PM +0200, Tomas Härdin wrote:
> > > tis 2019-09-10 klockan 16:16 +0200 skrev Michael Niedermayer:
> > > > On Mon, Sep 09, 2019 at
On Sat, Sep 28, 2019 at 03:09:35PM +0200, Paul B Mahol wrote:
> lgtm
will apply
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
The worst form of inequality is to try to make unequal things equal.
-- Aristotle
signature.asc
Description: PGP signature
On Sat, Sep 28, 2019 at 03:09:07PM +0200, Paul B Mahol wrote:
> lgtm
will apply
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Many that live deserve death. And some that die deserve life. Can you give
it to them? Then do not be too eager to deal out
On Sat, Sep 28, 2019 at 04:26:07AM +0200, Andreas Rheinhardt wrote:
> Fixes ticket #8160.
>
> Signed-off-by: Andreas Rheinhardt
> ---
> libavcodec/ituh263dec.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
will apply
thx
[...]
--
Michael GnuPG fingerprint:
On Sat, Sep 28, 2019 at 04:26:02AM +0200, Andreas Rheinhardt wrote:
> Affected the FATE-tests vsynth_lena-dv-411, vsynth1-dv-411,
> vsynth2-dv-411 and hevc-paramchange-yuv420p.yuv420p10.
>
> Signed-off-by: Andreas Rheinhardt
> ---
> libswscale/utils.c | 2 +-
> 1 file changed, 1 insertion(+), 1
On Sat, Sep 28, 2019 at 04:26:06AM +0200, Andreas Rheinhardt wrote:
> Affected 61 FATE-tests: 60 vsynth tests and lavf-mxf_opatom.
>
> Signed-off-by: Andreas Rheinhardt
> ---
> libavcodec/dnxhdenc.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
will apply
thx
[...]
--
On Sat, Sep 28, 2019 at 04:26:01AM +0200, Andreas Rheinhardt wrote:
> libavcodec/ffv1enc.c accessed an array of uint8_t [32] via array[0][j]
> in order to loop over all the uint8_t in this array of arrays. Of course
> this implied an out-of-bounds access for array[0] and UBSan complained
> about
On Sat, Sep 28, 2019 at 04:25:58AM +0200, Andreas Rheinhardt wrote:
> This affected many FATE-tests: The number of failing tests went down
> from 663 to 344. (Both numbers exclude tests that failed because of
> unaligned accesses in code that is inside #if HAVE_FAST_UNALIGNED.)
>
> Signed-off-by:
On 9/28/2019 12:05 PM, Paul B Mahol wrote:
> +static void frame_analysis(AudioRNNContext *s, DenoiseState *st,
> AVComplexFloat *X, float *Ex, const float *in)
> +{
> +LOCAL_ALIGNED_32(float, x, [WINDOW_SIZE]);
> +
> +RNN_COPY(x, st->analysis_mem, FRAME_SIZE);
> +RNN_COPY(x +
On Sat, Sep 28, 2019 at 04:26:10AM +0200, Andreas Rheinhardt wrote:
> Affected the vsynth*-jpeg2000 and the vsynth*-jpeg2000-97 FATE tests
> (where * ranges over { 1, 2, 3, _lena }) as well as ticket #7983.
>
> Signed-off-by: Andreas Rheinhardt
> ---
> libavcodec/jpeg2000dwt.c | 2 +-
> 1 file
On Sat, Sep 28, 2019 at 08:23:57PM +0800, Jun Zhao wrote:
> From: Jun Zhao
>
> add duration estimate method trace, it's can help to some duration
> issue trace or debug.
theres some english grammer issue here
>
> Signed-off-by: Jun Zhao
> ---
> libavformat/utils.c | 16 +++-
>
Signed-off-by: Paul B Mahol
---
doc/filters.texi | 11 +
libavfilter/Makefile |1 +
libavfilter/af_arnndn.c | 1544 ++
libavfilter/allfilters.c |1 +
4 files changed, 1557 insertions(+)
create mode 100644 libavfilter/af_arnndn.c
diff
tor 2019-09-26 klockan 23:10 +0200 skrev Michael Niedermayer:
> Fixes: Timeout(103sec -> 3sec)
> Fixes:
> 17678/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_FLIC_fuzzer-5715436989054976
>
> Found-by: continuous fuzzing process
>
lör 2019-09-28 klockan 04:26 +0200 skrev Andreas Rheinhardt:
> Fixes: left shift of 1 by 31 places cannot be represented in type 'int'.
> Affected the FATE-tests vsynth1-cinepak, vsynth2-cinepak and
> vsynth_lena-cinepak.
>
> Signed-off-by: Andreas Rheinhardt
> ---
> libavcodec/cinepakenc.c |
lgtm
On 9/28/19, Andreas Rheinhardt wrote:
> Affected every usage of vf_xbr, e.g. the FATE-tests filter-2xbr,
> filter-3xbr, filter-4xbr.
>
> Signed-off-by: Andreas Rheinhardt
> ---
> libavfilter/vf_xbr.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git
lgtm
On 9/28/19, Andreas Rheinhardt wrote:
> Affected every usage of this filter; in particular, it affected the
> FATE-tests filter-2xbr, filter-3xbr and filter-4xbr.
>
> Signed-off-by: Andreas Rheinhardt
> ---
> libavfilter/vf_hqx.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>
From: Jun Zhao
add duration estimate method trace, it's can help to some duration
issue trace or debug.
Signed-off-by: Jun Zhao
---
libavformat/utils.c | 16 +++-
1 files changed, 15 insertions(+), 1 deletions(-)
diff --git a/libavformat/utils.c b/libavformat/utils.c
index
From: Jun Zhao
in fact, nut demuxer use the PTS for duration estimation.
Signed-off-by: Jun Zhao
---
libavformat/utils.c |6 +-
1 files changed, 5 insertions(+), 1 deletions(-)
diff --git a/libavformat/utils.c b/libavformat/utils.c
index 45cd6b8..f62a5cb 100644
---
From: Jun Zhao
change the log level to warning if can't get duration, it's will help
to debug some duration issue
Signed-off-by: vacingfang
Signed-off-by: Jun Zhao
---
libavformat/utils.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavformat/utils.c
lgtm
On 9/28/19, Jun Zhao wrote:
> From: Jun Zhao
>
> Add logging context to log, it's will help debuging.
>
> Signed-off-by: Jun Zhao
> ---
> libavformat/nutdec.c |6 +++---
> 1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/libavformat/nutdec.c b/libavformat/nutdec.c
>
From: Jun Zhao
fix indentation for estimate_timings when dump start_time/duartion.
Signed-off-by: Jun Zhao
---
libavformat/utils.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/libavformat/utils.c b/libavformat/utils.c
index f62a5cb..c0ccd8c 100644
---
From: Jun Zhao
Add logging context to log, it's will help debuging.
Signed-off-by: Jun Zhao
---
libavformat/nutdec.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/libavformat/nutdec.c b/libavformat/nutdec.c
index 979cb9a..2b5adcb 100644
---
On 2019-09-27T19:37:40+0200, William Martin
wrote:
> On Wed, Sep 25, 2019 at 2:36 PM Reino Wijnsma wrote:
>> On 2019-09-25T20:57:08+0200, William Martin
>> wrote:
>>> +av_log(s, AV_LOG_ERROR, "App field don't match up: %s <-> %s. "
>> Although I'm not a native English speaker,
51 matches
Mail list logo
