new_init_section will return NULL under circumstances where the init
failed and this code needed to check for that. ret and goto fail are how
this is handled throughout the code.
---
libavformat/hls.c | 4
1 file changed, 4 insertions(+)
diff --git a/libavformat/hls.c b/libavformat/hls.c
ind
check that frame allocations return non-null.
---
libavfilter/dnn/dnn_backend_native.c | 13 -
1 file changed, 12 insertions(+), 1 deletion(-)
diff --git a/libavfilter/dnn/dnn_backend_native.c
b/libavfilter/dnn/dnn_backend_native.c
index d45e211f0c..7958ceb4f7 100644
--- a/libavfilte
6f69f7a8bf6a0d013985578df2ef42ee6b1c7994 introduced this and it was part
of a very large merging of refactoring. Current behaviour is what is
reflected by this indenting change, however my understanding of timing
is such that this correct behaviour.
---
libavformat/utils.c | 3 +--
1 file changed,
Chris Miceli:
> check that frame allocations return non-null.
> ---
> libavfilter/dnn/dnn_backend_native.c | 9 +
> 1 file changed, 9 insertions(+)
>
> diff --git a/libavfilter/dnn/dnn_backend_native.c
> b/libavfilter/dnn/dnn_backend_native.c
> index d45e211f0c..cda857a9c5 100644
> --- a
check that frame allocations return non-null.
---
libavfilter/dnn/dnn_backend_native.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/libavfilter/dnn/dnn_backend_native.c
b/libavfilter/dnn/dnn_backend_native.c
index d45e211f0c..cda857a9c5 100644
--- a/libavfilter/dnn/dnn_backend_nat
This commit fixes some unchecked memory allocations from #8931.
---
libavfilter/af_mcompand.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/libavfilter/af_mcompand.c b/libavfilter/af_mcompand.c
index f142573bea..96293909de 100644
--- a/libavfilter/af_mcompand.c
+++ b/libavfilter/af_
This commit fixes some unchecked memory allocations from #8931.
---
libavfilter/af_mcompand.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/libavfilter/af_mcompand.c b/libavfilter/af_mcompand.c
index f142573bea..96293909de 100644
--- a/libavfilter/af_mcompand.c
+++ b/libavfilter/af_
Memory allocation for AVIOContext should be checked. In this code,
all error conditions are sent to the "goto error".
---
libavformat/avidec.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/libavformat/avidec.c b/libavformat/avidec.c
index 9765e5e7b2..33e50e9878 100644
Chris Miceli:
> check that frame allocations return non-null.
> ---
> libavfilter/dnn/dnn_backend_native.c | 8
> 1 file changed, 8 insertions(+)
>
> diff --git a/libavfilter/dnn/dnn_backend_native.c
> b/libavfilter/dnn/dnn_backend_native.c
> index d45e211f0c..d1eb992335 100644
> --- a/
Please disregard this for the replacement titled:
libavfilter/dnn_backend_native: check mem allocation.
Thanks,
*Chris Miceli*
On Tue, Oct 13, 2020 at 1:34 PM Chris Miceli wrote:
> A report on Trac came in about this failure scenario where there are
> potential failures to allocate memory whic
check that frame allocations return non-null.
---
libavfilter/dnn/dnn_backend_native.c | 8
1 file changed, 8 insertions(+)
diff --git a/libavfilter/dnn/dnn_backend_native.c
b/libavfilter/dnn/dnn_backend_native.c
index d45e211f0c..d1eb992335 100644
--- a/libavfilter/dnn/dnn_backend_nati
Chris Miceli:
> This commit fixes some unchecked memory allocations from #8931.
> ---
> libavfilter/af_mcompand.c | 9 +
> 1 file changed, 9 insertions(+)
>
> diff --git a/libavfilter/af_mcompand.c b/libavfilter/af_mcompand.c
> index f142573bea..5309d884a5 100644
> --- a/libavfilter/af_mc
A report on Trac came in about this failure scenario where there are
potential failures to allocate memory which are going unchecked in the
code. This should fix that error in the same way that other parts of the
codebase already handle similar situations
---
libavfilter/dnn/dnn_backend_native.c |
This commit fixes some unchecked memory allocations from #8931.
---
libavfilter/af_mcompand.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/libavfilter/af_mcompand.c b/libavfilter/af_mcompand.c
index f142573bea..5309d884a5 100644
--- a/libavfilter/af_mcompand.c
+++ b/libavfilter/af_
Chris Miceli:
> ticket 8931 mentions some missing null pointer checks and so this is one
^
Nit: null pointer checks can be more than just checking allocations for
success. So how about "This commit fixes some of the unchecked
allocations mentioned in ticket #8931."
> diff of a few which are for
A report on Trac came in about this failure scenario where there are
potential failures to allocate memory which are going unchecked in the
code. This should fix that error in the same way that other parts of the
codebase already handle similar situations
---
libavfilter/dnn/dnn_backend_native.c |
ticket 8931 mentions some missing null pointer checks and so this is one
diff of a few which are for these fixes. There are previous code pieces
which return ENOMEM, so this should work as intended.
---
libavfilter/af_mcompand.c | 12
1 file changed, 12 insertions(+)
diff --git a/lib
> 2020年10月12日 下午11:43,Guangxin Xu 写道:
>
> On Mon, Oct 12, 2020 at 9:57 PM Steven Liu wrote:
>
>>
>>
>>> 在 2020年10月12日,21:45,Guangxin Xu 写道:
>>>
>>>
>>>
>>> On Mon, Oct 12, 2020 at 5:18 PM Steven Liu wrote:
>>> fix ticket: 8904
>>> parse the SPS from extradata and get profile_tier_level
On Sun, 11 Oct 2020 18:36:42 +0200
Thomas Mundt wrote:
> Hi Philip,
>
> Am Fr., 9. Okt. 2020 um 18:33 Uhr schrieb Philip Langdale
> >:
>
> > I've been sitting on this for a couple of years now, and I figured I
> > should just send it out. This is what I believe is a conceptually
> > correct
On Tue, Oct 13, 2020 at 12:07 AM Dave Evans wrote:
>
> This patch fixes the total failure to parse cues when style and region
> definition blocks are contained in the input file, and ensures those blocks
> are written to the output when copying.
>
Thank you for taking time to add a FATE test, but
This patch fixes the total failure to parse cues when style and region
definition blocks are contained in the input file, and ensures those blocks
are written to the output when copying.
The test attached needs to be added to samples at the path shown in the
patch in order to validate that the ori
mån 2020-10-12 klockan 21:31 +0200 skrev Jean-Baptiste Kempf:
> On Mon, 12 Oct 2020, at 20:34, Tomas Härdin wrote:
> > mån 2020-10-12 klockan 09:35 +0200 skrev Jean-Baptiste Kempf:
> > > +Contributors are considered "active contributors" if they have pushed
> > > more
> > > +than 20 patches in the
From: Andriy Gelman
In this error path ret still stores the number of bytes read in
ffurl_read().
Signed-off-by: Andriy Gelman
---
libavformat/rtsp.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c
index e9fca034b4..cb9fc31166 100644
--- a/libavforma
From: Andriy Gelman
Signed-off-by: Andriy Gelman
---
libavformat/rtspdec.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/libavformat/rtspdec.c b/libavformat/rtspdec.c
index 1ff84cdd56..5a383b5a19 100644
--- a/libavformat/rtspdec.c
+++ b/libavformat/rtspdec.c
@@ -694,12 +
From: Andriy Gelman
Make error check style consistent with rest of function.
---
libavformat/rtspdec.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavformat/rtspdec.c b/libavformat/rtspdec.c
index 7c617837ca..b9fbc45190 100644
--- a/libavformat/rtspdec.c
+++ b/libav
From: Andriy Gelman
In sdp_read_header() some ff_network_close() calls were missed.
Also in rtp_read_header() update comment to explain why a single
call to ff_network_close() is enough to cover all cases even if
sdp_read_header() returns an error.
Signed-off-by: Andriy Gelman
---
libavformat
From: Andriy Gelman
Fixes #6334
Signed-off-by: Andriy Gelman
---
libavformat/rtspdec.c | 14 +-
1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/libavformat/rtspdec.c b/libavformat/rtspdec.c
index 5a383b5a19..7c617837ca 100644
--- a/libavformat/rtspdec.c
+++ b/libavfor
From: Andriy Gelman
As per the docs network initialization is required before ff_url_join().
Furthermore, because the ff_network_init() was skipped, this makes
one additional call to ff_network_close() if the stream exits without
errors.
The was forgotten in the origin commit of the listen mode:
On Mon, 12 Oct 2020, at 20:34, Tomas Härdin wrote:
> mån 2020-10-12 klockan 09:35 +0200 skrev Jean-Baptiste Kempf:
> > +Contributors are considered "active contributors" if they have pushed more
> > +than 20 patches in the last 36 months in the main FFmpeg repository, or
> > +if they have been vote
On Mon, 12 Oct 2020, at 19:25, Michael Niedermayer wrote:
> On Tue, Oct 06, 2020 at 04:50:19PM +0200, Nicolas George wrote:
> > Jean-Baptiste Kempf (12020-10-05):
> > > General Assembly + Main Elections
> > > ---
> > > doc/dev_community/community.md | 60 ++
> > >
applied
___
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".
On Sat, Oct 10, 2020 at 05:41:26PM +0200, Michael Niedermayer wrote:
> Signed-off-by: Michael Niedermayer
> ---
> Makefile | 4
> tools/Makefile| 3 +++
> tools/target_dem_fuzzer.c | 11 ++-
> 3 files changed, 17 insertions(+), 1 deletion(-)
will apply
tis 2020-10-06 klockan 16:20 +0200 skrev Michael Niedermayer:
> On Mon, Oct 05, 2020 at 10:18:32AM +0200, Tomas Härdin wrote:
> > tor 2020-10-01 klockan 22:13 +0200 skrev Michael Niedermayer:
> > > On Thu, Oct 01, 2020 at 03:29:19PM +0100, Harry Mallon wrote:
> > > > > On 30 Sep 2020, at 08:32, Mic
On Mon, Oct 12, 2020 at 07:01:17PM +0200, Romane Lafon wrote:
> This patch extends aphasemeter to detect out of phase or mono sequences in
> stereo streams. Same patch as v8, but doc updated with the latest master
> branch.
>
probably fine, gonna apply, if I forgot, ping me.
> Regards,
> Romane
mån 2020-10-12 klockan 09:35 +0200 skrev Jean-Baptiste Kempf:
> +Contributors are considered "active contributors" if they have pushed more
> +than 20 patches in the last 36 months in the main FFmpeg repository, or
> +if they have been voted in by the GA.
This might be a problem for parts of the c
Hi
On Mon, Oct 12, 2020 at 10:12:14AM +0200, Andreas Rheinhardt wrote:
> This is easily possible because ff_init_vlc_sparse() already transforms
> both LE as well as BE codes to a normal form internally before
> processing them further. This will be used in subsequent commits.
>
> Signed-off-by:
This patch extends aphasemeter to detect out of phase or mono sequences in
stereo streams. Same patch as v8, but doc updated with the latest master
branch.
Regards,
Romane
From 10bcfc2652514e9e212c5a643e83f6614c8017bd Mon Sep 17 00:00:00 2001
From: Romane Lafon
Date: Mon, 12 Oct 2020 18:14:55 +02
On Tue, Oct 06, 2020 at 04:50:19PM +0200, Nicolas George wrote:
> Jean-Baptiste Kempf (12020-10-05):
> > General Assembly + Main Elections
> > ---
> > doc/dev_community/community.md | 60 ++
> > 1 file changed, 60 insertions(+)
> > create mode 100644 doc/dev_commun
On Sun, 11. Oct 22:04, Andriy Gelman wrote:
> On Sun, 11. Oct 22:44, Andreas Rheinhardt wrote:
> > Andriy Gelman:
> > > From: Andriy Gelman
> > >
> > > Fixes #6334
> > >
> > > Signed-off-by: Andriy Gelman
> > > ---
> > > libavformat/rtspdec.c | 17 ++---
> > > 1 file changed, 10 in
Fixes: out of array access
Fixes:
26087/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_AAC_FIXED_fuzzer-5724825462767616
Found-by: continuous fuzzing process
https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer
---
libavcodec/aacdec_fixed.c | 4 +
On Mon, Oct 12, 2020 at 9:57 PM Steven Liu wrote:
>
>
> > 在 2020年10月12日,21:45,Guangxin Xu 写道:
> >
> >
> >
> > On Mon, Oct 12, 2020 at 5:18 PM Steven Liu wrote:
> > fix ticket: 8904
> > parse the SPS from extradata and get profile_tier_level
> > write the profile_tier_level info into CODECS Attr
Am 12.10.20 um 11:34 schrieb Jean-Baptiste Kempf:
>
> On Mon, 12 Oct 2020, at 11:28, Steven Liu wrote:
>>> +Voting is done using a ranked voting system, currently running on
>>> https://vote.ffmpeg.org/ .
>> Hi JB,
>>
>> I cannot sure if vote.ffmpeg.org denny my IP,
>> When I click this
On Mon, Oct 12, 2020 at 5:18 PM Steven Liu wrote:
> fix ticket: 8904
> parse the SPS from extradata and get profile_tier_level
> write the profile_tier_level info into CODECS Attribute
>
> reference to :
> https://developer.apple.com/documentation/http_live_streaming/hls_authoring_specification_f
> 在 2020年10月12日,21:45,Guangxin Xu 写道:
>
>
>
> On Mon, Oct 12, 2020 at 5:18 PM Steven Liu wrote:
> fix ticket: 8904
> parse the SPS from extradata and get profile_tier_level
> write the profile_tier_level info into CODECS Attribute
>
> reference to
> :https://developer.apple.com/documentati
Signed-off-by: leozhang
---
libavfilter/vf_scale_cuda.c | 8
1 file changed, 8 insertions(+)
diff --git a/libavfilter/vf_scale_cuda.c b/libavfilter/vf_scale_cuda.c
index 1ffb73f831..46d69b00ee 100644
--- a/libavfilter/vf_scale_cuda.c
+++ b/libavfilter/vf_scale_cuda.c
@@ -119,7 +119,15 @
On Mon, Oct 12, 2020 at 10:12:15AM +0200, Andreas Rheinhardt wrote:
> Signed-off-by: Andreas Rheinhardt
> ---
> libavcodec/speedhq.c | 41 +
> 1 file changed, 9 insertions(+), 32 deletions(-)
lgtm
___
ffmpeg-deve
On Mon, Oct 12, 2020 at 10:12:11AM +0200, Andreas Rheinhardt wrote:
> Before the LE bitstream reader was used in the Indeo 2 decoder,
> a standard BE bitstream reader with swapped bits was used; when the LE
> bitstream reader was added, the old code was only #ifdef'ed away and not
> removed. Said c
On Mon, Oct 12, 2020 at 10:12:16AM +0200, Andreas Rheinhardt wrote:
> Signed-off-by: Andreas Rheinhardt
> ---
> libavcodec/ivi.c | 22 ++
> 1 file changed, 2 insertions(+), 20 deletions(-)
lgtm
___
ffmpeg-devel mailing list
ffmpeg-d
On Mon, Oct 12, 2020 at 10:12:17AM +0200, Andreas Rheinhardt wrote:
> The WebP format uses Huffman tables and the decoder therefore uses
> VLC tables. Given that WebP is a LE format, a LE bitreader is used;
> yet the VLC table is not created for a LE reader (the process used to
> create the tables
On Mon, Oct 12, 2020 at 10:12:13AM +0200, Andreas Rheinhardt wrote:
> Signed-off-by: Andreas Rheinhardt
> ---
> libavcodec/speedhq.c | 34 +-
> 1 file changed, 17 insertions(+), 17 deletions(-)
lgtm
>
> diff --git a/libavcodec/speedhq.c b/libavcodec/speedhq.c
>
On Mon, 12 Oct 2020, at 11:28, Steven Liu wrote:
> > +Voting is done using a ranked voting system, currently running on
> > https://vote.ffmpeg.org/ .
> Hi JB,
>
> I cannot sure if vote.ffmpeg.org denny my IP,
> When I click this link, the web return:
> 502 Bad Gateway
>
> ngi
> 2020年10月12日 下午3:35,Jean-Baptiste Kempf 写道:
>
> General Assembly + Main Elections
> ---
> doc/dev_community/community.md | 80 ++
> 1 file changed, 80 insertions(+)
> create mode 100644 doc/dev_community/community.md
>
> diff --git a/doc/dev_community/community.
fix ticket: 8904
parse the SPS from extradata and get profile_tier_level
write the profile_tier_level info into CODECS Attribute
reference to
:https://developer.apple.com/documentation/http_live_streaming/hls_authoring_specification_for_apple_devices/hls_authoring_specification_for_apple_devices_
This is easily possible because ff_init_vlc_sparse() already transforms
both LE as well as BE codes to a normal form internally before
processing them further. This will be used in subsequent commits.
Signed-off-by: Andreas Rheinhardt
---
libavcodec/bitstream.c | 6 +++---
libavcodec/vlc.h
Signed-off-by: Andreas Rheinhardt
---
libavcodec/speedhq.c | 41 +
1 file changed, 9 insertions(+), 32 deletions(-)
diff --git a/libavcodec/speedhq.c b/libavcodec/speedhq.c
index 5759cdb036..30a6924321 100644
--- a/libavcodec/speedhq.c
+++ b/libavcodec/spe
It can't, because the tables used don't have any loose ends. This also
fixes a bug in the only caller of decode_dc_le(): It didn't check the
return value.
Signed-off-by: Andreas Rheinhardt
---
libavcodec/speedhq.c | 4
1 file changed, 4 deletions(-)
diff --git a/libavcodec/speedhq.c b/liba
Signed-off-by: Andreas Rheinhardt
---
libavcodec/ivi.c | 22 ++
1 file changed, 2 insertions(+), 20 deletions(-)
diff --git a/libavcodec/ivi.c b/libavcodec/ivi.c
index c10984e83e..5e1180a5f0 100644
--- a/libavcodec/ivi.c
+++ b/libavcodec/ivi.c
@@ -35,7 +35,6 @@
#include "avc
Signed-off-by: Andreas Rheinhardt
---
libavcodec/speedhq.c | 34 +-
1 file changed, 17 insertions(+), 17 deletions(-)
diff --git a/libavcodec/speedhq.c b/libavcodec/speedhq.c
index 64758164de..5759cdb036 100644
--- a/libavcodec/speedhq.c
+++ b/libavcodec/speedhq.c
The WebP format uses Huffman tables and the decoder therefore uses
VLC tables. Given that WebP is a LE format, a LE bitreader is used;
yet the VLC table is not created for a LE reader (the process used to
create the tables puts the last bit to be read in the lowest bit) and
therefore custom code fo
Before the LE bitstream reader was used in the Indeo 2 decoder,
a standard BE bitstream reader with swapped bits was used; when the LE
bitstream reader was added, the old code was only #ifdef'ed away and not
removed. Said code has several problems: It modifies the input packet
without ensuring that
On Tue, 6 Oct 2020, at 16:50, Nicolas George wrote:
> Global nit: I think it would be more readable without a Markdown engine
> if it were pre-wrapped.
Done.
> > +Contributors are considered "active contributors" if they have pushed more
> > than 20 patches in the last 36 months in the main FFm
Hello,
On Tue, 6 Oct 2020, at 05:33, Chris Miceli wrote:
> - potentially rename Technical Committee to Technical Conflicts Committee to
> avoid the confusion you documented around it's position describing technical
> direction
This is not the usage in other communities.
> - "They will consid
General Assembly + Main Elections
---
doc/dev_community/community.md | 80 ++
1 file changed, 80 insertions(+)
create mode 100644 doc/dev_community/community.md
diff --git a/doc/dev_community/community.md b/doc/dev_community/community.md
new file mode 100644
index
63 matches
Mail list logo