Resolve missing printf args 55/13155/5
authorPat Riehecky <riehecky@fnal.gov>
Fri, 1 Jun 2018 20:32:57 +0000 (15:32 -0500)
committerBenjamin Kaduk <kaduk@mit.edu>
Fri, 26 Feb 2021 16:36:24 +0000 (11:36 -0500)
A handful of printf's requested more args than they were given.  The
missing args are now provided. (via cppcheck)

Change-Id: I3d2bfd1b68a3518ee4c8a65f02446a2bae85d926
Reviewed-on: https://gerrit.openafs.org/13155
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

src/ptserver/ptutils.c
src/viced/viced.c
src/vol/gi.c
src/vol/xfs_size_check.c

index c551c62..dfa54ee 100644 (file)
@@ -1559,7 +1559,7 @@ GetListSG2(struct ubik_trans *at, afs_int32 gid, prlist *alist, afs_int32 *sizeP
            didsomething ? "TRUE" : "FALSE");
     if (predictflagged && didsomething != predictfound)
        fprintf(stderr, "**** for gid=%d, didsomething=%d predictfound=%d\n",
-               didsomething, predictfound);
+               gid, didsomething, predictfound);
 #endif
     if (didsomething)
        sg_found = add_map(sg_found, -gid);
index b34436a..17cf1f8 100644 (file)
@@ -1218,7 +1218,7 @@ ParseArgs(int argc, char *argv[])
 #if defined(AFS_AIX32_ENV)
     if (cmd_OptionAsInt(opts, OPT_minspare, &aixlow_water) == 0) {
        if ((aixlow_water < 0) || (aixlow_water > 30)) {
-           printf("space reserved %d%% invalid; must be between 0-30%\n",
+           printf("space reserved %d%% invalid; must be between 0-30%%\n",
                   aixlow_water);
            return -1;
        }
index e4e7f7c..e193195 100644 (file)
@@ -73,7 +73,7 @@ main(int argc, char **argv)
        printf
            ("Inode status: dev=%d, ino=%d, mode=%o, nlink=%d, uid=%d, gid=%d, size=%d, mtime=%d, blocks=%d\n",
             status.st_dev, status.st_ino, status.st_mode, status.st_nlink,
-            status.st_uid, status.st_gid, status.st_size, status.st_mtime);
+            status.st_uid, status.st_gid, status.st_size, status.st_mtime, status.st_blocks);
     } else {
        /* Send the inode to standard out */
        char buf[4096];
index 5ee492b..a24364c 100644 (file)
@@ -114,7 +114,7 @@ CheckPartitions()
                break;
            case VERIFY_ERROR:
                printf("%s: Can't check XFS inode size: %s\n",
-                      strerror(errno));
+                      prog, strerror(errno));
                break;
            case VERIFY_FIX:
                if (nAvail <= nParts) {
@@ -172,7 +172,7 @@ main(int ac, char **av)
 #else /* AFS_SGI_XFS_IOPS_ENV */
 main()
 {
-    printf("%s only runs on XFS platforms.\n, prog");
+    printf("%s only runs on XFS platforms.\n", prog);
     exit(1);
 }
 #endif /* AFS_SGI_XFS_IOPS_ENV */