free(NULL) is safe.

diff --git a/src/misc/filter_chain.c b/src/misc/filter_chain.c
index d7de8805f08be8c107f98281021ae78398c2e87f..e470bef96313bd8f0b8eb08c428c674cd2d6b1b1 100644 (file)
--- a/src/misc/filter_chain.c
+++ b/src/misc/filter_chain.c
@@ -466,8 +466,7 @@ static int filter_chain_DeleteFilterInternal( filter_chain_t *p_chain,
     vlc_object_release( p_filter );
 
     vlc_mouse_t *p_mouse = vlc_array_item_at_index( &p_chain->mouses, i_filter_idx );
-    if( p_mouse )
-        free( p_mouse );
+    free( p_mouse );
     vlc_array_remove( &p_chain->mouses, i_filter_idx );
 
     /* FIXME: check fmt_in/fmt_out consitency */

diff --git a/src/text/filesystem.c b/src/text/filesystem.c
index bc602ae5f366c3511e74af4cf750a67a4d37efca..cb3dc3d261a2011eddb90f08d16330a2898d06f8 100644 (file)
--- a/src/text/filesystem.c
+++ b/src/text/filesystem.c
@@ -339,8 +339,7 @@ int utf8_loaddir( DIR *dir, char ***namelist,
 
         for( i = 0; i < num; i++ )
             free( tab[i] );
-        if( tab != NULL )
-            free( tab );
+        free( tab );
         }
     }
     return -1;