2017-11-12 20:30 GMT+01:00 Umair Khan <omerj...@gmail.com>: > Hi, > > On Mon, Nov 13, 2017 at 12:45 AM, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: >> 2017-11-12 20:05 GMT+01:00 Umair Khan <omerj...@gmail.com>: >> >>> The attached patch fixes the address sanitizer issue. >> >> Breaks compilation here, how did you test? >> >> libavcodec/alsdec.c: In function ‘decode_var_block_data’: >> libavcodec/alsdec.c:938:7: error: expected ‘}’ before ‘else’ > > Sorry for the faulty patch. Here is the fixed one.
The commit message of your patch is: libavcodec/als: fix address sanitization error in decoder Is there an error in current FFmpeg git head that asan shows? If not, the commit message makes no sense. I believe you should send two patches that are meant to be committed together, one of them fixing ticket #6297. Carl Eugen _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel