lkml.org 
[lkml]   [2017]   [Apr]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH] xen/9pfs: use %d format specifier instead of %u for a signed int
Date
From: Colin Ian King <colin.king@canonical.com>

variable i is a signed int, so use the %d print format specifier instead of
%u. Fixes two warnings by cppcheck.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
net/9p/trans_xen.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/net/9p/trans_xen.c b/net/9p/trans_xen.c
index 47864245c0d4..9b66087bbc91 100644
--- a/net/9p/trans_xen.c
+++ b/net/9p/trans_xen.c
@@ -441,13 +441,13 @@ static int xen_9pfs_front_probe(struct xenbus_device *dev,
char str[16];

BUILD_BUG_ON(XEN_9PFS_NUM_RINGS > 9);
- sprintf(str, "ring-ref%u", i);
+ sprintf(str, "ring-ref%d", i);
ret = xenbus_printf(xbt, dev->nodename, str, "%d",
priv->rings[i].ref);
if (ret)
goto error_xenbus;

- sprintf(str, "event-channel-%u", i);
+ sprintf(str, "event-channel-%d", i);
ret = xenbus_printf(xbt, dev->nodename, str, "%u",
priv->rings[i].evtchn);
if (ret)
--
2.11.0
\
 
 \ /
  Last update: 2017-04-11 23:30    [W:0.034 / U:0.340 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site