Rockbox

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

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#12081 - compile warnings (out-of-bounds) in libfaad

Attached to Project: Rockbox
Opened by Andree Buschmann (Buschel) - Tuesday, 26 April 2011, 08:07 GMT+2
Last edited by Andree Buschmann (Buschel) - Tuesday, 26 April 2011, 23:31 GMT+2
Task Type Bugs
Category Codecs
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Release 3.8.1
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

Magnus Holmgren reported compiler warnings with GCC 4.6.0 in libfaad/decoder.c since r29778. The multiple warning starts at line 605: "warning: array subscript is above array bounds [-Warray-bounds]"

Reason for this is that I reduced MAX_CHANNELS to 2 (= stereo). The reported section in function create_channel_config() accesses channel_position[] at too large indices.

I reviewed the code and did some changes:
1) comment create_channel_config() as this is only used to set variables which are not used later
2) let NeAACDecInit() and NeAACDecInit2() return an error if more than MAX_CHANNELS channels are recognized
   libfaad_channel_check_v01.patch (1.9 KiB) 
 apps/codecs/libfaad/decoder.c |   21 ++++++++++++++++++++-
 1 file changed, 20 insertions(+), 1 deletion(-)

This task depends upon

Closed by  Andree Buschmann (Buschel)
Tuesday, 26 April 2011, 23:31 GMT+2
Reason for closing:  Accepted
Additional comments about closing:  Validated by Magnus Holmgren, submitted with r29784.
Comment by Thomas Martitz (kugel.) - Tuesday, 26 April 2011, 14:07 GMT+2
You don't seem to be a fan of #ifndef ROCKBOX?
Comment by Andree Buschmann (Buschel) - Tuesday, 26 April 2011, 19:42 GMT+2
No. And as our libfaad-implementation has already been altered quite a lot I see no reason in keeping it syncable until we do not switch to another implementation (e.g. ffmpeg or google).

Loading...