[PATCH v7 2/4] libusbg: Add fputs()/fgets()/fprintf() error

[Stanislaw Wadas]

Add error handling to fputs()/fgets()/fprintf() functions.

Signed-off-by: Stanislaw Wadas <s.wa...@samsung.com>
---
 src/usbg.c |   16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/src/usbg.c b/src/usbg.c
index 198b504..bbc5db5 100644
--- a/src/usbg.c
+++ b/src/usbg.c
@@ -36,12 +36,14 @@
                         char *str;\
                         fprintf(stderr, "%s()  "msg" \n", \
                                 __func__, ##__VA_ARGS__);\
+                        fflush(stderr);\
                     } while (0)
 
 #define ERRORNO(msg, ...) do {\
                         char *str;\
                         fprintf(stderr, "%s()  %s: "msg" \n", \
                                 __func__, strerror(errno), ##__VA_ARGS__);\
+                        fflush(stderr);\
                     } while (0)
 
 static int usbg_lookup_function_type(char *name)
@@ -93,6 +95,11 @@ static char *usbg_read_buf(char *path, char *name, char 
*file, char *buf)
                goto out;
 
        ret = fgets(buf, USBG_MAX_STR_LENGTH, fp);
+       if (ret == NULL) {
+               ERROR("read error");
+               fclose(fp);
+               return ret;
+       }
 
        fclose(fp);
 
@@ -127,6 +134,7 @@ static void usbg_write_buf(char *path, char *name, char 
*file, char *buf)
 {
        char p[USBG_MAX_STR_LENGTH];
        FILE *fp;
+       int err;
 
        sprintf(p, "%s/%s/%s", path, name, file);
 
@@ -137,8 +145,14 @@ static void usbg_write_buf(char *path, char *name, char 
*file, char *buf)
        }
 
        fputs(buf, fp);
-
+       fflush(fp);
+       err = ferror(fp);
        fclose(fp);
+       
+       if (err){
+               ERROR("write error");
+               return;
+       }
 }
 
 static void usbg_write_int(char *path, char *name, char *file, int value, char 
*str)
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html