#189 vbr stack overflow

[Jérôme Perciot]

Hi,

don't know if you knwo speaking french, so i put this in english

I used functions fore reading id3v1 tags and VBR header information, but error Stack was corrupted occured in SOME Mp3 files only.

When i debugged id3li, i found :
const size_t VBR_HEADER_MAX_SIZE = 116; // frames, bytes, toc and scale are optional
in the file mp3_parse.cpp, but when i was in the line vbrheaderdata[vbr_header_size] = '\0'; the value of vbr_header_size was 120.

When i changed VBR_HEADER_MAX_SIZE to 120 instead of 116, all went good.

If i can help you for next versions...

Best regards.

[Carl Lewis]

I've hit this bug as well and can confirm that the fix works.

[Norbert]

This issue is fixed in the cvs version of 3.8.3

[Norbert]

This issue is fixed in the present CVS source tree.