[FFmpeg-devel] [PATCH v3] examples: Add check and replace av_free() to avoid potential memory errors

[Nicolas George]

Jiasheng Jiang (HE12025-08-06):
> Add check for the return value of av_packet_alloc() to avoid potential NULL pointer dereference.
> Moreover, replace redundant av_free() with fprintf().
> 
> Fixes: 9a38184a14 ("examples/decode_audio: allocate the packet dynamically")
> Signed-off-by: Jiasheng Jiang <jiashengjiangcool at gmail.com>
> ---
> Changelog:
> 
> v2 -> v3:
> 
> 1. Remove wrong Fixes tag.
> 
> v1 -> v2:
> 
> 1. Remove unnecessary goto.
> 2. Add fprintf() for error message.
> ---
>  doc/examples/decode_audio.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)

Still looks good to me, of course, thanks.

I do not maintain the docs.

Regards,

-- 
  Nicolas George