From b8cd7a3c8df2c3aac8d7a0b5a02d83caf61bd769 Mon Sep 17 00:00:00 2001 From: Diego Biurrun Date: Fri, 11 Dec 2015 19:52:36 +0100 Subject: [PATCH] dvbsub: Check for errors from system() MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit libavcodec/dvbsubdec.c:145:5: warning: ignoring return value of ‘system’, declared with attribute warn_unused_result [-Wunused-result] libavcodec/dvbsubdec.c:148:5: warning: ignoring return value of ‘system’, declared with attribute warn_unused_result [-Wunused-result] --- libavcodec/dvbsubdec.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/libavcodec/dvbsubdec.c b/libavcodec/dvbsubdec.c index ccdfc01089..e512b97a07 100644 --- a/libavcodec/dvbsubdec.c +++ b/libavcodec/dvbsubdec.c @@ -84,10 +84,16 @@ static void png_save(const char *filename, uint32_t *bitmap, int w, int h) 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"); + 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); + return; + } } #endif