[FFmpeg-devel] [PATCH] alsdec: channel sorting

Paul B Mahol onemda at gmail.com
Fri Dec 21 15:05:47 CET 2012


On 12/21/12, Thilo Borgmann <thilo.borgmann at googlemail.com> wrote:
> Am 21.12.12 14:21, schrieb Paul B Mahol:
>> Signed-off-by: Paul B Mahol <onemda at gmail.com>
>> ---
>>  libavcodec/alsdec.c | 13 ++++++++++---
>>  1 file changed, 10 insertions(+), 3 deletions(-)
>
> Probably ok. Have you tested and veryfied using the reference
> encoder/decoder?
>

Yes, without this patch ffmpeg reports CRC error.

Though instead of aborting i will just ignore channel sorting if invalid values
are encountered.

Note that reference encoder segv when you give him invalid channel sort numbers,
also reference decoder gives broken output for some 6ch files i found on their
ftp server.


More information about the ffmpeg-devel mailing list