The variable AVFrame *frame could be a null pointer, now add a null
pointer check to avoid dereferencing the null pointer.
Signed-off-by: Tong Wu
---
libavcodec/qsvenc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c
index
add a return value check for function MFXQueryIMPL to handle the error
message.
Signed-off-by: Tong Wu
---
libavcodec/qsv.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/libavcodec/qsv.c b/libavcodec/qsv.c
index 1a432dbd82..18f34f05fd 100644
--- a/libavcodec/qsv.c
+++
This also follows the suggested API usage in the doxy.
Signed-off-by: James Almer
---
tools/target_bsf_fuzzer.c | 15 ++-
1 file changed, 6 insertions(+), 9 deletions(-)
diff --git a/tools/target_bsf_fuzzer.c b/tools/target_bsf_fuzzer.c
index d6aaee3bd9..1aae4a12e5 100644
---
Fixes: Timeout
Fixes:
44877/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_TARGA_fuzzer-4870505251864576
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 +
1 file
Fixes: NULL pointer dereference
Fixes:
44884/clusterfuzz-testcase-minimized-ffmpeg_DEMUXER_fuzzer-4656748688965632
Found-by: continuous fuzzing process
https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer
---
tools/target_dem_fuzzer.c | 2 +-
1
Fixes: memleaks
Fixes:
44810/clusterfuzz-testcase-minimized-ffmpeg_dem_IVR_fuzzer-5619494647627776
Found-by: continuous fuzzing process
https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer
---
libavformat/rmdec.c | 9 +
1 file changed, 5
Fixes segfaults upon allocation failure.
Signed-off-by: Andreas Rheinhardt
---
fftools/ffmpeg.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c
index 7beea11933..e9de8fb430 100644
--- a/fftools/ffmpeg.c
+++ b/fftools/ffmpeg.c
@@
Signed-off-by: Andreas Rheinhardt
---
fftools/ffmpeg_opt.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c
index c22c6aca06..44ec759e35 100644
--- a/fftools/ffmpeg_opt.c
+++ b/fftools/ffmpeg_opt.c
@@ -3271,9 +3271,8 @@ static
Signed-off-by: Andreas Rheinhardt
---
fftools/ffmpeg_opt.c | 14 +++---
1 file changed, 3 insertions(+), 11 deletions(-)
diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c
index 3102851885..c22c6aca06 100644
--- a/fftools/ffmpeg_opt.c
+++ b/fftools/ffmpeg_opt.c
@@ -784,7 +784,7 @@
The range parameters need to be set up before calling
sws_init_context (which selects which fastpaths can be used;
this gets called by sws_getContext); solely passing them via
sws_setColorspaceDetails isn't enough.
This fixes producing full range YUV range output when doing
YUV->YUV conversions
On Wed, 23 Feb 2022, Andreas Rheinhardt wrote:
Martin Storsjö:
On Tue, 15 Feb 2022, Andreas Rheinhardt wrote:
Forgotten in e7bd47e657bbf9e1ce9915e93bc80cb1a29fb7f3.
Signed-off-by: Andreas Rheinhardt
---
libavcodec/vc2enc.c | 2 +-
libavcodec/x86/blockdsp_init.c | 2 --
Greetings
I encountered some RPL acorn replay video files of a compression type 7
that ffmpeg doesn't have support for.
I submitted a bug report here: https://trac.ffmpeg.org/ticket/9655
It includes sample files and a link to the file format.
I'd love to see support added to ffmpeg for this
On 2/19/22 7:09 PM, Stephen Hutchinson wrote:
It's just a simple index.
Addresses Coverity issue 1500290
Signed-off-by: Stephen Hutchinson
---
libavformat/avisynth.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavformat/avisynth.c b/libavformat/avisynth.c
On 2/19/22 4:41 PM, Stephen Hutchinson wrote:
Since the check got simplified and stdbool was no longer necessary
to include, neither is that variable. Silences a warning.
Signed-off-by: Stephen Hutchinson
---
libavformat/avisynth.c | 1 -
1 file changed, 1 deletion(-)
diff --git
On Wed, Feb 23, 2022 at 5:22 PM Michael Niedermayer
wrote:
> On Mon, Feb 21, 2022 at 07:24:15PM +0100, Michael Niedermayer wrote:
> > On Mon, Feb 21, 2022 at 08:59:16AM -0800, Pierre-Anthony Lemieux wrote:
> > > Hi Michael,
> > >
> > > What is the typical size of a successful GSOC project? Any
On Mon, Feb 21, 2022 at 07:24:15PM +0100, Michael Niedermayer wrote:
> On Mon, Feb 21, 2022 at 08:59:16AM -0800, Pierre-Anthony Lemieux wrote:
> > Hi Michael,
> >
> > What is the typical size of a successful GSOC project? Any good
> > example(s) from the past?
>
> for past Results
> see: (and
Martin Storsjö:
> On Tue, 15 Feb 2022, Andreas Rheinhardt wrote:
>
>> Signed-off-by: Andreas Rheinhardt
>> ---
>> libavformat/avio.h | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/libavformat/avio.h b/libavformat/avio.h
>> index cd63322a62..ca970b1ce3 100644
>> ---
Martin Storsjö:
> On Tue, 15 Feb 2022, Andreas Rheinhardt wrote:
>
>> Forgotten in e7bd47e657bbf9e1ce9915e93bc80cb1a29fb7f3.
>>
>> Signed-off-by: Andreas Rheinhardt
>> ---
>> libavcodec/vc2enc.c | 2 +-
>> libavcodec/x86/blockdsp_init.c | 2 --
>> libavfilter/internal.h | 1 -
>>
Signed-off-by: James Almer
---
No changes since last version. Already LGTM'd by Paul.
libavcodec/setts_bsf.c | 25 -
1 file changed, 24 insertions(+), 1 deletion(-)
diff --git a/libavcodec/setts_bsf.c b/libavcodec/setts_bsf.c
index f0243a1114..eadc4e4d34 100644
---
They correspond to the relevant fields from the packet that follows the
one where the expressions are being applied.
Signed-off-by: James Almer
---
No changes since last version. Already LGTM'd by Paul.
libavcodec/setts_bsf.c | 75 --
1 file changed, 50
A keyframe could be buffered in the bsf and not be output until more packets
had been fed to it.
Signed-off-by: James Almer
---
Changed the check from pkt to !eof, since a packet is always provided.
fftools/ffmpeg.c | 4 +++-
fftools/ffmpeg.h | 1 +
2 files changed, 4 insertions(+), 1
Bitstream filters inserted between the input and output were
never drained, resulting packets being lost if the bsf had any
buffered.
Signed-off-by: James Almer
---
Now also flushing packets when forcing a record duration.
fftools/ffmpeg.c| 13 -
fftools/ffmpeg.h|
On Tue, 15 Feb 2022, Andreas Rheinhardt wrote:
Some of these were made possible by moving several common macros to
libavutil/macros.h.
While just at it, also improve the other headers a bit.
Signed-off-by: Andreas Rheinhardt
---
libavcodec/ac3.c | 3 ++-
On Tue, 15 Feb 2022, Andreas Rheinhardt wrote:
Signed-off-by: Andreas Rheinhardt
---
libavcodec/dct.c | 2 ++
libavcodec/mpegaudiodec_common.c | 1 +
libavcodec/mqcenc.c | 2 ++
libavcodec/put_bits.h | 1 +
libavcodec/rdft.c | 1 +
Previously, the default timebase caused two warnings during decoding about not
being able to update timestamps for skipped and discarded samples, respectively.
Signed-off-by: Andreas Unterweger
---
doc/examples/transcode_aac.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff
The last encoded frame is now fetched on EOF. It was previously left in the encoder and
caused a "1 frame left in queue" warning.
Signed-off-by: Andreas Unterweger
---
doc/examples/transcode_aac.c | 22 +++---
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git
On Tue, 15 Feb 2022, Andreas Rheinhardt wrote:
Forgotten in e7bd47e657bbf9e1ce9915e93bc80cb1a29fb7f3.
Signed-off-by: Andreas Rheinhardt
---
libavcodec/vc2enc.c| 2 +-
libavcodec/x86/blockdsp_init.c | 2 --
libavfilter/internal.h | 1 -
libavfilter/vf_swapuv.c| 1 -
On Tue, 15 Feb 2022, Andreas Rheinhardt wrote:
Signed-off-by: Andreas Rheinhardt
---
libavutil/display.c | 1 +
libavutil/display.h | 1 -
libavutil/tests/display.c | 1 +
3 files changed, 2 insertions(+), 1 deletion(-)
LGTM
// Martin
On Tue, 15 Feb 2022, Andreas Rheinhardt wrote:
Signed-off-by: Andreas Rheinhardt
---
libavutil/integer.c | 4 +++-
libavutil/integer.h | 2 +-
2 files changed, 4 insertions(+), 2 deletions(-)
LGTM
// Martin
___
ffmpeg-devel mailing list
On Tue, 15 Feb 2022, Andreas Rheinhardt wrote:
Signed-off-by: Andreas Rheinhardt
---
libavutil/pixelutils.h | 1 -
1 file changed, 1 deletion(-)
LGTM
// Martin
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
On Tue, 15 Feb 2022, Andreas Rheinhardt wrote:
Signed-off-by: Andreas Rheinhardt
---
libavutil/samplefmt.c | 6 --
libavutil/samplefmt.h | 3 ---
2 files changed, 4 insertions(+), 5 deletions(-)
LGTM
// Martin
___
ffmpeg-devel mailing list
On Tue, 15 Feb 2022, Andreas Rheinhardt wrote:
It is a remnant of an FF_API_* inclusion (back from when they were in
avutil.h and not in version.h).
Signed-off-by: Andreas Rheinhardt
---
libavutil/imgutils.h | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
LGTM
// Martin
On Tue, 15 Feb 2022, Andreas Rheinhardt wrote:
It has been added for an FF_API_* at a time when these were in avutil.h.
Signed-off-by: Andreas Rheinhardt
---
libavutil/eval.h | 2 --
1 file changed, 2 deletions(-)
LGTM
// Martin
___
ffmpeg-devel
On Tue, 15 Feb 2022, Andreas Rheinhardt wrote:
Signed-off-by: Andreas Rheinhardt
---
libavutil/file.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
LGTM
// Martin
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
On Tue, 15 Feb 2022, Andreas Rheinhardt wrote:
Signed-off-by: Andreas Rheinhardt
---
libavutil/fifo.c | 5 -
libavutil/fifo.h | 4 +++-
libavutil/threadmessage.c | 2 ++
3 files changed, 9 insertions(+), 2 deletions(-)
LGTM
// Martin
On Tue, 15 Feb 2022, Andreas Rheinhardt wrote:
Signed-off-by: Andreas Rheinhardt
---
libavutil/audio_fifo.c | 7 +--
libavutil/audio_fifo.h | 2 +-
2 files changed, 6 insertions(+), 3 deletions(-)
LGTM
// Martin
___
ffmpeg-devel mailing list
On Tue, 15 Feb 2022, Andreas Rheinhardt wrote:
It has been included since af5f434f8c0fb3b4ee3b206ebc1946ca660a8abe
for deprecation reasons, but removing it has been forgotten after
it had served is purpose. So remove it.
For convenience, include version.h instead as LIBAVUTIL_VERSION_INT
is
On Tue, 15 Feb 2022, Andreas Rheinhardt wrote:
Signed-off-by: Andreas Rheinhardt
---
libavformat/avio.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/libavformat/avio.h b/libavformat/avio.h
index cd63322a62..ca970b1ce3 100644
--- a/libavformat/avio.h
+++
Signed-off-by: Martin Storsjö
---
.gitignore | 1 -
Makefile | 2 +-
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/.gitignore b/.gitignore
index 1a5bb29ad5..f06d853edf 100644
--- a/.gitignore
+++ b/.gitignore
@@ -36,7 +36,6 @@
/config.asm
/config.h
/coverage.info
This avoids unnecessary rebuilds of most source files if only the
list of enabled components has changed, but not the other properties
of the build, set in config.h.
---
configure | 17 +++--
fftools/ffplay.c | 1 +
libavcodec/8svx.c
---
doc/APIchanges | 6 ++
1 file changed, 6 insertions(+)
diff --git a/doc/APIchanges b/doc/APIchanges
index ea402f6118..adbfc79e13 100644
--- a/doc/APIchanges
+++ b/doc/APIchanges
@@ -14,6 +14,12 @@ libavutil: 2021-04-27
API changes, most recent first:
+2022-*-* - xx - all
---
fftools/cmdutils.c | 1 +
fftools/ffprobe.c | 1 +
libavfilter/Makefile| 1 +
libavfilter/avfilter.c | 1 +
libavfilter/avfilter.h | 2 +-
libavfilter/internal.h | 1 -
libavfilter/version.h | 13 ++--
libavfilter/version_major.h |
---
fftools/cmdutils.c | 1 +
fftools/ffprobe.c | 1 +
libavdevice/Makefile| 1 +
libavdevice/avdevice.c | 1 +
libavdevice/avdevice.h | 2 +-
libavdevice/version.h | 10 ++
libavdevice/version_major.h | 37
---
fftools/cmdutils.c| 1 +
fftools/ffprobe.c | 1 +
libswscale/Makefile | 1 +
libswscale/swscale.h | 2 +-
libswscale/swscale_internal.h | 1 -
libswscale/utils.c| 1 +
libswscale/version.h | 9 ++---
---
fftools/cmdutils.c| 1 +
fftools/ffprobe.c | 1 +
libswresample/Makefile| 1 +
libswresample/swresample.c| 1 +
libswresample/swresample.h| 2 +-
libswresample/version.h | 3 ++-
libswresample/version_major.h | 31
---
fftools/cmdutils.c| 1 +
fftools/ffprobe.c | 1 +
libavdevice/pulse_audio_dec.c | 1 +
libavdevice/pulse_audio_enc.c | 1 +
libavformat/Makefile | 1 +
libavformat/avformat.h| 2 +-
libavformat/avio.h| 2 +-
libavformat/flacenc.c
This avoids including version.h in all source files, avoiding
unnecessary rebuilds when the version number is bumped. Only
version_major.h is included by the main header, which defines
availability of e.g. FF_API_* macros, and which is bumped much
less often.
---
fftools/cmdutils.c |
---
fftools/cmdutils.c | 1 +
fftools/ffprobe.c | 1 +
libpostproc/Makefile| 1 +
libpostproc/postprocess.c | 1 +
libpostproc/postprocess.h | 2 +-
libpostproc/version.h | 3 ++-
libpostproc/version_major.h | 31 +++
7 files
---
libavdevice/android_camera.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/libavdevice/android_camera.c b/libavdevice/android_camera.c
index f089d1b6f9..1934999c18 100644
--- a/libavdevice/android_camera.c
+++ b/libavdevice/android_camera.c
@@ -43,8 +43,6 @@
#include
This header probably has used defines like FF_API_* before,
but no longer do that, and doesn't directly seem to use anything
else from that header either.
---
libavformat/url.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/libavformat/url.h b/libavformat/url.h
index a129150d76..3cfe3ecc5c
These headers probably have used defines like FF_API_* before,
but no longer do that, and doesn't directly seem to use anything
else from that header either.
---
libavutil/common.h | 1 -
libavutil/internal.h | 1 -
2 files changed, 2 deletions(-)
diff --git a/libavutil/common.h
These files probably have used defines like FF_API_* before,
but no longer do that, and doesn't directly seem to use anything
else from that header either.
---
libavcodec/mediacodec.c | 1 -
libavcodec/mediacodec_wrapper.c | 1 -
libavcodec/x86/blockdsp_init.c | 1 -
libavcodec/xvmc.h
Hi,
When updating the ffmpeg source, one quite often ends up in a situation
where practically all of the codebase (or all of a library) gets rebuilt,
due to updates to headers that are included in most files.
In some cases, full rebuilds are warranted of course, but they could also
be avoided in
While trying to detect getauxval, this actually never output
HAVE_GETAUXVAL into config.h before.
Signed-off-by: Martin Storsjö
---
Fixed the alphabetical ordering.
---
configure | 1 +
1 file changed, 1 insertion(+)
diff --git a/configure b/configure
index 4f30140221..e8fdb3813d 100755
---
Martin Storsjö:
> While trying to detect getauxval, this actually never output
> HAVE_GETAUXVAL into config.h before.
>
> Signed-off-by: Martin Storsjö
> ---
> configure | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/configure b/configure
> index 4f30140221..d4502ba90c 100755
> ---
While trying to detect getauxval, this actually never output
HAVE_GETAUXVAL into config.h before.
Signed-off-by: Martin Storsjö
---
configure | 1 +
1 file changed, 1 insertion(+)
diff --git a/configure b/configure
index 4f30140221..d4502ba90c 100755
--- a/configure
+++ b/configure
@@ -2266,6
> -Original Message-
> From: ffmpeg-devel On Behalf Of Wang,
> Fei W
> Sent: Tuesday, February 22, 2022 4:49 PM
> To: ffmpeg-devel@ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH v2 1/4] lavc/vaapi_encode_h265: Add P
> frame to GPB frame support for hevc_vaapi
>
> On Tue, 2022-02-22
On Wed, Feb 23, 2022 at 10:31:59AM +0800, Jiasheng Jiang wrote:
> As the potential failure of the memory allocation,
> the avformat_new_stream() could return NULL pointer.
> Therefore, it should be better to check it and return
> error if fails.
> Also, the caller, nut_read_header(), needs to deal
On 2/23/2022 7:24 AM, Anton Khirnov wrote:
Quoting James Almer (2022-01-13 03:04:50)
From: Anton Khirnov
Signed-off-by: Vittorio Giovara
Signed-off-by: Anton Khirnov
Signed-off-by: James Almer
---
libavcodec/flac.c| 39 +--
On 23/2/22 19:54, Lynne wrote:
23 Feb 2022, 07:48 by z...@zanevaniperen.com:
Why the double header in the header file? It doesn't contain
any libuuid code.
It does, this was copy/pasted from libuuid, then had things stripped from it.
Some examples:
-
Quoting James Almer (2022-01-13 03:04:50)
> From: Anton Khirnov
>
> Signed-off-by: Vittorio Giovara
> Signed-off-by: Anton Khirnov
> Signed-off-by: James Almer
> ---
> libavcodec/flac.c| 39 +--
> libavcodec/flac.h| 2 +-
>
23 Feb 2022, 07:48 by z...@zanevaniperen.com:
>> Why the double header in the header file? It doesn't contain
>> any libuuid code.
>>
>
> It does, this was copy/pasted from libuuid, then had things stripped from it.
> Some examples:
> -
>
From: Wu Jianhua
ff_hevc_put_hevc_qpel_h64_8_sse4 56782981
ff_hevc_put_hevc_qpel_h64_8_avx2 40097816
ff_hevc_put_hevc_qpel_h64_8_avx512icl 25488576
Signed-off-by: Wu Jianhua
---
libavcodec/x86/hevc_mc.asm| 12 +++-
libavcodec/x86/hevcdsp.h | 1 +
From: Wu Jianhua
ff_hevc_put_hevc_qpel_h32_8_sse4 14122151
ff_hevc_put_hevc_qpel_h32_8_avx29337675
ff_hevc_put_hevc_qpel_h32_8_avx512icl 6424654
Signed-off-by: Wu Jianhua
---
libavcodec/x86/hevc_mc.asm| 7 +++
libavcodec/x86/hevcdsp.h | 1 +
From: Wu Jianhua
ff_hevc_put_hevc_qpel_h4_8_sse4 993694
ff_hevc_put_hevc_qpel_h4_8_avx512icl 686647
Signed-off-by: Wu Jianhua
---
libavcodec/x86/hevc_mc.asm| 12 ++--
libavcodec/x86/hevcdsp.h | 1 +
libavcodec/x86/hevcdsp_init.c | 1 +
3 files changed, 12
From: Wu Jianhua
ff_hevc_put_hevc_qpel_h16_8_sse4 3290870
ff_hevc_put_hevc_qpel_h16_8_avx512icl 1730033
Signed-off-by: Wu Jianhua
---
libavcodec/x86/hevc_mc.asm| 39 ++-
libavcodec/x86/hevcdsp.h | 1 +
libavcodec/x86/hevcdsp_init.c | 1 +
3
From: Wu Jianhua
This commit uses the instruction `vpdpbusd` introduced by AVX512 VNNI
to calculate the horizontal filter.
ff_hevc_put_hevc_qpel_h8_8_sse4 1039169
ff_hevc_put_hevc_qpel_h8_8_avx512icl 677153
ff_hevc_put_hevc_qpel_hv8_8_sse4 3603511
From: Wu Jianhua
Signed-off-by: Wu Jianhua
---
configure | 13 +++---
libavutil/cpu.c | 1 +
libavutil/cpu.h | 1 +
libavutil/x86/cpu.c | 8 --
libavutil/x86/cpu.h | 1 +
libavutil/x86/x86inc.asm | 53
lgtm
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
lgtm
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
lgtm
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
71 matches
Mail list logo
