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

[Stanislaw Wadas <s.wadas@xxxxxxxxxxx>]

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

Signed-off-by: Stanislaw Wadas <s.wadas@xxxxxxxxxxx>
---
 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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html