Merge commit 'b8cd7a3c8df2c3aac8d7a0b5a02d83caf61bd769'
authorClément Bœsch <u@pkh.me>
Mon, 3 Apr 2017 18:57:03 +0000 (20:57 +0200)
committerClément Bœsch <u@pkh.me>
Mon, 3 Apr 2017 19:01:40 +0000 (21:01 +0200)
* commit 'b8cd7a3c8df2c3aac8d7a0b5a02d83caf61bd769':
  dvbsub: Check for errors from system()

printf(...) replaced with fprintf(stderr, ...) for logging errors.

Merged-by: Clément Bœsch <u@pkh.me>
1  2 
libavcodec/dvbsubdec.c

@@@ -85,10 -84,16 +85,16 @@@ static void png_save(const char *filena
      fclose(f);
  
      snprintf(command, sizeof(command), "pnmtopng -alpha %s %s > %s.png 2> /dev/null", fname2, fname, filename);
-     system(command);
+     if (system(command) != 0) {
 -        printf("Error running pnmtopng\n");
++        fprintf(stderr, "Error running pnmtopng\n");
+         return;
+     }
  
      snprintf(command, sizeof(command), "rm %s %s", fname, fname2);
-     system(command);
+     if (system(command) != 0) {
 -        printf("Error removing %s and %s\n", fname, fname2);
++        fprintf(stderr, "Error removing %s and %s\n", fname, fname2);
+         return;
+     }
  }
  #endif