Log only the filename, not the full path in LOGGER.

auto_tests/helpers.h

@@ -43,16 +43,14 @@ static const char *tox_log_level_name(TOX_LOG_LEVEL level)
     return "<unknown>";
 }
 
-static void print_debug_log(Tox *m, TOX_LOG_LEVEL level, const char *path, uint32_t line, const char *func,
+static void print_debug_log(Tox *m, TOX_LOG_LEVEL level, const char *file, uint32_t line, const char *func,
                             const char *message, void *user_data)
 {
     if (level == TOX_LOG_LEVEL_TRACE) {
         return;
     }
 
     uint32_t index = user_data ? *(uint32_t *)user_data : 0;
-    const char *file = strrchr(path, '/');
-    file = file ? file + 1 : path;
     fprintf(stderr, "[#%d] %s %s:%d\t%s:\t%s\n", index, tox_log_level_name(level), file, line, func, message);
 }
 

toxcore/logger.c

@@ -31,6 +31,7 @@
 #include <stdarg.h>
 #include <stdio.h>
 #include <stdlib.h>
+#include <string.h>
 
 
 struct Logger {
@@ -111,7 +112,19 @@ void logger_write(const Logger *log, LOGGER_LEVEL level, const char *file, int l
         return;
     }
 
-    /* Format message */
+    // Only pass the file name, not the entire file path, for privacy reasons.
+    // The full path may contain PII of the person compiling toxcore (their
+    // username and directory layout).
+    const char *filename = strrchr(file, '/');
+    file = filename ? filename + 1 : file;
+#if defined(_WIN32) || defined(__CYGWIN__)
+    // On Windows, the path separator *may* be a backslash, so we look for that
+    // one too.
+    const char *windows_filename = strrchr(file, '\\');
+    file = windows_filename ? windows_filename + 1 : file;
+#endif
+
+    // Format message
     char msg[1024];
     va_list args;
     va_start(args, format);