winetest: Don't use plain strings as format specifiers.

When they contain a percent character they would, at best, not be printed correctly, and at worst cause a crash.

winetest: Don't use plain strings as format specifiers.
b07129ba · Francois Gouget · Alexandre Julliard · be6cc4f1 · b07129ba
Commit b07129ba authored Feb 13, 2023 by Francois Gouget Committed by Alexandre Julliard Feb 14, 2023
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

main.c programs/winetest/main.c +2 -2

No files found.
--- a/programs/winetest/main.c
+++ b/programs/winetest/main.c
@@ -1122,7 +1122,7 @@ run_tests (char *logname, char *outdir)
        logfile = create_temp_file( tmpname );
        logname = tmpname;
    }
-    report (R_OUT, logname);
+    report (R_OUT, "%s", logname);

    if (logfile == INVALID_HANDLE_VALUE)
        report (R_FATAL, "Could not open logfile: %u", GetLastError());
@@ -1148,7 +1148,7 @@ run_tests (char *logname, char *outdir)
    if (!newdir && (!outdir || GetLastError() != ERROR_ALREADY_EXISTS))
        report (R_FATAL, "Could not create directory %s (%d)", tempdir, GetLastError());

-    report (R_DIR, tempdir);
+    report (R_DIR, "%s", tempdir);

    xprintf ("Version 4\n");
    xprintf ("Tests from build %s\n", build_id[0] ? build_id : "-" );