]> git.ipfire.org Git - thirdparty/freeswitch.git/commitdiff
Another couple of Coverity related cleanups
authorSteve Underwood <steveu@coppice.org>
Wed, 30 Apr 2014 14:51:59 +0000 (22:51 +0800)
committerSteve Underwood <steveu@coppice.org>
Wed, 30 Apr 2014 14:51:59 +0000 (22:51 +0800)
libs/spandsp/src/schedule.c
libs/spandsp/src/t4_rx.c

index 6e38e6bd9485399c9bce34deedd1f377361699cc..93916dea1d63ff1c4052ad5e163c986823e885b8 100644 (file)
@@ -156,9 +156,11 @@ SPAN_DECLARE(int) span_schedule_release(span_sched_state_t *s)
 
 SPAN_DECLARE(int) span_schedule_free(span_sched_state_t *s)
 {
-    span_schedule_release(s);
     if (s)
+    {
+        span_schedule_release(s);
         span_free(s);
+    }
     return 0;
 }
 /*- End of function --------------------------------------------------------*/
index f14521b5f0bfe8cc498291e153a9688c80b9a216..7a39dfd1448eb208d8aa1290b7450d70968b2f87 100644 (file)
@@ -733,7 +733,10 @@ static int close_tiff_output_file(t4_rx_state_t *s)
         /* Try not to leave a file behind, if we didn't receive any pages to
            put in it. */
         if (s->current_page == 0)
-            remove(s->tiff.file);
+        {
+            if (remove(s->tiff.file) < 0)
+                span_log(&s->logging, SPAN_LOG_WARNING, "%s: Failed to remove file.\n", s->tiff.file);
+        }
         span_free((char *) s->tiff.file);
     }
     s->tiff.file = NULL;