You've already forked FFmpeg
mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2025-08-04 22:03:09 +02:00
fftools/cmdutils: Add av_freep() to avoid memory leak
Add av_freep() to free new_elem to avoid memory leak.
Fixes: 3ca1e31e63
("fftools/cmdutils: Atomically add elements to list of pointers, fix crash")
Signed-off-by: Jiasheng Jiang <jiashengjiangcool@gmail.com>
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
This commit is contained in:
committed by
Michael Niedermayer
parent
85ac589211
commit
f7a6965308
@ -1471,9 +1471,12 @@ void *allocate_array_elem(void *ptr, size_t elem_size, int *nb_elems)
|
||||
{
|
||||
void *new_elem;
|
||||
|
||||
if (!(new_elem = av_mallocz(elem_size)) ||
|
||||
av_dynarray_add_nofree(ptr, nb_elems, new_elem) < 0)
|
||||
new_elem = av_mallocz(elem_size);
|
||||
if (!new_elem)
|
||||
return NULL;
|
||||
if (av_dynarray_add_nofree(ptr, nb_elems, new_elem) < 0)
|
||||
av_freep(&new_elem);
|
||||
|
||||
return new_elem;
|
||||
}
|
||||
|
||||
|
Reference in New Issue
Block a user