[Ffmpeg-cvslog] r6925 - trunk/libavformat/matroska.c

aurel subversion
Tue Nov 7 00:57:35 CET 2006


Author: aurel
Date: Tue Nov  7 00:57:34 2006
New Revision: 6925

Modified:
   trunk/libavformat/matroska.c

Log:
Don't need to check for NULL before av_free().
And don't need to memset(0) the no more used priv_data.


Modified: trunk/libavformat/matroska.c
==============================================================================
--- trunk/libavformat/matroska.c	(original)
+++ trunk/libavformat/matroska.c	Tue Nov  7 00:57:34 2006
@@ -2609,11 +2609,8 @@
     MatroskaDemuxContext *matroska = s->priv_data;
     int n = 0;
 
-    if (matroska->writing_app)
         av_free(matroska->writing_app);
-    if (matroska->muxing_app)
         av_free(matroska->muxing_app);
-    if (matroska->index)
         av_free(matroska->index);
 
     if (matroska->packets != NULL) {
@@ -2626,22 +2623,15 @@
 
     for (n = 0; n < matroska->num_tracks; n++) {
         MatroskaTrack *track = matroska->tracks[n];
-        if (track->codec_id)
             av_free(track->codec_id);
-        if (track->codec_name)
             av_free(track->codec_name);
-        if (track->codec_priv)
             av_free(track->codec_priv);
-        if (track->name)
             av_free(track->name);
-        if (track->language)
             av_free(track->language);
 
         av_free(track);
     }
 
-    memset(matroska, 0, sizeof(MatroskaDemuxContext));
-
     return 0;
 }
 




More information about the ffmpeg-cvslog mailing list