From: Rafaël Carré <funman@videolan.org>
Date: Wed, 3 Oct 2007 21:07:01 +0000 (+0000)
Subject: taglib: warn the user when the embedded image is invalid (i.e. was embedded by a... 
X-Git-Tag: 0.9.0-test0~5228
X-Git-Url: https://git.sesse.net/?a=commitdiff_plain;h=f06a603621a4ff672e760a698a75f15d28da6c75;p=vlc

taglib: warn the user when the embedded image is invalid (i.e. was embedded by a broken version of iTunes)
---

diff --git a/modules/meta_engine/taglib.cpp b/modules/meta_engine/taglib.cpp
index 894f171dc3..eb421349c9 100644
--- a/modules/meta_engine/taglib.cpp
+++ b/modules/meta_engine/taglib.cpp
@@ -119,6 +119,18 @@ static void DetectImage( FileRef f, demux_t *p_demux )
             ByteVector p_data_taglib; const char *p_data; int i_data;
 
             psz_mime = p_apic->mimeType().toCString(true);
+
+            /* some old iTunes version not only sets incorrectly the mime type
+             * but also embeds incorrectly the image.
+             * Recent versions seem to behave correctly */
+            if( !strncmp( psz_mime, "PNG", 3 ) )
+            {
+                msg_Warn( p_demux,
+                    "%s: Invalid picture embedded by broken iTunes version",
+                    f.file()->name() );
+                break;
+            }
+
             psz_description = p_apic->description().toCString(true);
             psz_name = psz_description;