From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36553) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vl2Ej-0008C3-Nx for qemu-devel@nongnu.org; Mon, 25 Nov 2013 14:54:26 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Vl2Ei-0001No-UV for qemu-devel@nongnu.org; Mon, 25 Nov 2013 14:54:25 -0500 Received: from qemu.weilnetz.de ([2a03:4000:2:362::1]:60929 helo=v2201305906712890.yourvserver.net) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vl2Ei-0001NV-OA for qemu-devel@nongnu.org; Mon, 25 Nov 2013 14:54:24 -0500 From: Stefan Weil Date: Mon, 25 Nov 2013 20:54:17 +0100 Message-Id: <1385409257-2522-1-git-send-email-sw@weilnetz.de> Subject: [Qemu-devel] [PATCH for 1.7] qga: Fix two format strings for MinGW List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: Stefan Weil , qemu-devel Both code locations cause a compiler warning. Using "%s" instead of "%lu" would result in a program crash if the wrong code were executed. Signed-off-by: Stefan Weil --- qga/commands-win32.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/qga/commands-win32.c b/qga/commands-win32.c index 7a37f5c..a6a0af2 100644 --- a/qga/commands-win32.c +++ b/qga/commands-win32.c @@ -110,7 +110,7 @@ void qmp_guest_shutdown(bool has_mode, const char *mode, Error **err) } if (!ExitWindowsEx(shutdown_flag, SHTDN_REASON_FLAG_PLANNED)) { - slog("guest-shutdown failed: %d", GetLastError()); + slog("guest-shutdown failed: %lu", GetLastError()); error_set(err, QERR_UNDEFINED_ERROR); } } @@ -301,7 +301,7 @@ static DWORD WINAPI do_suspend(LPVOID opaque) DWORD ret = 0; if (!SetSuspendState(*mode == GUEST_SUSPEND_MODE_DISK, TRUE, TRUE)) { - slog("failed to suspend guest, %s", GetLastError()); + slog("failed to suspend guest, %lu", GetLastError()); ret = -1; } g_free(mode); -- 1.7.10.4