On 3/29/2022 6:24 PM, Michael Niedermayer wrote:
Fixes: memleak
Fixes: 
45982/clusterfuzz-testcase-minimized-ffmpeg_BSF_PCM_RECHUNK_fuzzer-5562089618407424

Found-by: continuous fuzzing process 
https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc>
---
  libavcodec/pcm_rechunk_bsf.c | 1 +
  1 file changed, 1 insertion(+)

diff --git a/libavcodec/pcm_rechunk_bsf.c b/libavcodec/pcm_rechunk_bsf.c
index 108d9e90b9..3f43934fe9 100644
--- a/libavcodec/pcm_rechunk_bsf.c
+++ b/libavcodec/pcm_rechunk_bsf.c
@@ -153,6 +153,7 @@ static int rechunk_filter(AVBSFContext *ctx, AVPacket *pkt)
              }
          }
+ av_packet_unref(s->in_pkt);

This looks to me like it revealed a bug in the code above, which is meant to ensure s->in_pkt will be blank at this point. It should be fixed there instead.

          ret = ff_bsf_get_packet_ref(ctx, s->in_pkt);
          if (ret == AVERROR_EOF && s->out_pkt->size) {
              if (s->pad) {
_______________________________________________
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".

Reply via email to