[FFmpeg-devel] [PATCH v10 2/3][GSOC] avformat/tee: Fix leaks in tee muxer when open_slave fails
Nicolas George
george at nsup.org
Fri Apr 22 17:15:17 CEST 2016
Le duodi 2 floréal, an CCXXIV, sebechlebskyjan at gmail.com a écrit :
> From: Jan Sebechlebsky <sebechlebskyjan at gmail.com>
>
> In open_slave failure can happen before bsfs array is initialized,
> close_slave must check that bsfs is not NULL before accessing
> tee_slave->bsfs[i] element.
>
> Slave muxer expects write_trailer to be called if it's
> write_header suceeded (so resources allocated in write_header
> are freed). Therefore if failure happens after successfull
> write_header call, we must ensure that write_trailer of
> that particular slave is called.
>
> Some cleanups are made by Marton Balint.
>
> Signed-off-by: Jan Sebechlebsky <sebechlebskyjan at gmail.com>
> Signed-off-by: Marton Balint <cus at passwd.hu>
Looks ok.
Regards,
--
Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160422/ab5de729/attachment.sig>
More information about the ffmpeg-devel
mailing list