[FFmpeg-devel] [PATCH 1/2] avcodec/mjpegdec: Fix av_frame_unref(NULL)

Michael Niedermayer michael at niedermayer.cc
Thu Jun 13 09:47:44 EEST 2024


On Thu, Jun 13, 2024 at 04:02:52AM +0200, Andreas Rheinhardt wrote:
> The smv_frame is only allocated for the SMV decoder, yet
> it is unreferenced in the other decoders' flush functions, too.
> av_frame_unref(NULL) is not documented to be allowed, it just
> happens to work. Avoid it by using a dedicated flush function
> for SMV.

why not docuement av_frame_unref(NULL) ?
seems simpler if thats the only reason

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have often repented speaking, but never of holding my tongue.
-- Xenocrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20240613/2a97d133/attachment.sig>


More information about the ffmpeg-devel mailing list