[FFmpeg-devel] [PATCH] Fix bug in voc demuxer that was introduced in SVN r26309.
Justin Ruggles
justin.ruggles
Tue Jan 18 14:08:13 CET 2011
On 01/18/2011 07:50 AM, Carl Eugen Hoyos wrote:
> Ronald S. Bultje <rsbultje <at> gmail.com> writes:
>
>> @@ -117,7 +117,7 @@ voc_get_packet(AVFormatContext *s, AVPacket *pkt,
>> AVStream *st, int max_size)
>> dec->sample_rate = get_le32(pb);
>> dec->bits_per_coded_sample = get_byte(pb);
>> dec->channels = get_byte(pb);
>> - tmp_codec = ff_codec_get_id(ff_voc_codec_tags, get_byte(pb));
>> + tmp_codec = ff_codec_get_id(ff_voc_codec_tags, get_le16(pb));
>>
>> Does this have (a) fate test(s)?
>
> No, since there is a native muxer, this should be a regression test.
That might be a good idea, especially considering our demuxer apparently
can't read a stereo s16 file created by our muxer. I'll try to fix it,
then send a patch for adding a regression test for voc stereo s16.
-Justin
More information about the ffmpeg-devel
mailing list