File mediatomb-0.12.1-mp4v2_2.0.0.patch of Package mediatomb

File mediatomb-0.12.1-mp4v2_2.0.0.patch of Package mediatomb (Revision 9342cf9cff65b50bf2895bc945a60533)

Currently displaying revision 9342cf9cff65b50bf2895bc945a60533 , Show latest

--- a/configure.ac
+++ b/configure.ac
@@ -1541,52 +1541,10 @@ fi
 ######## ffmpeg
 
 if test "x$FFMPEG_OPTION_ENABLED" = xyes; then
-    AC_LANG_SAVE
-    AC_LANG_C
-
-
-    FFMPEG_EXTRA_CFLAGS=""
-
-    # the header location of the older and newer ffmpeg version is different
-    MT_CHECK_HEADER([avformat], [ffmpeg/avformat], [pass])
-    if test "x$AVFORMAT_STATUS" != xyes; then
-        MT_CHECK_HEADER([avformat], [libavformat/avformat], [pass])
-        if test "x$AVFORMAT_STATUS" = xyes; then
-            AC_DEFINE_UNQUOTED([AVFORMAT_INCLUDE], [<libavformat/avformat.h>], [libavformat/avformat.h])
-        else
-            CFLAGS="$CFLAGS -I/usr/include/ffmpeg"
-            CPPFLAGS="$CPPFLAGS -I/usr/include/ffmpeg"
-            MT_CHECK_HEADER([avformat], [libavformat/avformat], [pass])
-            if test "x$AVFORMAT_STATUS" = xyes; then
-              AC_DEFINE_UNQUOTED([AVFORMAT_INCLUDE], [<libavformat/avformat.h>], [libavformat/avformat.h])
-              FFMPEG_EXTRA_CFLAGS="$FFMPEG_CFLAGS -I/usr/include/ffmpeg"
-            else
-                CFLAGS="$CFLAGS_SAVE"
-                CPPFLAGS="$CPPFLAGS_SAVE"
-                CFLAGS="$CFLAGS -I/usr/local/include/ffmpeg"
-                CPPFLAGS="$CPPFLAGS -I/usr/local/include/ffmpeg"
-                MT_CHECK_HEADER([avformat], [libavformat/avformat])
-                if test "x$AVFORMAT_STATUS" = xyes; then
-                    FFMPEG_EXTRA_CFLAGS="$FFMPEG_CFLAGS -I/usr/include/ffmpeg"
-                    AC_DEFINE_UNQUOTED([AVFORMAT_INCLUDE], [<libavformat/avformat.h>], [libavformat/avformat.h])
-                fi
-            fi
-        fi
-    else
-       AC_DEFINE_UNQUOTED([AVFORMAT_INCLUDE], [<ffmpeg/avformat.h>], [ffmpeg/avformat.h])
-    fi
-
-    if test "x$AVFORMAT_STATUS" = xyes; then
-        MT_CHECK_LIBRARY([avformat], [avformat], [av_register_all])
-        MT_CHECK_LIBRARY([avutil], [avutil], [av_log_set_callback])
-    fi
-    AC_LANG_RESTORE
-
-    if ((test "x$AVFORMAT_STATUS" = xyes) && (test "x$AVUTIL_STATUS" = xyes)); then
-        FFMPEG_STATUS=yes
-    else
-        FFMPEG_STATUS=missing
-    fi
+    FFMPEG_STATUS=yes
+    AC_DEFINE_UNQUOTED([AVFORMAT_INCLUDE], [<libavformat/avformat.h>], [libavformat/avformat.h])
+    AVFORMAT_CFLAGS="`pkg-config --cflags libavformat libavutil`"
+    AVFORMAT_LDFLAGS="`pkg-config --libs libavformat libavutil`"
 fi
 
 if test "x$FFMPEG_STATUS" = xyes; then
--- a/src/metadata/libmp4v2_handler.cc
+++ b/src/metadata/libmp4v2_handler.cc
@@ -62,28 +62,26 @@ LibMP4V2Handler::LibMP4V2Handler() : Met
 {
 }
 
-static void addMetaField(metadata_fields_t field, MP4FileHandle mp4, Ref<CdsItem> item)
+static void addMetaField(metadata_fields_t field, const MP4Tags* mp4tags, Ref<CdsItem> item)
 {
     String value;
     char*  mp4_retval = NULL;
-    u_int16_t track;
-    u_int16_t total_tracks;
  
     Ref<StringConverter> sc = StringConverter::i2i();
     
     switch (field)
     {
         case M_TITLE:
-            MP4GetMetadataName(mp4, &mp4_retval);
+	    mp4_retval = (char *)mp4tags->name;
             break;
         case M_ARTIST:
-            MP4GetMetadataArtist(mp4, &mp4_retval);
+            mp4_retval = (char *)mp4tags->artist;
             break;
         case M_ALBUM:
-            MP4GetMetadataAlbum(mp4, &mp4_retval);
+            mp4_retval = (char *)mp4tags->album;
             break;
         case M_DATE:
-            MP4GetMetadataYear(mp4, &mp4_retval);
+            mp4_retval = (char *)mp4tags->releaseDate;
             if (mp4_retval)
             {
                 value = mp4_retval;
@@ -95,17 +93,16 @@ static void addMetaField(metadata_fields
             }
             break;
         case M_GENRE:
-            MP4GetMetadataGenre(mp4, &mp4_retval);
+            mp4_retval = (char *)mp4tags->genre;
             break;
         case M_DESCRIPTION:
-            MP4GetMetadataComment(mp4, &mp4_retval);
+            mp4_retval = (char *)mp4tags->description;
             break;
         case M_TRACKNUMBER:
-            MP4GetMetadataTrack(mp4, &track, &total_tracks);
-            if (track > 0)
+            if (mp4tags->track->index > 0)
             {
-                value = String::from(track);
-                item->setTrackNumber((int)track);
+                value = String::from(mp4tags->track->index);
+                item->setTrackNumber((int)mp4tags->track->index);
             }
             else
                 return;
@@ -128,6 +125,7 @@ static void addMetaField(metadata_fields
         item->setMetadata(MT_KEYS[field].upnp, sc->convert(value));
         log_debug("mp4 handler: setting metadata on item: %d, %s\n", field, sc->convert(value).c_str());
     }
+
 }
 
 void LibMP4V2Handler::fillMetadata(Ref<CdsItem> item)
@@ -145,8 +143,13 @@ void LibMP4V2Handler::fillMetadata(Ref<C
 
     try
     {
+        /* allocate */
+        const MP4Tags* tags = MP4TagsAlloc();
+        /* fetch data from MP4 file and populate structure */
+        MP4TagsFetch( tags, mp4 );
+ 
         for (int i = 0; i < M_MAX; i++)
-            addMetaField((metadata_fields_t) i, mp4, item);
+            addMetaField((metadata_fields_t) i, tags, item);
 
         Ref<ConfigManager> cm = ConfigManager::getInstance();
 
@@ -190,32 +193,35 @@ void LibMP4V2Handler::fillMetadata(Ref<C
         }
 
 #if defined(HAVE_MAGIC)
-        u_int8_t *art_data;
-        u_int32_t art_data_len;
+        //u_int8_t *art_data;
+        //u_int32_t art_data_len;
         String art_mimetype;
+	const MP4TagArtwork *art;
 #ifdef HAVE_MP4_GET_METADATA_COVER_ART_COUNT
-        if (MP4GetMetadataCoverArtCount(mp4) && 
-            MP4GetMetadataCoverArt(mp4, &art_data, &art_data_len))
+        //if (MP4GetMetadataCoverArtCount(mp4) && 
+        //    MP4GetMetadataCoverArt(mp4, &art_data, &art_data_len))
+	if (tags->artworkCount)
 #else
-            MP4GetMetadataCoverArt(mp4, &art_data, &art_data_len);
+            //MP4GetMetadataCoverArt(mp4, &art_data, &art_data_len);
 #endif
         {
-            if (art_data)
+	    art = tags->artwork;
+            if (art->data)
             {
                 try
                 {
-                    art_mimetype = ContentManager::getInstance()->getMimeTypeFromBuffer((void *)art_data, art_data_len);
+                    art_mimetype = ContentManager::getInstance()->getMimeTypeFromBuffer((void *)art->data, art->size);
                     if (!string_ok(art_mimetype))
                         art_mimetype = _(MIMETYPE_DEFAULT);
 
                 }
                 catch (Exception ex)
                 {
-                    free(art_data);
+                    //free(art_data);
                     throw ex;
                 }
 
-                free(art_data);
+                //free(art_data);
                 if (art_mimetype != _(MIMETYPE_DEFAULT))
                 {
                     Ref<CdsResource> resource(new CdsResource(CH_MP4));
@@ -226,6 +232,7 @@ void LibMP4V2Handler::fillMetadata(Ref<C
             }
         }
 #endif
+	MP4TagsFree( tags );
         MP4Close(mp4);
     }
     catch (Exception ex)
@@ -247,24 +254,30 @@ Ref<IOHandler> LibMP4V2Handler::serveCon
 
     String ctype = res->getParameters()->get(_(RESOURCE_CONTENT_TYPE));
 
+    const MP4Tags* tags = MP4TagsAlloc();
+    MP4TagsFetch( tags, mp4 );
+
     if (ctype != ID3_ALBUM_ART)
         throw _Exception(_("LibMP4V2Handler: got unknown content type: ") + ctype);
 #ifdef HAVE_MP4_GET_METADATA_COVER_ART_COUNT
-    if (!MP4GetMetadataCoverArtCount(mp4))
+    if (!tags->artworkCount)
         throw _Exception(_("LibMP4V2Handler: resource has no album art information"));
 #endif
-    u_int8_t *art_data;
-    u_int32_t art_data_len;
-    if (MP4GetMetadataCoverArt(mp4, &art_data, &art_data_len))
+    //u_int8_t *art_data;
+    //u_int32_t art_data_len;
+    const MP4TagArtwork *art;
+    art = tags->artwork;
+    if (art)
     {
-        if (art_data)
+        if (art->data)
         {
-            *data_size = (off_t)art_data_len;
-            Ref<IOHandler> h(new MemIOHandler((void *)art_data, art_data_len));
-            free(art_data);
+            *data_size = (off_t)art->size;
+            Ref<IOHandler> h(new MemIOHandler((void *)art->data, art->size));
+            //free(art_data);
             return h;
         }
     }
+    MP4TagsFree( tags );
         
     throw _Exception(_("LibMP4V2Handler: could not serve album art "
                            "for file") + item->getLocation() +