common/logf: Check return value of fflush

This commit adds proper checks for the return value of fsync. Signed-off-by: Felix Wruck <[email protected]>
gyroidos · Feb 20, 2025 · f57ae88 · f57ae88
1 parent 0fc105d
commit f57ae88
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/common/logf.c b/common/logf.c
@@ -378,7 +378,12 @@ logf_file_write(logf_prio_t prio, const char *msg, void *data)
 
 	logf_file_write_timestamp(data);
 	fprintf(data, "[%u] %s %s\n", getpid(), prio_str(prio), msg);
-	fflush(data);
+
+	int res = -1;
+	if (0 != (res = fflush(data))) {
+		ERROR_ERRNO("Failed to flush log file for PID %d after message %s: %d", getpid(),
+			    msg, res);
+	}
 }
 
 void
@@ -388,7 +393,12 @@ logf_test_write(logf_prio_t prio, const char *msg, void *data)
 		return;
 
 	fprintf(data, "[%u] %s %s\n", getpid(), prio_str(prio), msg);
-	fflush(data);
+
+	int res = -1;
+	if (0 != (res = fflush(data))) {
+		ERROR_ERRNO("Failed to flush log file for PID %d after message %s: %d", getpid(),
+			    msg, res);
+	}
 }
 
 void *