Improve type safety of new logging system and its handling of varargs

- Replaced loglevel arguments and globals using the `int` type with the `loglevel` enum.
- Moved logging print func handler function declaration to typedef.
- Fixed misuse of `print_func` where a char* was passed in place of `va_list` via a wrapper function `print_funcf`.
- Fixed reuse of varargs in `logger` causing a segfault when `stderr_enabled` is true.
- Fixed length in `snprintf` call inside `logger_hex_dump` truncating the printed text.

1 files changed, 2 insertions, 2 deletions

diff --git a/src/idevicerestore.c b/src/idevicerestore.c
index 3dd4bb9..f4ced47 100644
--- a/src/idevicerestore.c
+++ b/src/idevicerestore.c
@@ -1744,7 +1744,7 @@ static void plain_progress_func(struct progress_info_entry** progress_info, int
 	}
 }
 
-static void tty_print(int level, const char* format, va_list varglist)
+static void tty_print(enum loglevel level, const char* fmt, va_list ap)
 {
 	switch (level) {
 		case 0:
@@ -1760,7 +1760,7 @@ static void tty_print(int level, const char* format, va_list varglist)
 			break;
 	}
 
-	cvfprintf(stdout, format, varglist);
+	cvfprintf(stdout, fmt, ap);
 
 	cprintf(COLOR_RESET);
 }