Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Codecs
  • 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
Attached to Project: Rockbox
Opened by salty-horse - 2009-11-16
Last edited by learman - 2009-11-22

FS#10790 - Support for MPEG-4 'isom' brand

Using r23643

While playing the attached song.m4a (truncated), the simulator reported the following error:

Unknown MP4 file type: ‘isom’

I have tried adding the ‘isom’ magic number to the various checks that cause that error (see patch), and now the file plays fine in the simulator.

However, it fails to play on my Sansa e280. The player goes through all of the files in the directory, while loading their metadata, and then returns to the directory file list without crashing.

Any help with adding support for ‘isom’ will be greatly appreciated. The relevant ISO standard is here:
http://standards.iso.org/ittf/PubliclyAvailableStandards/c051533_ISO_IEC_14496-12_2008.zip

   isom.patch (1.5 KiB)
   song.m4a (326.3 KiB)
Closed by  learman
2009-11-22 17:19
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

Committed, with an additional fix. Seem to work as well as other MP4/AAC files now (e.g., metadata need to be at the beginning of the file).

salty-horse commented on 2009-11-18 22:45

Correction: No metadata is being loaded on the Sansa.

learman commented on 2009-11-22 13:10

The attached file fails because it is truncated. A final seek in read_mp4_container fails on target because of it (not sure if that is proper behavior; should seeking past end-of-file really return an error?). Adding a “if (id3→filesize == 0)” guard to the lseek at the end of the loop in read_mp4_container makes the file work for me. Not a bad idea to add that guard regardless of lseek behavior; the final seek isn’t needed.

Don’t know if this seek is the problem with non-truncated files.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing