Fix: relayd: wrong specifier used in DBG format string
authorJérémie Galarneau <jeremie.galarneau@efficios.com>
Fri, 17 Jul 2020 18:26:17 +0000 (14:26 -0400)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Fri, 17 Jul 2020 18:29:31 +0000 (14:29 -0400)
`len` is of type uint64_t while the format string specifies a size of
`%zd`. This results in a warning on most 32-bit architectures.

  In file included from ../../../src/common/common.h:12:0,
                   from live.c:33:
  live.c: In function `viewer_get_metadata`:
  ../../../src/common/error.h:161:35: warning: format `%zd` expects
  argument of type `signed size_t`, but argument 6 has type `uint64_t
  {aka long long unsigned int}` [-Wformat=]
   #define DBG(fmt, args...) _ERRMSG("DEBUG1", PRINT_DBG, fmt, ## args)
                                     ^
  ../../../src/common/error.h:136:51: note: in definition of macro `__lttng_print`
      fprintf((type) == PRINT_MSG ? stdout : stderr, fmt, ## args); \
                                                     ^~~
  ../../../src/common/error.h:161:27: note: in expansion of macro `_ERRMSG`
   #define DBG(fmt, args...) _ERRMSG("DEBUG1", PRINT_DBG, fmt, ## args)
                             ^~~~~~~
  live.c:2051:4: note: in expansion of macro `DBG`
      DBG("Failed to read metadata: requested = %zd, got = %zd",
      ^~~

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Change-Id: I108191e0a94e712b0596d0d52a21cf8c971d6c1e

src/bin/lttng-relayd/live.c

index 9074f24f367e8ca88196ef763781b9f16493ef16..423a97975587880c0665b173f42d24c599d60494 100644 (file)
@@ -2048,7 +2048,7 @@ int viewer_get_metadata(struct relay_connection *conn)
                                        vstream->stream_file.handle, -read_len,
                                        SEEK_CUR);
 
-                       DBG("Failed to read metadata: requested = %zd, got = %zd",
+                       DBG("Failed to read metadata: requested = %" PRIu64 ", got = %zd",
                                        len, read_len);
                        read_len = 0;
                        len = 0;
This page took 0.041458 seconds and 4 git commands to generate.