summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndreas Rheinhardt <andreas.rheinhardt@outlook.com>2022-03-31 12:38:24 +0200
committerAndreas Rheinhardt <andreas.rheinhardt@outlook.com>2022-04-05 22:19:50 +0200
commitebec6079414262fdc6e161b73f4ac38cd6e4a276 (patch)
tree17af1f2ca0bfb0379b1c3180d22040b5dbb0eaee
parenta68a6322b78c27a58ab32174814efadf79701226 (diff)
avcodec/vp9_raw_reorder_bsf: Fix leak of cached packet
In case the BSF has not been drained before flushing/closing, the context's next_frame might be set; yet it is not freed in flush or close. The former only zeroes it (which automatically causes a leak in case it was set). So do this when closing and flushing. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
-rw-r--r--libavcodec/vp9_raw_reorder_bsf.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/libavcodec/vp9_raw_reorder_bsf.c b/libavcodec/vp9_raw_reorder_bsf.c
index e7d301cb85..368dcb26c2 100644
--- a/libavcodec/vp9_raw_reorder_bsf.c
+++ b/libavcodec/vp9_raw_reorder_bsf.c
@@ -396,7 +396,7 @@ static void vp9_raw_reorder_flush(AVBSFContext *bsf)
for (int s = 0; s < FRAME_SLOTS; s++)
vp9_raw_reorder_clear_slot(ctx, s);
- ctx->next_frame = NULL;
+ vp9_raw_reorder_frame_free(&ctx->next_frame);
ctx->sequence = 0;
}
@@ -407,6 +407,7 @@ static void vp9_raw_reorder_close(AVBSFContext *bsf)
for (s = 0; s < FRAME_SLOTS; s++)
vp9_raw_reorder_clear_slot(ctx, s);
+ vp9_raw_reorder_frame_free(&ctx->next_frame);
}
static const enum AVCodecID vp9_raw_reorder_codec_ids[] = {