[PATCH] bs_sheepdog.c: fix length modifier of eprintf in read_write_object()
Ryusuke Konishi <[email protected]>
| Newsgroups | org.kernel.vger.stgt |
|---|---|
| Message-ID | <[email protected]> |
On 32-bit environments, the sheepdog driver causes the following compile warning because "%lx" is used for uint64_t type variables: cc -c -DUSE_SIGNALFD -DUSE_TIMERFD -D_GNU_SOURCE -I. -g -O2 -fno-strict-aliasing -Wall -Wstrict-prototypes -fPIC -DTGT_VERSION=\"\" -DBSDIR=\"/lib/tgt/backing-store\" bs_sheepdog.c -o bs_sheepdog.o bs_sheepdog.c: In function 'read_write_object': bs_sheepdog.c:682: warning: format '%lx' expects type 'long unsigned int', but argument 5 has type 'uint64_t' bs_sheepdog.c:682: warning: format '%lx' expects type 'long unsigned int', but argument 6 has type 'uint64_t' This fixes the issue by using PRIx64 length modifier instead of "%lx". Signed-off-by: Ryusuke Konishi <[email protected]> Cc: Hitoshi Mitake <[email protected]> --- usr/bs_sheepdog.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/usr/bs_sheepdog.c b/usr/bs_sheepdog.c index 9848e9f..315f3aa 100644 --- a/usr/bs_sheepdog.c +++ b/usr/bs_sheepdog.c @@ -679,7 +679,7 @@ static int read_write_object(struct sheepdog_access_info *ai, char *buf, *need_reload = 1; return 0; default: - eprintf("%s (oid: %lx, old_oid: %lx)\n", + eprintf("%s (oid: %" PRIx64 ", old_oid: %" PRIx64 ")\n", sd_strerror(rsp->result), oid, old_oid); return -1; } -- 1.7.9.3