FS#11956 - False usage of parse_replaygain() in ID3 metadat parsing

Attached to Project: Rockbox
Opened by Andree Buschmann (Buschel) - Monday, 21 February 2011, 21:53 GMT
Last edited by Andree Buschmann (Buschel) - Tuesday, 22 February 2011, 07:11 GMT
Task Type Bugs
Category ID3 / meta data
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Release 3.7.1
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No


The ID3 metadata parser does not correctly handle parse_replaygain(). The last parameter of parse_replaygain() is the buffer size which is left to save the string representation of replaygain. The current svn implementation calls parse_replaygain() with the length of the string to be evaluated by parse_replaygain().
Possible error case: It may happen that the left buffer is smaller than needed for the string representation.

I am providing this fix as a patch to have some reviewers and testers before submitting it.

Edit: Fixed some types in the comments.
This task depends upon

Closed by  Andree Buschmann (Buschel)
Tuesday, 22 February 2011, 07:11 GMT
Reason for closing:  Fixed
Additional comments about closing:  Submitted with r29375.