[FFmpeg-devel] [PATCH 05/10] avformat/hls: fix leak of variant when dynarray_add fail
Zhao Zhili
quinkblack at foxmail.com
Tue Apr 12 11:15:17 EEST 2022
---
libavformat/hls.c | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/libavformat/hls.c b/libavformat/hls.c
index d5c3009d07..b5cdf158c6 100644
--- a/libavformat/hls.c
+++ b/libavformat/hls.c
@@ -366,8 +366,16 @@ static struct variant *new_variant(HLSContext *c, struct variant_info *info,
strcpy(var->subtitles_group, info->subtitles);
}
- dynarray_add(&c->variants, &c->n_variants, var);
- dynarray_add(&var->playlists, &var->n_playlists, pls);
+ if (av_dynarray_add_nofree(&var->playlists, &var->n_playlists, pls) < 0) {
+ /* Don't free pls since it's not owned by variant */
+ av_free(var);
+ return NULL;
+ }
+ if (av_dynarray_add_nofree(&c->variants, &c->n_variants, var) < 0) {
+ av_free(var->playlists);
+ av_free(var);
+ return NULL;
+ }
return var;
}
--
2.31.1
More information about the ffmpeg-devel
mailing list