[FFmpeg-devel] [PATCH 2/2] avformat/movenchint: use av_freep() for saftey

Paul B Mahol onemda at gmail.com
Sun Aug 17 12:24:27 CEST 2014


On 8/16/14, Michael Niedermayer <michaelni at gmx.at> wrote:
> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> ---
>  libavformat/movenchint.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libavformat/movenchint.c b/libavformat/movenchint.c
> index 2602254..006aa09 100644
> --- a/libavformat/movenchint.c
> +++ b/libavformat/movenchint.c
> @@ -72,7 +72,7 @@ static void sample_queue_pop(HintSampleQueue *queue)
>      if (queue->len <= 0)
>          return;
>      if (queue->samples[0].own_data)
> -        av_free(queue->samples[0].data);
> +        av_freep(&queue->samples[0].data);
>      queue->len--;
>      memmove(queue->samples, queue->samples + 1,
> sizeof(HintSample)*queue->len);
>  }
> @@ -85,7 +85,7 @@ static void sample_queue_free(HintSampleQueue *queue)
>      int i;
>      for (i = 0; i < queue->len; i++)
>          if (queue->samples[i].own_data)
> -            av_free(queue->samples[i].data);
> +            av_freep(&queue->samples[i].data);
>      av_freep(&queue->samples);
>      queue->len  = 0;
>      queue->size = 0;
> --
> 1.7.9.5
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

typo saftey -> safety

lgtm


More information about the ffmpeg-devel mailing list