diff options
author | Jack Whitham <jack.d.whitham@gmail.com> | 2013-07-11 21:09:58 +0100 |
---|---|---|
committer | Thomas Martitz <kugel@rockbox.org> | 2013-12-23 17:55:15 +0100 |
commit | ca423ed0e317645c8febb8dc604743f0fd47139a (patch) | |
tree | 4f8ab0164eb64558d0b8db8286ffb92655b4e993 | |
parent | 10d71bb910d96aae5d0ac2e868c961d6b104c1bd (diff) | |
download | rockbox-ca423ed.tar.gz rockbox-ca423ed.zip |
Proposed fix for FS#12878: Zero-length embedded album art prevents mp3 playback
see http://www.rockbox.org/tracker/task/12878
Change-Id: Ib4233c06e18d1d193dfb9e73e745ca5d174e40b2
Reviewed-on: http://gerrit.rockbox.org/507
Reviewed-by: Michael Giacomelli <giac2000@hotmail.com>
Reviewed-by: Thomas Martitz <kugel@rockbox.org>
-rw-r--r-- | lib/rbcodec/metadata/id3tags.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/rbcodec/metadata/id3tags.c b/lib/rbcodec/metadata/id3tags.c index fed99d81a0..3492197c40 100644 --- a/lib/rbcodec/metadata/id3tags.c +++ b/lib/rbcodec/metadata/id3tags.c @@ -355,7 +355,8 @@ static int parsealbumart( struct mp3entry* entry, char* tag, int bufferpos ) /* fixup offset&size for image data */ entry->albumart.pos += tag - start; entry->albumart.size -= tag - start; - entry->has_embedded_albumart = true; + /* check for malformed tag with no picture data */ + entry->has_embedded_albumart = (entry->albumart.size != 0); } /* return bufferpos as we didn't store anything in id3v2buf */ return bufferpos; |