Rockbox.org home
release
dev builds
extras
themes manual
wiki
device status forums
mailing lists
IRC bugs
patches
dev guide
translations



Rockbox mail archive

Subject: Re: id3.c: reading id3 tags, is this a bug?

Re: id3.c: reading id3 tags, is this a bug?

From: Linus Nielsen Feltzing <linus_at_haxx.se>
Date: Mon, 17 Mar 2003 12:01:23 +0100

TP Diffenbach wrote:
> However, size refers to the size of the frame being read from, in the
> mp3's id3 tag. It seems to me that even if unicode_munge is adjusting
> the size of the copy of the string that we are saving, the size
> remaining on disk in the mp3's id3 tag remains the same. If this is
> the case, size should be set /before/ bytesread is (possibly)
> adjusted by unicode_munge:

You are absolutely correct. Thanks for pointing that out.

> Also, it seems that the call to read should be checked for error or
> EOF:

I agree.

> Thanks.-- Archos FM needs a Rockbox!

Thank you for finding bugs in our code.

/Linus
Received on 2003-03-17

Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy