From 4d5292c4da4583a4cc9602c14254ca84d9449d99 Mon Sep 17 00:00:00 2001 From: Daniel Veillard Date: Mon, 19 Mar 2007 14:20:30 +0000 Subject: [PATCH] * src/virsh.c: applied patch from Jim Meyering to add printf-style function checking in virsh.c and fix one warning raised Daniel --- ChangeLog | 5 +++++ src/virsh.c | 8 ++++---- 2 files changed, 9 insertions(+), 4 deletions(-) diff --git a/ChangeLog b/ChangeLog index a1a03db28e..dae6b5da7a 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +Mon Mar 19 15:19:06 CET 2007 Daniel Veillard + + * src/virsh.c: applied patch from Jim Meyering to add printf-style + function checking in virsh.c and fix one warning raised + Mon Mar 19 15:15:51 CET 2007 Daniel Veillard * qemud/protocol.h src/internal.h: applied patch from Jim Meyering diff --git a/src/virsh.c b/src/virsh.c index 521e2fcc8a..5302d4e8f7 100644 --- a/src/virsh.c +++ b/src/virsh.c @@ -179,8 +179,8 @@ typedef struct __vshControl { static vshCmdDef commands[]; -static void vshError(vshControl * ctl, int doexit, const char *format, - ...); +static void vshError(vshControl * ctl, int doexit, const char *format, ...) + ATTRIBUTE_FORMAT(printf, 3, 4); static int vshInit(vshControl * ctl); static int vshDeinit(vshControl * ctl); static void vshUsage(vshControl * ctl, const char *cmdname); @@ -3158,8 +3158,8 @@ _vshStrdup(vshControl * ctl, const char *s, const char *filename, int line) if ((x = strdup(s))) return x; - vshError(ctl, TRUE, _("%s: %d: failed to allocate %d bytes"), - filename, line, strlen(s)); + vshError(ctl, TRUE, _("%s: %d: failed to allocate %lu bytes"), + filename, line, (unsigned long)strlen(s)); return NULL; } -- GitLab