Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Bugs
  • Category Codecs
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Release 3.8.1
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by Buschel - 2011-04-26
Last edited by Buschel - 2011-04-26

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

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.pat... (1.9 KiB)
Closed by  Buschel
2011-04-26 21:31
Reason for closing:  Accepted
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

Validated by Magnus Holmgren, submitted with r29784.

kugel. commented on 2011-04-26 12:07

You don't seem to be a fan of #ifndef ROCKBOX?

Buschel commented on 2011-04-26 17:42

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...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing