- Status Closed
- Percent Complete
- Task Type Patches
- Category Applications
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#9354 - iso_decode function illegal address access
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.
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
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?
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”.