diff options
| author | Stefan Weil <sw@weilnetz.de> | 2012-02-25 14:11:46 +0100 |
|---|---|---|
| committer | Stefan Hajnoczi <stefanha@linux.vnet.ibm.com> | 2012-03-07 13:03:51 +0000 |
| commit | 1c467855da1c5568e8968fe1b9dd3b4ad9f8637a (patch) | |
| tree | 1472085571adcc81b4178bf270b78d031f07a338 | |
| parent | 75d1234103d47e59311203bf5e73f85b2e0fd983 (diff) | |
| download | focaccia-qemu-1c467855da1c5568e8968fe1b9dd3b4ad9f8637a.tar.gz focaccia-qemu-1c467855da1c5568e8968fe1b9dd3b4ad9f8637a.zip | |
Fix sign of sscanf format specifiers
All values read by sscanf are unsigned, so replace %d by %u. This signed / unsigned mismatch was detected by splint. Signed-off-by: Stefan Weil <sw@weilnetz.de> Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
| -rw-r--r-- | cursor.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/cursor.c b/cursor.c index efc5917029..76e262caf7 100644 --- a/cursor.c +++ b/cursor.c @@ -15,7 +15,8 @@ static QEMUCursor *cursor_parse_xpm(const char *xpm[]) uint8_t idx; /* parse header line: width, height, #colors, #chars */ - if (sscanf(xpm[line], "%d %d %d %d", &width, &height, &colors, &chars) != 4) { + if (sscanf(xpm[line], "%u %u %u %u", + &width, &height, &colors, &chars) != 4) { fprintf(stderr, "%s: header parse error: \"%s\"\n", __FUNCTION__, xpm[line]); return NULL; |