Fix some inconsequential file descriptor leaks in the demo tools to silence cppcheck...
authorGregory Maxwell <greg@xiph.org>
Sun, 30 Oct 2011 03:54:45 +0000 (23:54 -0400)
committerJean-Marc Valin <jmvalin@jmvalin.ca>
Sun, 30 Oct 2011 04:02:28 +0000 (00:02 -0400)
celt/opus_custom_demo.c
src/opus_demo.c
src/repacketizer_demo.c

index 741d2bd..e25341b 100644 (file)
@@ -97,6 +97,7 @@ int main(int argc, char *argv[])
    if (!fout)
    {
       fprintf (stderr, "Could not open output file %s\n", argv[argc-1]);
+      fclose(fin);
       return 1;
    }
    
@@ -104,12 +105,16 @@ int main(int argc, char *argv[])
    if (err != 0)
    {
       fprintf(stderr, "Failed to create the encoder: %s\n", opus_strerror(err));
+      fclose(fin);
+      fclose(fout);
       return 1;
    }
    dec = opus_custom_decoder_create(mode, channels, &err);
    if (err != 0)
    {
       fprintf(stderr, "Failed to create the decoder: %s\n", opus_strerror(err));
+      fclose(fin);
+      fclose(fout);
       return 1;
    }
    opus_custom_decoder_ctl(dec, OPUS_GET_LOOKAHEAD(&skip));
index 23c2128..e1884f3 100644 (file)
@@ -312,6 +312,7 @@ int main(int argc, char *argv[])
     if (!fout)
     {
         fprintf (stderr, "Could not open output file %s\n", argv[argc-1]);
+        fclose(fin);
         return EXIT_FAILURE;
     }
 
@@ -321,6 +322,8 @@ int main(int argc, char *argv[])
        if (err != OPUS_OK)
        {
           fprintf(stderr, "Cannot create encoder: %s\n", opus_strerror(err));
+          fclose(fin);
+          fclose(fout);
           return EXIT_FAILURE;
        }
        opus_encoder_ctl(enc, OPUS_SET_BITRATE(bitrate_bps));
@@ -341,6 +344,8 @@ int main(int argc, char *argv[])
        if (err != OPUS_OK)
        {
           fprintf(stderr, "Cannot create decoder: %s\n", opus_strerror(err));
+          fclose(fin);
+          fclose(fout);
           return EXIT_FAILURE;
        }
     }
@@ -434,6 +439,8 @@ int main(int argc, char *argv[])
             if (len[toggle] < 0)
             {
                 fprintf (stderr, "opus_encode() returned %d\n", len[toggle]);
+                fclose(fin);
+                fclose(fout);
                 return EXIT_FAILURE;
             }
         }
@@ -485,6 +492,8 @@ int main(int argc, char *argv[])
                          (long)count,
                          (unsigned long)enc_final_range[toggle^use_inbandfec],
                          (unsigned long)dec_final_range);
+            fclose(fin);
+            fclose(fout);
             return EXIT_FAILURE;
         }
 
index 6554c8c..9a93df5 100644 (file)
@@ -72,6 +72,7 @@ int main(int argc, char *argv[])
    if(fout==NULL)
    {
      fprintf(stderr, "Error opening output file: %s\n", argv[argc-1]);
+     fclose(fin);
      return 1;
    }
 
@@ -94,6 +95,8 @@ int main(int argc, char *argv[])
                 eof = 1;
              } else {
                 fprintf(stderr, "Invalid payload length\n");
+                fclose(fin);
+                fclose(fout);
                 return 1;
              }
              break;