Workaround f*cked up iTunes

diff --git a/modules/meta_engine/taglib.cpp b/modules/meta_engine/taglib.cpp
index c1ba0f2acd0fcee1e2ac711fe0fe1e6b0a9fb7ea..810fd3f54082c711fef0d4f9fde606a81dff1186 100644 (file)
--- a/modules/meta_engine/taglib.cpp
+++ b/modules/meta_engine/taglib.cpp
@@ -124,21 +124,22 @@ 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);
+            psz_description = psz_name = p_apic->description().toCString(true);
 
             /* some old iTunes version not only sets incorrectly the mime type
+             * or the description of the image,
              * but also embeds incorrectly the image.
              * Recent versions seem to behave correctly */
-            if( !strncmp( psz_mime, "PNG", 3 ) )
+            if( !strncmp( psz_mime, "PNG", 3 ) ||
+                !strncmp( psz_name, "\xC2\x89PNG", 5 ) )
             {
                 msg_Warn( p_demux,
-                    "%s: Invalid picture embedded by broken iTunes version",
+                    "%s: Invalid picture embedded by broken iTunes version, "
+                    "you really shouldn't use this crappy software.",
                     f.file()->name() );
                 break;
             }
 
-            psz_description = p_apic->description().toCString(true);
-            psz_name = psz_description;
-
             p_data_taglib = p_apic->picture();
             p_data = p_data_taglib.data();
             i_data = p_data_taglib.size();