Hi,

On Mon, Nov 13, 2017 at 1:09 AM, Carl Eugen Hoyos <ceffm...@gmail.com> wrote:
> 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.

This is the complete patchset.

-Umair

Attachment: 0001-libavcodec-als-remove-check-for-predictor-order-of-a.patch
Description: Binary data

Attachment: 0002-libavcodec-als-fix-address-sanitization-error-in-dec.patch
Description: Binary data

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to