Remove ostream_printf

By cobbling together the xattr lines manually in libtar, the
need (and thus the function itself) are removed.

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

3 files changed, 12 insertions, 34 deletions

diff --git a/lib/io/Makemodule.am b/lib/io/Makemodule.am
index b795788..71f01e8 100644
--- a/lib/io/Makemodule.am
+++ b/lib/io/Makemodule.am
@@ -1,6 +1,6 @@
 libio_a_SOURCES = include/io/istream.h include/io/ostream.h include/io/xfrm.h \
 	include/io/file.h include/io/std.h \
-	lib/io/src/internal.h lib/io/src/ostream.c lib/io/src/printf.c \
+	lib/io/src/internal.h lib/io/src/ostream.c \
 	lib/io/src/istream.c lib/io/src/get_line.c lib/io/src/xfrm/ostream.c \
 	lib/io/src/xfrm/istream.c
 libio_a_CFLAGS = $(AM_CFLAGS) $(ZLIB_CFLAGS) $(XZ_CFLAGS)
diff --git a/lib/io/src/printf.c b/lib/io/src/printf.c
deleted file mode 100644
index 3850487..0000000
--- a/lib/io/src/printf.c
+++ /dev/null
@@ -1,30 +0,0 @@
-/* SPDX-License-Identifier: GPL-3.0-or-later */
-/*
- * printf.c
- *
- * Copyright (C) 2019 David Oberhollenzer <goliath@infraroot.at>
- */
-#include "internal.h"
-
-int ostream_printf(ostream_t *strm, const char *fmt, ...)
-{
-	char *temp = NULL;
-	va_list ap;
-	int ret;
-
-	va_start(ap, fmt);
-
-	ret = vasprintf(&temp, fmt, ap);
-	if (ret < 0)
-		perror(strm->get_filename(strm));
-	va_end(ap);
-
-	if (ret < 0)
-		return -1;
-
-	if (strm->append(strm, temp, ret))
-		ret = -1;
-
-	free(temp);
-	return ret;
-}
diff --git a/lib/tar/src/write_header.c b/lib/tar/src/write_header.c
index 5d98686..ed780b5 100644
--- a/lib/tar/src/write_header.c
+++ b/lib/tar/src/write_header.c
@@ -164,13 +164,21 @@ static int write_schily_xattr(ostream_t *fp, const struct stat *orig,
 		return -1;
 
 	for (it = xattr; it != NULL; it = it->next) {
+		char buffer[64];
+
 		len = strlen(prefix) + strlen(it->key) + it->value_len + 3;
 		len += prefix_digit_len(len);
 
-		if (ostream_printf(fp, PRI_SZ " %s%s=",
-				   len, prefix, it->key) < 0) {
+		sprintf(buffer, PRI_SZ " ", len);
+
+		if (ostream_append(fp, buffer, strlen(buffer)))
+			return -1;
+		if (ostream_append(fp, prefix, strlen(prefix)))
+			return -1;
+		if (ostream_append(fp, it->key, strlen(it->key)))
+			return -1;
+		if (ostream_append(fp, "=", 1))
 			return -1;
-		}
 		if (ostream_append(fp, it->value, it->value_len))
 			return -1;
 		if (ostream_append(fp, "\n", 1))