Coverity: Reverse INULL: CID 130
authorNelson Ferreira <nelson.ferreira@ieee.org>
Fri, 20 Jan 2012 03:22:37 +0000 (22:22 -0500)
committerNelson Ferreira <nelson.ferreira@ieee.org>
Fri, 20 Jan 2012 03:22:37 +0000 (22:22 -0500)
* src/media/sound.c (audio_device_finalise): Do nothing if header is NULL

Signed-off-by: Nelson Ferreira <nelson.ferreira@ieee.org>
src/media/sound.c

index 6b941f5..704fd59 100644 (file)
@@ -857,6 +857,9 @@ audio_device_finalise(void *header, int for_disksave)
        SOUND_DEBUG_DEV("GCor asked me to finalise: 0x%lx\n",
                        (long unsigned int)ad);
 
+       if ( ad == NULL ) 
+               return;
+
        if (audio_device_data(ad) &&
            audio_device_meth(ad, finish))
                audio_device_meth(ad, finish)(audio_device_data(ad));
@@ -866,7 +869,7 @@ audio_device_finalise(void *header, int for_disksave)
        audio_device_data(ad) = NULL;
 
        /* avoid some warning */
-       if (for_disksave || ad == NULL);
+       if (for_disksave);
 }
 
 static void