[FFmpeg-devel] [PATCH] lavc/utils: simplify lockmgr

Michael Niedermayer michael at niedermayer.cc
Tue Nov 28 04:03:22 EET 2017


On Mon, Nov 27, 2017 at 04:44:19AM +0000, Rostislav Pehlivanov wrote:
> Again, totally unneded use of the atomic function to set/NULL a local variable.
> 
> Signed-off-by: Rostislav Pehlivanov <atomnuker at gmail.com>
> ---
>  libavcodec/utils.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)

this crashs ffplay

ffplay_g && gdb --args ./ffplay_g tests/data/fate/vsynth1-snow.avi

#0  0x000000000106f8a3 in av_freep ()
#1  0x0000000000b103c4 in default_lockmgr_cb ()
#2  0x0000000000b13dd3 in av_lockmgr_register ()
#3  0x00000000004ab00d in main ()

if you cant reproduce, say so, and ill rebuild and produce a better
backtrace

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

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20171128/19e894e3/attachment.sig>


More information about the ffmpeg-devel mailing list