Re: [FFmpeg-devel] [PATCH v3 2/3] Fix leak and crash in tee muxer when open_slave fails

2016-04-06 Thread Nicolas George
Le quintidi 15 germinal, an CCXXIV, Marton Balint a écrit : > This looks good to me as well. Nicolas? I am ok with the patch. Regards, -- Nicolas George signature.asc Description: Digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg

Re: [FFmpeg-devel] [PATCH v3 2/3] Fix leak and crash in tee muxer when open_slave fails

2016-04-03 Thread Marton Balint
On Tue, 29 Mar 2016, sebechlebsky...@gmail.com wrote: The commit message is a bit measleasing, because as far as I see a crash could not happen with the old code, only a leak. Obviously the patch and the commit description is correct and bsfs needs to be checked in the new setup. Also try t

[FFmpeg-devel] [PATCH v3 2/3] Fix leak and crash in tee muxer when open_slave fails

2016-03-29 Thread sebechlebskyjan
From: Jan Sebechlebsky Calling close_slave in case error is to be returned from open_slave will free allocated resources. Since failure can happen before bsfs array is initialized, close_slave must check that bsfs is not NULL before accessing tee_slave->bsfs[i] element. Signed-off-by: Jan Sebec