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

[Paul B Mahol]

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.