This is the bug/patch tracker for Rockbox. Click here for more information.
Quick links: Bugs · Patches · Rockbox frontpage
FS#2917 - FIX: retrieval of total sample count in get_vorbis_metadata
Attached to Project:
Rockbox
Opened by Gerhard Dirschl (gdirschl) - Saturday, 14 January 2006, 16:12 GMT+2
Last edited by Björn Stenberg (zagor) - Friday, 24 March 2006, 15:23 GMT+2
Opened by Gerhard Dirschl (gdirschl) - Saturday, 14 January 2006, 16:12 GMT+2
Last edited by Björn Stenberg (zagor) - Friday, 24 March 2006, 15:23 GMT+2
|
DetailsRepost as the attachment was missed/lost.
get_vorbis_metadata may return a too small sample count (depending on the ogg file beeing analysed). The reasons are: - the last five bytes of a data chunk in buffer are always discarded if no "OggS" mark was found in buffer so far. Hence the beginning of a header in these five bytes is lost sight of. - Moving an OggS mark to start of buffer (after it was found in the last 17 bytes of a data chunk) doesn't work like expected (out of boundary read, buffer garbling). - If a header was found in buffer it is not checked if there are other ones. OK, as the buffer is very small this is unlikely to happen. A patch is hopefully attached this time. 
metadata1.diff
(2 KiB)
rockbox-devel/apps/metadata.c | 25 ++++++++++++++++--------- 1 file changed, 16 insertions(+), 9 deletions(-) |
This task depends upon
Closed by Magnus Holmgren (learman)
Monday, 01 May 2006, 15:58 GMT+2
Reason for closing: Accepted
Additional comments about closing: Patch committed, thanks!
Monday, 01 May 2006, 15:58 GMT+2
Reason for closing: Accepted
Additional comments about closing: Patch committed, thanks!
Comment by Brandon Low (lostlogic) -
Friday, 24 March 2006, 16:47 GMT+2
Would this by chance lead to crashing on certain not quite right oggs?