From: Jean-Philippe Andre <jp.andre@samsung.com>
Date: Wed, 13 Jul 2016 06:21:06 +0000 (+0900)
Subject: evas: Fix some CID in generic loaders
X-Git-Tag: upstream/1.20.0~5196
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3ad848f5c5a580d0c07178c457a687b2bd99de2b;p=platform%2Fupstream%2Fefl.git

evas: Fix some CID in generic loaders

Check the return value of fwrite. There is nothing
sensible to do as the receiver needs to handle the error
gracefully.

CID 1356613, 1356614, 1356615, 1356200
---

diff --git a/src/generic/evas/pdf/main.cpp b/src/generic/evas/pdf/main.cpp
index 8623179..eab137b 100644
--- a/src/generic/evas/pdf/main.cpp
+++ b/src/generic/evas/pdf/main.cpp
@@ -321,7 +321,12 @@ main(int argc, char **argv)
              // could also to "tmpfile %s\n" like shmfile but just
              // a mmaped tmp file on the system
              printf("data\n");
-             fwrite(data, crop_width * crop_height * sizeof(DATA32), 1, stdout);
+             if (fwrite(data, crop_width * crop_height * sizeof(DATA32), 1, stdout) != 1)
+               {
+                  // nothing much to do, the receiver will simply ignore the
+                  // data as it's too short
+                  D("fwrite failed (%d bytes): %m\n", crop_width * crop_height * sizeof(DATA32));
+               }
           }
         shm_free();
      }
diff --git a/src/generic/evas/ps/main.c b/src/generic/evas/ps/main.c
index 48e8a4b..c5b8a18 100644
--- a/src/generic/evas/ps/main.c
+++ b/src/generic/evas/ps/main.c
@@ -276,7 +276,12 @@ main(int argc, char **argv)
              // could also to "tmpfile %s\n" like shmfile but just
              // a mmaped tmp file on the system
              printf("data\n");
-             fwrite(data, width * height * sizeof(DATA32), 1, stdout);
+             if (fwrite(data, width * height * sizeof(DATA32), 1, stdout) != 1)
+               {
+                  // nothing much to do, the receiver will simply ignore the
+                  // data as it's too short
+                  D("fwrite failed (%d bytes): %m\n", width * height * sizeof(DATA32));
+               }
           }
         shm_free();
      }
diff --git a/src/generic/evas/svg/main.c b/src/generic/evas/svg/main.c
index 8c46708..f197330 100644
--- a/src/generic/evas/svg/main.c
+++ b/src/generic/evas/svg/main.c
@@ -218,7 +218,12 @@ int main(int argc, char **argv)
              else
                {
                   printf("data\n");
-                  fwrite(shm_addr, width * height * sizeof(DATA32), 1, stdout);
+                  if (fwrite(shm_addr, width * height * sizeof(DATA32), 1, stdout) != 1)
+                    {
+                       // nothing much to do, the receiver will simply ignore the
+                       // data as it's too short
+                       //D("fwrite failed (%d bytes): %m\n", width * height * sizeof(DATA32));
+                    }
                }
              shm_free();
           }
diff --git a/src/generic/evas/xcf/main.c b/src/generic/evas/xcf/main.c
index 4de03f8..c5ef6a8 100644
--- a/src/generic/evas/xcf/main.c
+++ b/src/generic/evas/xcf/main.c
@@ -1723,7 +1723,12 @@ main(int argc, char **argv)
              // could also to "tmpfile %s\n" like shmfile but just
              // a mmaped tmp file on the system
              printf("data\n");
-             fwrite(image->data, w * h * sizeof(DATA32), 1, stdout);
+             if (fwrite(image->data, w * h * sizeof(DATA32), 1, stdout) != 1)
+             {
+                // nothing much to do, the receiver will simply ignore the
+                // data as it's too short
+                D("fwrite failed (%d bytes): %m\n", w * h * sizeof(DATA32));
+             }
           }
         shm_free();
      }