[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

[Libguestfs] [PATCH] Fix various -Wformat problems.



Updating gnulib has caused -Wformat-signedness to be enabled.  This
has revealed many problems in C format strings.  The fixes here fall
into the following main categories:

 - Using %d with an unsigned parameter.

 - %x and %o expect an unsigned argument.

 - uid_t and gid_t are unsigned on Linux.  The safe way to print these
   is to cast them to uintmax_t and then print then using the %ju
   modifier (see http://stackoverflow.com/a/1401581).

 - Using %d to print an enum.  Since enums may be either char or int,
   I fixed this by casting the enum to int.

 - strtol_error & lzma_ret are both unsigned types.
---
 builder/pxzcat-c.c     | 24 ++++++++++++------------
 cat/filesystems.c      |  2 +-
 cat/ls.c               |  5 +++--
 daemon/ext2.c          |  7 ++++++-
 daemon/file.c          |  2 +-
 daemon/guestfsd.c      |  3 ++-
 daemon/parted.c        |  5 +++--
 daemon/proto.c         | 14 ++++++++------
 daemon/umask.c         |  2 +-
 daemon/zero.c          |  2 +-
 diff/diff.c            |  7 ++++---
 fish/alloc.c           |  2 +-
 fish/rc.c              |  8 ++++----
 generator/bindtests.ml |  2 +-
 generator/daemon.ml    |  4 ++--
 generator/fish.ml      | 17 +++++++++--------
 make-fs/make-fs.c      |  2 +-
 p2v/main.c             |  5 +++--
 src/actions-support.c  | 10 +++++-----
 src/appliance.c        |  6 +++---
 src/fuse.c             | 14 +++++++-------
 src/handle.c           |  2 +-
 src/launch-libvirt.c   |  3 ++-
 src/launch.c           |  2 +-
 src/proto.c            |  2 +-
 test-tool/test-tool.c  |  2 +-
 26 files changed, 84 insertions(+), 70 deletions(-)

diff --git a/builder/pxzcat-c.c b/builder/pxzcat-c.c
index fb1a865..0c4a4be 100644
--- a/builder/pxzcat-c.c
+++ b/builder/pxzcat-c.c
@@ -298,7 +298,7 @@ parse_indexes (value filenamev, int fd)
     /* Does the stream footer look reasonable? */
     r = lzma_stream_footer_decode (&footer_flags, footer);
     if (r != LZMA_OK) {
-      fprintf (stderr, "invalid stream footer - error %d\n", r);
+      fprintf (stderr, "invalid stream footer - error %u\n", r);
       caml_invalid_argument ("invalid stream footer");
     }
 
@@ -317,7 +317,7 @@ parse_indexes (value filenamev, int fd)
     /* Decode the index. */
     r = lzma_index_decoder (&strm, &this_index, UINT64_MAX);
     if (r != LZMA_OK) {
-      fprintf (stderr, "invalid stream index - error %d\n", r);
+      fprintf (stderr, "invalid stream index - error %u\n", r);
       caml_invalid_argument ("invalid stream index");
     }
 
@@ -339,7 +339,7 @@ parse_indexes (value filenamev, int fd)
     } while (r == LZMA_OK);
 
     if (r != LZMA_STREAM_END) {
-      fprintf (stderr, "could not parse index - error %d\n", r);
+      fprintf (stderr, "could not parse index - error %u\n", r);
       caml_invalid_argument ("could not parse index");
     }
 
@@ -356,14 +356,14 @@ parse_indexes (value filenamev, int fd)
 
     r = lzma_stream_header_decode (&header_flags, header);
     if (r != LZMA_OK) {
-      fprintf (stderr, "invalid stream header - error %d\n", r);
+      fprintf (stderr, "invalid stream header - error %u\n", r);
       caml_invalid_argument ("invalid stream header");
     }
 
     /* Header and footer of the stream should be equal. */
     r = lzma_stream_flags_compare (&header_flags, &footer_flags);
     if (r != LZMA_OK) {
-      fprintf (stderr, "header and footer of stream are not equal - error %d\n",
+      fprintf (stderr, "header and footer of stream are not equal - error %u\n",
                r);
       caml_invalid_argument ("header and footer of stream are not equal");
     }
@@ -371,7 +371,7 @@ parse_indexes (value filenamev, int fd)
     /* Store the decoded stream flags in this_index. */
     r = lzma_index_stream_flags (this_index, &footer_flags);
     if (r != LZMA_OK) {
-      fprintf (stderr, "cannot read stream_flags from index - error %d\n", r);
+      fprintf (stderr, "cannot read stream_flags from index - error %u\n", r);
       caml_invalid_argument ("cannot read stream_flags from index");
     }
 
@@ -380,14 +380,14 @@ parse_indexes (value filenamev, int fd)
      */
     r = lzma_index_stream_padding (this_index, stream_padding);
     if (r != LZMA_OK) {
-      fprintf (stderr, "cannot set stream_padding in index - error %d\n", r);
+      fprintf (stderr, "cannot set stream_padding in index - error %u\n", r);
       caml_invalid_argument ("cannot set stream_padding in index");
     }
 
     if (combined_index != NULL) {
       r = lzma_index_cat (this_index, combined_index, NULL);
       if (r != LZMA_OK) {
-        fprintf (stderr, "cannot combine indexes - error %d\n", r);
+        fprintf (stderr, "cannot combine indexes - error %u\n", r);
         caml_invalid_argument ("cannot combine indexes");
       }
     }
@@ -613,7 +613,7 @@ worker_thread (void *vp)
 
     r = lzma_block_header_decode (&block, NULL, header);
     if (r != LZMA_OK) {
-      fprintf (stderr, "%s: invalid block header (error %d)\n",
+      fprintf (stderr, "%s: invalid block header (error %u)\n",
                global->filename, r);
       return &state->status;
     }
@@ -624,7 +624,7 @@ worker_thread (void *vp)
     r = lzma_block_compressed_size (&block, iter.block.unpadded_size);
     if (r != LZMA_OK) {
       fprintf (stderr,
-               "%s: cannot calculate compressed size (error %d)\n",
+               "%s: cannot calculate compressed size (error %u)\n",
                global->filename, r);
       return &state->status;
     }
@@ -635,7 +635,7 @@ worker_thread (void *vp)
     /* Read the block data and uncompress it. */
     r = lzma_block_decoder (&strm, &block);
     if (r != LZMA_OK) {
-      fprintf (stderr, "%s: invalid block (error %d)\n", global->filename, r);
+      fprintf (stderr, "%s: invalid block (error %u)\n", global->filename, r);
       return &state->status;
     }
 
@@ -684,7 +684,7 @@ worker_thread (void *vp)
         break;
       if (r != LZMA_OK) {
         fprintf (stderr,
-                 "%s: could not parse block data (error %d)\n",
+                 "%s: could not parse block data (error %u)\n",
                  global->filename, r);
         return &state->status;
       }
diff --git a/cat/filesystems.c b/cat/filesystems.c
index 44defe0..0e64e00 100644
--- a/cat/filesystems.c
+++ b/cat/filesystems.c
@@ -868,7 +868,7 @@ write_row (const char *name, const char *type,
     strings[len++] = vfs_label;
   if ((columns & COLUMN_MBR)) {
     if (mbr_id >= 0) {
-      snprintf (mbr_id_str, sizeof mbr_id_str, "%02x", mbr_id);
+      snprintf (mbr_id_str, sizeof mbr_id_str, "%02x", (unsigned) mbr_id);
       strings[len++] = mbr_id_str;
     } else
       strings[len++] = NULL;
diff --git a/cat/ls.c b/cat/ls.c
index 3bced54..987dcef 100644
--- a/cat/ls.c
+++ b/cat/ls.c
@@ -702,7 +702,7 @@ output_int64_perms (int64_t i)
 {
   next_field ();
   /* csv doesn't need escaping */
-  if (printf ("%04" PRIo64, i) < 0) {
+  if (printf ("%04" PRIo64, (uint64_t) i) < 0) {
     perror ("printf");
     exit (EXIT_FAILURE);
   }
@@ -774,7 +774,8 @@ output_int64_dev (int64_t i)
   next_field ();
 
   /* csv doesn't need escaping */
-  if (printf ("%d:%d", major (dev), minor (dev)) < 0) {
+  if (printf ("%ju:%ju",
+              (uintmax_t) major (dev), (uintmax_t) minor (dev)) < 0) {
     perror ("printf");
     exit (EXIT_FAILURE);
   }
diff --git a/daemon/ext2.c b/daemon/ext2.c
index 8ef6d5f..9142a38 100644
--- a/daemon/ext2.c
+++ b/daemon/ext2.c
@@ -780,11 +780,16 @@ do_get_e2generation (const char *filename)
     return -1;
   }
 
-  if (sscanf (out, "%" SCNu64, &ret) != 1) {
+  if (sscanf (out, "%" SCNi64, &ret) != 1) {
     reply_with_error ("cannot parse output from '%s' command: %s",
                       "lsattr", out);
     return -1;
   }
+  if (ret < 0) {
+    reply_with_error ("unexpected negative number from '%s' command: %s",
+                      "lsattr", out);
+    return -1;
+  }
 
   return ret;
 }
diff --git a/daemon/file.c b/daemon/file.c
index bb3b3c1..c609a01 100644
--- a/daemon/file.c
+++ b/daemon/file.c
@@ -140,7 +140,7 @@ do_chmod (int mode, const char *path)
   CHROOT_OUT;
 
   if (r == -1) {
-    reply_with_perror ("%s: 0%o", path, mode);
+    reply_with_perror ("%s: 0%o", path, (unsigned) mode);
     return -1;
   }
 
diff --git a/daemon/guestfsd.c b/daemon/guestfsd.c
index 7f4b2f2..ee0aa43 100644
--- a/daemon/guestfsd.c
+++ b/daemon/guestfsd.c
@@ -224,7 +224,8 @@ main (int argc, char *argv[])
       exit (EXIT_SUCCESS);
 
     default:
-      fprintf (stderr, "guestfsd: unexpected command line option 0x%x\n", c);
+      fprintf (stderr, "guestfsd: unexpected command line option 0x%x\n",
+               (unsigned) c);
       exit (EXIT_FAILURE);
     }
   }
diff --git a/daemon/parted.c b/daemon/parted.c
index b516067..cb6c486 100644
--- a/daemon/parted.c
+++ b/daemon/parted.c
@@ -752,7 +752,7 @@ do_part_get_mbr_id (const char *device, int partnum)
   udev_settle ();
 
   /* It's printed in hex ... */
-  int id;
+  unsigned id;
   if (sscanf (out, "%x", &id) != 1) {
     reply_with_error ("sfdisk --print-id: cannot parse output: %s", out);
     return -1;
@@ -775,7 +775,8 @@ do_part_set_mbr_id (const char *device, int partnum, int idbyte)
   snprintf (partnum_str, sizeof partnum_str, "%d", partnum);
 
   char idbyte_str[16];
-  snprintf (idbyte_str, sizeof partnum_str, "%x", idbyte); /* NB: hex */
+  /* NB: hex */
+  snprintf (idbyte_str, sizeof partnum_str, "%x", (unsigned) idbyte);
 
   CLEANUP_FREE char *err = NULL;
   int r;
diff --git a/daemon/proto.c b/daemon/proto.c
index 7ae8c66..df63bfd 100644
--- a/daemon/proto.c
+++ b/daemon/proto.c
@@ -157,19 +157,20 @@ main_loop (int _sock)
 
     /* Check the version etc. */
     if (hdr.prog != GUESTFS_PROGRAM) {
-      reply_with_error ("wrong program (%d)", hdr.prog);
+      reply_with_error ("wrong program (%u)", hdr.prog);
       goto cont;
     }
     if (hdr.vers != GUESTFS_PROTOCOL_VERSION) {
-      reply_with_error ("wrong protocol version (%d)", hdr.vers);
+      reply_with_error ("wrong protocol version (%u)", hdr.vers);
       goto cont;
     }
     if (hdr.direction != GUESTFS_DIRECTION_CALL) {
-      reply_with_error ("unexpected message direction (%d)", hdr.direction);
+      reply_with_error ("unexpected message direction (%d)",
+                        (int) hdr.direction);
       goto cont;
     }
     if (hdr.status != GUESTFS_STATUS_OK) {
-      reply_with_error ("unexpected message status (%d)", hdr.status);
+      reply_with_error ("unexpected message status (%d)", (int) hdr.status);
       goto cont;
     }
 
@@ -444,8 +445,9 @@ receive_file (receive_cb cb, void *opaque)
 
     if (verbose)
       fprintf (stderr,
-               "guestfsd: receive_file: got chunk: cancel = 0x%x, len = %d, buf = %p\n",
-               chunk.cancel, chunk.data.data_len, chunk.data.data_val);
+               "guestfsd: receive_file: got chunk: cancel = 0x%x, len = %u, buf = %p\n",
+               (unsigned) chunk.cancel,
+               chunk.data.data_len, chunk.data.data_val);
 
     if (chunk.cancel != 0 && chunk.cancel != 1) {
       fprintf (stderr,
diff --git a/daemon/umask.c b/daemon/umask.c
index 52e854e..475c820 100644
--- a/daemon/umask.c
+++ b/daemon/umask.c
@@ -36,7 +36,7 @@ do_umask (int mask)
   int r;
 
   if (mask < 0 || mask > 0777) {
-    reply_with_error ("0%o: mask negative or out of range", mask);
+    reply_with_error ("0%o: mask negative or out of range", (unsigned) mask);
     return -1;
   }
 
diff --git a/daemon/zero.c b/daemon/zero.c
index 505c4bb..d152210 100644
--- a/daemon/zero.c
+++ b/daemon/zero.c
@@ -172,7 +172,7 @@ do_zero_device (const char *device)
     if (!is_zero (buf, sizeof buf)) {
       r = pwrite (fd, zero_buf, n, pos);
       if (r == -1) {
-        reply_with_perror ("pwrite: %s (with %" PRId64 " bytes left to write)",
+        reply_with_perror ("pwrite: %s (with %" PRIu64 " bytes left to write)",
                            device, size);
         close (fd);
         return -1;
diff --git a/diff/diff.c b/diff/diff.c
index 13ecca1..1261439 100644
--- a/diff/diff.c
+++ b/diff/diff.c
@@ -994,7 +994,7 @@ output_binary (const char *s, size_t len)
             exit (EXIT_FAILURE);
           }
         } else {
-          if (printf ("\\x%2x", s[i]) < 0) {
+          if (printf ("\\x%2x", (unsigned) s[i]) < 0) {
             perror ("printf");
             exit (EXIT_FAILURE);
           }
@@ -1054,7 +1054,7 @@ output_int64_perms (int64_t i)
 {
   next_field ();
   /* csv doesn't need escaping */
-  if (printf ("%04" PRIo64, i) < 0) {
+  if (printf ("%04" PRIo64, (uint64_t) i) < 0) {
     perror ("printf");
     exit (EXIT_FAILURE);
   }
@@ -1126,7 +1126,8 @@ output_int64_dev (int64_t i)
   next_field ();
 
   /* csv doesn't need escaping */
-  if (printf ("%d:%d", major (dev), minor (dev)) < 0) {
+  if (printf ("%ju:%ju",
+              (uintmax_t) major (dev), (uintmax_t) minor (dev)) < 0) {
     perror ("printf");
     exit (EXIT_FAILURE);
   }
diff --git a/fish/alloc.c b/fish/alloc.c
index b40284a..9f41915 100644
--- a/fish/alloc.c
+++ b/fish/alloc.c
@@ -97,7 +97,7 @@ parse_size (const char *str, off_t *size_rtn)
   xerr = xstrtoull (str, NULL, 0, &size, "0kKMGTPEZY");
   if (xerr != LONGINT_OK) {
     fprintf (stderr,
-             _("%s: invalid integer parameter (%s returned %d)\n"),
+             _("%s: invalid integer parameter (%s returned %u)\n"),
              "parse_size", "xstrtoull", xerr);
     return -1;
   }
diff --git a/fish/rc.c b/fish/rc.c
index 9ccd5f3..9b0c9c5 100644
--- a/fish/rc.c
+++ b/fish/rc.c
@@ -40,8 +40,8 @@
 /* Because this is a Unix domain socket, the total path length must be
  * under 108 bytes.
  */
-#define SOCKET_DIR "/tmp/.guestfish-%d" /* euid */
-#define SOCKET_PATH "/tmp/.guestfish-%d/socket-%d" /* euid, pid */
+#define SOCKET_DIR "/tmp/.guestfish-%ju" /* euid */
+#define SOCKET_PATH "/tmp/.guestfish-%ju/socket-%ju" /* euid, pid */
 
 static void
 create_sockdir (void)
@@ -52,7 +52,7 @@ create_sockdir (void)
   struct stat statbuf;
 
   /* Create the directory, and ensure it is owned by the user. */
-  snprintf (dir, sizeof dir, SOCKET_DIR, euid);
+  snprintf (dir, sizeof dir, SOCKET_DIR, (uintmax_t) euid);
   r = mkdir (dir, 0700);
   if (r == -1 && errno != EEXIST) {
   error:
@@ -79,7 +79,7 @@ create_sockpath (pid_t pid, char *sockpath, size_t len,
 
   create_sockdir ();
 
-  snprintf (sockpath, len, SOCKET_PATH, euid, pid);
+  snprintf (sockpath, len, SOCKET_PATH, (uintmax_t) euid, (uintmax_t) pid);
 
   addr->sun_family = AF_UNIX;
   strcpy (addr->sun_path, sockpath);
diff --git a/generator/bindtests.ml b/generator/bindtests.ml
index 9558a74..5358ff1 100644
--- a/generator/bindtests.ml
+++ b/generator/bindtests.ml
@@ -159,7 +159,7 @@ fill_lvm_pv (guestfs_h *g, struct guestfs_lvm_pv *pv, size_t i)
           pr "  {\n";
           pr "    size_t i;\n";
           pr "    for (i = 0; i < %s_size; ++i)\n" n;
-          pr "      fprintf (fp, \"<%%02x>\", %s[i]);\n" n;
+          pr "      fprintf (fp, \"<%%02x>\", (unsigned) %s[i]);\n" n;
           pr "    fprintf (fp, \"\\n\");\n";
           pr "  }\n";
         | OptString n -> pr "  fprintf (fp, \"%%s\\n\", %s ? %s : \"null\");\n" n n
diff --git a/generator/daemon.ml b/generator/daemon.ml
index 0c98d14..1825de4 100644
--- a/generator/daemon.ml
+++ b/generator/daemon.ml
@@ -564,12 +564,12 @@ cleanup_free_mountable (mountable_t *mountable)
                  pr "      r->%s[i++] = tok[j];\n" name;
                  pr "  }\n";
              | FBytes ->
-                 pr "  if (sscanf (tok, \"%%\"SCNu64, &r->%s) != 1) {\n" name;
+                 pr "  if (sscanf (tok, \"%%\" SCNi64, &r->%s) != 1) {\n" name;
                  pr "    fprintf (stderr, \"%%s: failed to parse size '%%s' from token %%s\\n\", __func__, tok, \"%s\");\n" name;
                  pr "    return -1;\n";
                  pr "  }\n";
              | FInt64 ->
-                 pr "  if (sscanf (tok, \"%%\"SCNi64, &r->%s) != 1) {\n" name;
+                 pr "  if (sscanf (tok, \"%%\" SCNi64, &r->%s) != 1) {\n" name;
                  pr "    fprintf (stderr, \"%%s: failed to parse int '%%s' from token %%s\\n\", __func__, tok, \"%s\");\n" name;
                  pr "    return -1;\n";
                  pr "  }\n";
diff --git a/generator/fish.ml b/generator/fish.ml
index 05bbdad..6f68e65 100644
--- a/generator/fish.ml
+++ b/generator/fish.ml
@@ -249,10 +249,10 @@ Guestfish will prompt for these separately."
     pr "print_%s_list (struct guestfs_%s_list *%ss)\n"
       typ typ typ;
     pr "{\n";
-    pr "  unsigned int i;\n";
+    pr "  size_t i;\n";
     pr "\n";
     pr "  for (i = 0; i < %ss->len; ++i) {\n" typ;
-    pr "    printf (\"[%%d] = {\\n\", i);\n";
+    pr "    printf (\"[%%zu] = {\\n\", i);\n";
     pr "    print_%s_indent (&%ss->val[i], \"  \");\n" typ typ;
     pr "    printf (\"}\\n\");\n";
     pr "  }\n";
@@ -270,7 +270,7 @@ Guestfish will prompt for these separately."
       pr "print_%s_indent (struct guestfs_%s *%s, const char *indent)\n" typ typ typ;
       pr "{\n";
       if needs_i then (
-        pr "  unsigned int i;\n";
+        pr "  size_t i;\n";
         pr "\n"
       );
       List.iter (
@@ -288,7 +288,8 @@ Guestfish will prompt for these separately."
             pr "    if (c_isprint (%s->%s[i]))\n" typ name;
             pr "      printf (\"%%c\", %s->%s[i]);\n" typ name;
             pr "    else\n";
-            pr "      printf (\"\\\\x%%02x\", %s->%s[i]);\n" typ name;
+            pr "      printf (\"\\\\x%%02x\", (unsigned) %s->%s[i]);\n"
+               typ name;
             pr "  printf (\"\\n\");\n"
         | name, (FUInt64|FBytes) ->
             pr "  printf (\"%%s%s: %%\" PRIu64 \"\\n\", indent, %s->%s);\n"
@@ -422,7 +423,7 @@ Guestfish will prompt for these separately."
           indent fn expr;
         pr "%s  if (xerr != LONGINT_OK) {\n" indent;
         pr "%s    fprintf (stderr,\n" indent;
-        pr "%s             _(\"%%s: %%s: invalid integer parameter (%%s returned %%d)\\n\"),\n" indent;
+        pr "%s             _(\"%%s: %%s: invalid integer parameter (%%s returned %%u)\\n\"),\n" indent;
         pr "%s             cmd, \"%s\", \"%s\", xerr);\n" indent name fn;
         pr "%s    goto %s;\n" indent out;
         pr "%s  }\n" indent;
@@ -585,9 +586,9 @@ Guestfish will prompt for these separately."
             | None ->
                 pr "  printf (\"%%d\\n\", r);\n";
             | Some FishOutputOctal ->
-                pr "  printf (\"%%s%%o\\n\", r != 0 ? \"0\" : \"\", r);\n";
+                pr "  printf (\"%%s%%o\\n\", r != 0 ? \"0\" : \"\", (unsigned) r);\n";
             | Some FishOutputHexadecimal ->
-                pr "  printf (\"%%s%%x\\n\", r != 0 ? \"0x\" : \"\", r);\n"
+                pr "  printf (\"%%s%%x\\n\", r != 0 ? \"0x\" : \"\", (unsigned) r);\n"
            )
        | RInt64 _ ->
            pr "  if (r == -1) goto out;\n";
@@ -598,7 +599,7 @@ Guestfish will prompt for these separately."
             | Some FishOutputOctal ->
                 pr "  printf (\"%%s%%\" PRIo64 \"\\n\", r != 0 ? \"0\" : \"\", r);\n";
             | Some FishOutputHexadecimal ->
-                pr "  printf (\"%%s%%\" PRIx64 \"\\n\", r != 0 ? \"0x\" : \"\", r);\n"
+                pr "  printf (\"%%s%%\" PRIx64 \"\\n\", r != 0 ? \"0x\" : \"\", (uint64_t) r);\n"
            )
        | RBool _ ->
            pr "  if (r == -1) goto out;\n";
diff --git a/make-fs/make-fs.c b/make-fs/make-fs.c
index cbf49d9..980bfeb 100644
--- a/make-fs/make-fs.c
+++ b/make-fs/make-fs.c
@@ -620,7 +620,7 @@ parse_size (const char *str, uint64_t estimate, uint64_t *size_rtn)
   xerr = xstrtoull (str, NULL, 0, &size, "0kKMGTPEZY");
   if (xerr != LONGINT_OK) {
     fprintf (stderr,
-             _("%s: %s: invalid size parameter '%s' (%s returned %d)\n"),
+             _("%s: %s: invalid size parameter '%s' (%s returned %u)\n"),
              guestfs_int_program_name, "parse_size", str, "xstrtoull", xerr);
     return -1;
   }
diff --git a/p2v/main.c b/p2v/main.c
index 666faf1..be32e4b 100644
--- a/p2v/main.c
+++ b/p2v/main.c
@@ -312,8 +312,9 @@ partition_parent (dev_t part_dev)
   size_t len = 0;
   unsigned parent_major, parent_minor;
 
-  if (asprintf (&path, "/sys/dev/block/%d:%d/../dev",
-                major (part_dev), minor (part_dev)) == -1) {
+  if (asprintf (&path, "/sys/dev/block/%ju:%ju/../dev",
+                (uintmax_t) major (part_dev),
+                (uintmax_t) minor (part_dev)) == -1) {
     perror ("asprintf");
     exit (EXIT_FAILURE);
   }
diff --git a/src/actions-support.c b/src/actions-support.c
index e671ed8..23b9ba7 100644
--- a/src/actions-support.c
+++ b/src/actions-support.c
@@ -38,25 +38,25 @@ guestfs_int_check_reply_header (guestfs_h *g,
                               unsigned int proc_nr, unsigned int serial)
 {
   if (hdr->prog != GUESTFS_PROGRAM) {
-    error (g, "wrong program (%d/%d)", hdr->prog, GUESTFS_PROGRAM);
+    error (g, "wrong program (%u/%d)", hdr->prog, GUESTFS_PROGRAM);
     return -1;
   }
   if (hdr->vers != GUESTFS_PROTOCOL_VERSION) {
-    error (g, "wrong protocol version (%d/%d)",
+    error (g, "wrong protocol version (%u/%d)",
            hdr->vers, GUESTFS_PROTOCOL_VERSION);
     return -1;
   }
   if (hdr->direction != GUESTFS_DIRECTION_REPLY) {
     error (g, "unexpected message direction (%d/%d)",
-           hdr->direction, GUESTFS_DIRECTION_REPLY);
+           (int) hdr->direction, GUESTFS_DIRECTION_REPLY);
     return -1;
   }
   if (hdr->proc != proc_nr) {
-    error (g, "unexpected procedure number (%d/%d)", hdr->proc, proc_nr);
+    error (g, "unexpected procedure number (%d/%u)", (int) hdr->proc, proc_nr);
     return -1;
   }
   if (hdr->serial != serial) {
-    error (g, "unexpected serial (%d/%d)", hdr->serial, serial);
+    error (g, "unexpected serial (%u/%u)", hdr->serial, serial);
     return -1;
   }
 
diff --git a/src/appliance.c b/src/appliance.c
index 2167ac3..2645cca 100644
--- a/src/appliance.c
+++ b/src/appliance.c
@@ -229,7 +229,7 @@ build_supermin_appliance (guestfs_h *g,
    */
   len = strlen (tmpdir) + 128;
   char cachedir[len];
-  snprintf (cachedir, len, "%s/.guestfs-%d", tmpdir, uid);
+  snprintf (cachedir, len, "%s/.guestfs-%ju", tmpdir, (uintmax_t) uid);
   char lockfile[len];
   snprintf (lockfile, len, "%s/lock", cachedir);
   char appliancedir[len];
@@ -244,8 +244,8 @@ build_supermin_appliance (guestfs_h *g,
   if (lstat (cachedir, &statbuf) == -1)
     return 0;
   if (statbuf.st_uid != uid) {
-    error (g, _("security: cached appliance %s is not owned by UID %d"),
-           cachedir, uid);
+    error (g, _("security: cached appliance %s is not owned by UID %ju"),
+           cachedir, (uintmax_t) uid);
     return -1;
   }
   if (!S_ISDIR (statbuf.st_mode)) {
diff --git a/src/fuse.c b/src/fuse.c
index 3fdb1d4..332c1be 100644
--- a/src/fuse.c
+++ b/src/fuse.c
@@ -345,8 +345,8 @@ mount_local_access (const char *path, int mask)
 
   debug (g, "%s: "
          "testing access mask%s%s%s%s: "
-         "caller UID:GID = %d:%d, "
-         "file UID:GID = %d:%d, "
+         "caller UID:GID = %ju:%ju, "
+         "file UID:GID = %ju:%ju, "
          "file mode = %o, "
          "result = %s",
          path,
@@ -354,8 +354,8 @@ mount_local_access (const char *path, int mask)
          mask & W_OK ? " W_OK" : "",
          mask & X_OK ? " X_OK" : "",
          mask == 0 ? " 0" : "",
-         fuse->uid, fuse->gid,
-         statbuf.st_uid, statbuf.st_gid,
+         (uintmax_t) fuse->uid, (uintmax_t) fuse->gid,
+         (uintmax_t) statbuf.st_uid, (uintmax_t) statbuf.st_gid,
          statbuf.st_mode,
          ok ? "OK" : "EACCESS");
 
@@ -402,7 +402,7 @@ mount_local_mknod (const char *path, mode_t mode, dev_t rdev)
 {
   int r;
   DECL_G ();
-  DEBUG_CALL ("%s, 0%o, 0x%lx", path, mode, (long) rdev);
+  DEBUG_CALL ("%s, 0%o, 0x%jx", path, mode, (uintmax_t) rdev);
 
   if (g->ml_read_only) return -EROFS;
 
@@ -548,7 +548,7 @@ mount_local_chown (const char *path, uid_t uid, gid_t gid)
 {
   int r;
   DECL_G ();
-  DEBUG_CALL ("%s, %ld, %ld", path, (long) uid, (long) gid);
+  DEBUG_CALL ("%s, %ju, %ju", path, (uintmax_t) uid, (uintmax_t) gid);
 
   if (g->ml_read_only) return -EROFS;
 
@@ -630,7 +630,7 @@ mount_local_open (const char *path, struct fuse_file_info *fi)
 {
   int flags = fi->flags & O_ACCMODE;
   DECL_G ();
-  DEBUG_CALL ("%s, 0%o", path, fi->flags);
+  DEBUG_CALL ("%s, 0%o", path, (unsigned) fi->flags);
 
   if (g->ml_read_only && flags != O_RDONLY)
     return -EROFS;
diff --git a/src/handle.c b/src/handle.c
index 51b9572..12c1fb2 100644
--- a/src/handle.c
+++ b/src/handle.c
@@ -336,7 +336,7 @@ guestfs_close (guestfs_h *g)
                                       trace_msg, strlen (trace_msg));
   }
 
-  debug (g, "closing guestfs handle %p (state %d)", g, g->state);
+  debug (g, "closing guestfs handle %p (state %d)", g, (int) g->state);
 
   if (g->state != CONFIG)
     shutdown_backend (g, 0);
diff --git a/src/launch-libvirt.c b/src/launch-libvirt.c
index f46782c..1c0bfac 100644
--- a/src/launch-libvirt.c
+++ b/src/launch-libvirt.c
@@ -857,7 +857,8 @@ debug_appliance_permissions (guestfs_h *g)
   CLEANUP_FREE char *cachedir = guestfs_get_cachedir (g);
   CLEANUP_FREE char *appliance = NULL;
 
-  appliance = safe_asprintf (g, "%s/.guestfs-%d", cachedir, geteuid ());
+  appliance = safe_asprintf (g, "%s/.guestfs-%ju",
+                             cachedir, (uintmax_t) geteuid ());
 
   guestfs_int_cmd_add_arg (cmd, "ls");
   guestfs_int_cmd_add_arg (cmd, "-a");
diff --git a/src/launch.c b/src/launch.c
index fd5479e..343f4ea 100644
--- a/src/launch.c
+++ b/src/launch.c
@@ -86,7 +86,7 @@ guestfs_impl_launch (guestfs_h *g)
 
     debug (g, "launch: tmpdir=%s", g->tmpdir);
     debug (g, "launch: umask=0%03o", get_umask (g));
-    debug (g, "launch: euid=%d", geteuid ());
+    debug (g, "launch: euid=%ju", (uintmax_t) geteuid ());
   }
 
   /* Launch the appliance. */
diff --git a/src/proto.c b/src/proto.c
index 4ddd164..815a4d2 100644
--- a/src/proto.c
+++ b/src/proto.c
@@ -527,7 +527,7 @@ recv_from_daemon (guestfs_h *g, uint32_t *size_rtn, void **buf_rtn)
   if (*size_rtn == GUESTFS_LAUNCH_FLAG) {
     if (g->state != LAUNCHING)
       error (g, _("received magic signature from guestfsd, but in state %d"),
-             g->state);
+             (int) g->state);
     else {
       g->state = READY;
       guestfs_int_call_callbacks_void (g, GUESTFS_EVENT_LAUNCH_DONE);
diff --git a/test-tool/test-tool.c b/test-tool/test-tool.c
index f41b8fd..def21e2 100644
--- a/test-tool/test-tool.c
+++ b/test-tool/test-tool.c
@@ -151,7 +151,7 @@ main (int argc, char *argv[])
 
     default:
       fprintf (stderr,
-               _("libguestfs-test-tool: unexpected command line option 0x%x\n"),
+               _("libguestfs-test-tool: unexpected command line option 0x%d\n"),
                c);
       exit (EXIT_FAILURE);
     }
-- 
2.3.1


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]