ID3Lib 3.8.3 Stack Corruption Fix

[WiZaRd]

I just ran into that error:

... so I checked the source code and found the obvious problem, so here is the patch for you (for ID3Lib mp3_parse.cpp, Line 468):

//>>> WiZaRd
//WiZaRd: see below! vbr_header_size might get up to 8+4+4+100+4 = 120 bytes! that means that we overrun our buffer!
const size_t VBR_HEADER_MAX_SIZE = 120; // frames, bytes, toc and scale are optional
// const size_t VBR_HEADER_MAX_SIZE = 116; // frames, bytes, toc and scale are optional
//<<< WiZaRd

Best regards,
WiZ

[Tuxman]

Thank you.
Now when might this happen?

[WiZaRd]

Dunno where exactly it was called, but ID3 functions are only used within 2 files in eMule so it should be possible to trace the origin (if you really care).

[Tuxman]

Hab das gerade geprüft; bei mir ist der Fehler offenbar schon behoben.
Seltsam.

Weiß natürlich auch nicht mehr so genau, wo ich die diversen Libs für beba zusammengesucht habe. 'n Teil aus'm X-Ray, 'n Teil aus'm Arabella...



Müsste beizeiten mal sämtliche Libs auf Konsistenz prüfen. Das wird ein Spaß.
Und dann noch mit meiner hoffnungslos veralteten Entwicklungsumgebung zum Laufen bekommen...

[JvA]

also ich hatte den fehler auch schon soooo oft....hab mich aber nicht weiter drum gekümmert.....da wars auch an der stelle....ich hatte es wenn ich nen rar archiv mit mp3's am laden war und im detail-dlg mir den inhalt hab anzeigen lassen.....

danke wizard...
cya
JvA

[WiZaRd]

NP... ich wollt das auch im Bugtracker eintragen, aber auf SF ist das schon (MEHRMALS!) drin - evtl. bräuchten die mal neue Devs!