On 03/21/2012 01:52 PM, Justin Ruggles wrote:

> On 03/21/2012 08:17 AM, Tomas Härdin wrote:
> 
>> Hi
>>
>> Running ffprobe/avprobe on
>> http://titan.codemill.se/~tomhar/samples/sigfpe.aif results in SIGFPE at
>> libavformat/aiffdec.c:162 since aiff->block_duration == 0.
>> Attached patch sets block_duration = 1 in the default case. Fixes the
>> problem for both projects.
> 
> 
> LGTM. But we should also set bits_per_coded_sample for the default case
> so that block_align will be set correctly.


Nevermind, it's already set above. We just override it for integer pcm.
I want to rework the codec handling a bit. I'll send a separate patch
for that.

Thanks,
Justin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to