cleanup: move error handling into write_retry

If write_retry fails to write everything, it is *always* an error.

This commit renames write_retry to write_data and moves error handling
into the function, making a lot of error handling code redundant.

Signed-off-by: David Oberhollenzer <david.oberhollenzer@sigma-star.at>

1 files changed, 30 insertions, 0 deletions

diff --git a/lib/util/write_data.c b/lib/util/write_data.c
new file mode 100644
index 0000000..82f3aca
--- /dev/null
+++ b/lib/util/write_data.c
@@ -0,0 +1,30 @@
+/* SPDX-License-Identifier: GPL-3.0-or-later */
+#include <unistd.h>
+#include <errno.h>
+#include <stdio.h>
+
+#include "util.h"
+
+int write_data(const char *errstr, int fd, const void *data, size_t size)
+{
+	ssize_t ret;
+
+	while (size > 0) {
+		ret = write(fd, data, size);
+		if (ret == 0) {
+			fprintf(stderr, "%s: write truncated\n", errstr);
+			return -1;
+		}
+		if (ret < 0) {
+			if (errno == EINTR)
+				continue;
+			perror(errstr);
+			return -1;
+		}
+
+		data = (const char *)data + ret;
+		size -= ret;
+	}
+
+	return 0;
+}