mp3 header check bug
lalinsky at gmail.com
Mon Apr 15 07:48:14 UTC 2013
I've looked it up and you are right, the second byte can be 0xFF, so
I've removed the check:
On Mon, Apr 15, 2013 at 9:29 AM, Lukáš Lalinský <lalinsky at gmail.com> wrote:
> I'm not very familiar with the MPEG stream format myself, so it's hard
> to review this change for me.
> Could you please add some reference for this change?
> On Tue, Mar 5, 2013 at 9:02 AM, Jingmin Wei <weijingmin.cn at gmail.com> wrote:
>> Hi All,
>> I found some mp3 length returned with bad value, so I have checked the
>> source code. I found something could be wrong with function
>> bool MPEG::File::secondSynchByte(char byte) .
>> In this case "the mp3 is MPEG1 Layer1 no CRC" the second byte could be FF.
>> So I modify the file as the diff.
>> taglib-devel mailing list
>> taglib-devel at kde.org
More information about the taglib-devel