[Pkg-libvirt-commits] [libguestfs] 19/26: Fix various -Wformat problems.

Hilko Bengen bengen at moszumanska.debian.org
Sun Nov 1 17:11:29 UTC 2015


This is an automated email from the git hooks/post-receive script.

bengen pushed a commit to annotated tag upstream/1.28.12
in repository libguestfs.

commit 39ecb900978f941a281f506b082684878bf7a0f1
Author: Richard W.M. Jones <rjones at redhat.com>
Date:   Thu Jul 2 15:36:04 2015 +0100

    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 them 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.
    
    (cherry picked from commit ccdbbc7fe43e313c16a00dd30c57a04134e60b2c)
---
 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 28a021b..e2fe0c8 100644
--- a/builder/pxzcat-c.c
+++ b/builder/pxzcat-c.c
@@ -292,7 +292,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");
     }
 
@@ -311,7 +311,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");
     }
 
@@ -333,7 +333,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");
     }
 
@@ -350,14 +350,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");
     }
@@ -365,7 +365,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");
     }
 
@@ -374,14 +374,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");
       }
     }
@@ -579,7 +579,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;
     }
@@ -590,7 +590,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;
     }
@@ -601,7 +601,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;
     }
 
@@ -651,7 +651,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 19b494e..6fb17fd 100644
--- a/cat/filesystems.c
+++ b/cat/filesystems.c
@@ -865,7 +865,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 8d7dfc9..25f4245 100644
--- a/cat/ls.c
+++ b/cat/ls.c
@@ -699,7 +699,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);
   }
@@ -771,7 +771,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 a576983..0996736 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 cdf1661..171e0af 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 181754d..dc8f19d 100644
--- a/daemon/guestfsd.c
+++ b/daemon/guestfsd.c
@@ -200,7 +200,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 f73e3e9..eedb995 100644
--- a/daemon/parted.c
+++ b/daemon/parted.c
@@ -744,7 +744,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;
@@ -767,7 +767,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 c9de36d..b15bd43 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 ae8e84d..bb88f38 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 22e94b0..a440349 100644
--- a/diff/diff.c
+++ b/diff/diff.c
@@ -991,7 +991,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);
           }
@@ -1051,7 +1051,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);
   }
@@ -1123,7 +1123,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 06dd8cf..6c45662 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 8e22c53..c770f26 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 cad123e..13316ec 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 06d49ef..9a43036 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 fce7c92..2b22f13 100644
--- a/generator/fish.ml
+++ b/generator/fish.ml
@@ -243,10 +243,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";
@@ -264,7 +264,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 (
@@ -282,7 +282,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"
@@ -432,7 +433,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;
@@ -595,9 +596,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";
@@ -608,7 +609,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 22658fb..392bc80 100644
--- a/make-fs/make-fs.c
+++ b/make-fs/make-fs.c
@@ -616,7 +616,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___program_name, "parse_size", str, "xstrtoull", xerr);
     return -1;
   }
diff --git a/p2v/main.c b/p2v/main.c
index 75fd8d4..5f413c5 100644
--- a/p2v/main.c
+++ b/p2v/main.c
@@ -292,8 +292,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 c3bd863..9c20705 100644
--- a/src/actions-support.c
+++ b/src/actions-support.c
@@ -38,25 +38,25 @@ guestfs___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 5fa47f2..7010002 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 258a863..69aa744 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 b509406..dec78ce 100644
--- a/src/handle.c
+++ b/src/handle.c
@@ -324,7 +324,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 6041a28..365d1a6 100644
--- a/src/launch-libvirt.c
+++ b/src/launch-libvirt.c
@@ -845,7 +845,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___cmd_add_arg (cmd, "ls");
   guestfs___cmd_add_arg (cmd, "-a");
diff --git a/src/launch.c b/src/launch.c
index 78d26fe..9f1ea72 100644
--- a/src/launch.c
+++ b/src/launch.c
@@ -86,7 +86,7 @@ guestfs__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 53d1d6b..1d0919d 100644
--- a/src/proto.c
+++ b/src/proto.c
@@ -528,7 +528,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___call_callbacks_void (g, GUESTFS_EVENT_LAUNCH_DONE);
diff --git a/test-tool/test-tool.c b/test-tool/test-tool.c
index 2c72185..f64679d 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 %d\n"),
                c);
       exit (EXIT_FAILURE);
     }

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-libvirt/libguestfs.git



More information about the Pkg-libvirt-commits mailing list