Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#9354 - iso_decode function illegal address access

Attached to Project: Rockbox
Opened by Yoshihisa Uchida (Uchida) - Tuesday, 02 September 2008, 17:29 GMT+2
Task Type Patches
Category Applications
Status Unconfirmed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Private No

Details

For
iso_decode(const unsigned char *iso, unsigned char *utf8, int cp, int count) (in firmware/common/unicode.c),

if count > strlen(iso), then "iso" accesses an illegal address. Therefore, a fatal error might occur.
   iso_decode.patch (0.5 KiB) 
 firmware/common/unicode.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

This task depends upon

Comment by Linus Nielsen Feltzing (linusnielsen) - Wednesday, 03 September 2008, 08:08 GMT+2
That is correct, thanks for pointing it out.

However, we have plenty of code in Rockbox without bounds checking. The philosophy is that we generally don't want to waste code space with unnecessarily defensive programming.

Have you discovered a crash in Rockbox caused by iso_decode()?

What is your real name, so I can give you proper credit?
Comment by Yoshihisa Uchida (Uchida) - Thursday, 04 September 2008, 16:15 GMT+2
Fortunately, a fatal error has not been seen to occur.

I check the source that uses iso_decode(), because of it is sizeof(iso) >= count,
an illegal address is not accessed.

I'm sorry firing.

Finally, my real name is "Yoshihisa Uchida".

Loading...