[PATCH 12/18] wmcdplay: Fix -Wunused-result compiler warnings.

[Doug Torrance]

In particular, fix warnings of the form "warning: ignoring return value of
‘char* fgets(char*, int, FILE*)’, declared with attribute warn_unused_result" by
checking return value of fgets.
---
 wmcdplay/wmcdplay.cc | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/wmcdplay/wmcdplay.cc b/wmcdplay/wmcdplay.cc
index 0108b89..ab18f4a 100644
--- a/wmcdplay/wmcdplay.cc
+++ b/wmcdplay/wmcdplay.cc
@@ -644,7 +644,10 @@ bool readArtwork(char *artfilen){
    char buf[256];
    bool done=false;
    while(!done){
-      fgets(buf, 250, artfile);
+      if (fgets(buf, 250, artfile) == NULL) {
+         fprintf(stderr,"%s : Error reading artwork file.\n", NAME);
+         return false;
+      }
       done=(feof(artfile)!=0);
       if(!done){
 
@@ -734,7 +737,10 @@ char *readBlock(FILE *dfile){
    long bytes=0;
    char *block=NULL;
    do{
-      fgets(buf, 250, dfile);
+      if (fgets(buf, 250, dfile) == NULL) {
+         fprintf(stderr,"%s : Error reading artwork file.\n", NAME);
+         return NULL;
+      }
       int buflen=strlen(buf);
       block=(char *)realloc(block, sizeof(char)*(bytes+buflen+1));
       strcpy(block+bytes, buf);
-- 
2.1.0


-- 
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.