Re: [FFmpeg-devel] [PATCH 2/3] libavcodec/mlp_parser.c: fix a use_of_uninitialized_value in target_dec_fuzzer.

2020-06-05 Thread Michael Niedermayer
On Thu, Jun 04, 2020 at 01:03:15PM -0700, Thierry Foucu wrote:
> the target_dec_fuzzer is checking for the avpkt.data pointer but if the
> mlp parser cannot combine the frame, the poutbuf is not set.
> ---
>  libavcodec/mlp_parser.c | 1 +
>  1 file changed, 1 insertion(+)

will apply

thx

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The educated differ from the uneducated as much as the living from the
dead. -- Aristotle 


signature.asc
Description: PGP signature
___
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".

[FFmpeg-devel] [PATCH 2/3] libavcodec/mlp_parser.c: fix a use_of_uninitialized_value in target_dec_fuzzer.

2020-06-04 Thread Thierry Foucu
the target_dec_fuzzer is checking for the avpkt.data pointer but if the
mlp parser cannot combine the frame, the poutbuf is not set.
---
 libavcodec/mlp_parser.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/libavcodec/mlp_parser.c b/libavcodec/mlp_parser.c
index 5d2ddc5a70..e7162f4aa8 100644
--- a/libavcodec/mlp_parser.c
+++ b/libavcodec/mlp_parser.c
@@ -64,6 +64,7 @@ static int mlp_parse(AVCodecParserContext *s,
 s->key_frame = 0;
 
 *poutbuf_size = 0;
+*poutbuf = NULL;
 if (buf_size == 0)
 return 0;
 
-- 
2.27.0.278.ge193c7cf3a9-goog

___
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".