]> git.sesse.net Git - ffmpeg/commit
avcodec/smacker: Directly goto error in case of error
authorAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Sat, 25 Jul 2020 11:05:06 +0000 (13:05 +0200)
committerAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Thu, 17 Sep 2020 23:35:43 +0000 (01:35 +0200)
commit4656c1771b4ac12d9df9812390551f32fda181bc
tree9a00cdac993c38cccff62467f7b688e79a088e70
parentbd076cacc313b19e5af895df6d332aa48c33cf2a
avcodec/smacker: Directly goto error in case of error

The earlier version did not error out directly in case an error happens,
because it would lead to a leak: An allocated array is only reachable
via a local variable at that time; it is only attached to more permanent
storage at the end. While it would be possible to add custom code for
freeing on error (instead of reusing the ordinary code for doing so),
this commit takes the opposite approach and attaches the newly allocated
array to its permanent place immediately after its allocation.

Reviewed-by: Paul B Mahol <onemda@gmail.com>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
libavcodec/smacker.c