lance.lmw...@gmail.com:
> From: Limin Wang <lance.lmw...@gmail.com>
>
> Signed-off-by: Limin Wang <lance.lmw...@gmail.com>
> ---
> libavformat/mov.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavformat/mov.c b/libavformat/mov.c
> index 3d6fef685d..03c8801402 100644
> --- a/libavformat/mov.c
> +++ b/libavformat/mov.c
> @@ -6827,7 +6827,7 @@ static int mov_read_dvcc_dvvc(MOVContext *c,
> AVIOContext *pb, MOVAtom atom)
> ret = av_stream_add_side_data(st, AV_PKT_DATA_DOVI_CONF,
> (uint8_t *)dovi, dovi_size);
> if (ret < 0) {
> - av_freep(dovi);
> + av_freep(&dovi);
> return ret;
> }
>
>
Nice catch. LGTM; but you could also just use av_free() given that the
lifetime of the dovi pointer ends automatically when you exit the function.
- Andreas
_______________________________________________
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".
