xfstests: eliminate warnings under dmapi/src/suite1/cmd (3)
authorAlex Elder <aelder@sgi.com>
Fri, 4 Mar 2011 19:38:12 +0000 (19:38 +0000)
committerAlex Elder <aelder@sgi.com>
Thu, 28 Apr 2011 17:36:33 +0000 (12:36 -0500)
Eliminate build warnings reported on files located under
dmapi/src/suite1/cmd/ (third of three).

Signed-off-by: Alex Elder <aelder@sgi.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
21 files changed:
dmapi/src/suite1/cmd/print_event.c
dmapi/src/suite1/cmd/probe_hole.c
dmapi/src/suite1/cmd/probe_punch_xfsctl_hole.c
dmapi/src/suite1/cmd/punch_hole.c
dmapi/src/suite1/cmd/query_right.c
dmapi/src/suite1/cmd/randomize_file.c
dmapi/src/suite1/cmd/release_right.c
dmapi/src/suite1/cmd/remove_dmattr.c
dmapi/src/suite1/cmd/request_right.c
dmapi/src/suite1/cmd/respond_event.c
dmapi/src/suite1/cmd/rwt.c
dmapi/src/suite1/cmd/security_hole.c
dmapi/src/suite1/cmd/security_hole2.c
dmapi/src/suite1/cmd/set_disp.c
dmapi/src/suite1/cmd/set_dmattr.c
dmapi/src/suite1/cmd/set_eventlist.c
dmapi/src/suite1/cmd/set_fileattr.c
dmapi/src/suite1/cmd/struct_test.c
dmapi/src/suite1/cmd/sync_by_handle.c
dmapi/src/suite1/cmd/test_assumption.c
dmapi/src/suite1/cmd/upgrade_right.c

index e34cafe5f925d0c935f984bd723bf43e122e8df2..afe2c27aa2e2a3480c28c2289c6ba749e0375f67 100644 (file)
@@ -331,31 +331,31 @@ print_one_mount_event(
        if (hanp1 && hlen1) {
                hantoa(hanp1, hlen1, hans1);
        } else {
-               sprintf(hans1, "<BAD HANDLE, hlen %d>", hlen1);
+               sprintf(hans1, "<BAD HANDLE, hlen %zd>", hlen1);
        }
        if (hanp2 && hlen2) {
                hantoa(hanp2, hlen2, hans2);
        } else {
-               sprintf(hans2, "<BAD HANDLE, hlen %d>", hlen2);
+               sprintf(hans2, "<BAD HANDLE, hlen %zd>", hlen2);
        }
        if (hanp3 && hlen3) {
                hantoa(hanp3, hlen3, hans3);
        } else {
-               sprintf(hans3, "<BAD HANDLE, hlen %d>", hlen3);
+               sprintf(hans3, "<BAD HANDLE, hlen %zd>", hlen3);
        }
        if (namp1 && nlen1) {
                strncpy(nams1, namp1, nlen1);
                if (nlen1 != sizeof(nams1))
                        nams1[nlen1] = '\0';
        } else {
-               sprintf(nams1, "<BAD STRING, nlen %d>", nlen1);
+               sprintf(nams1, "<BAD STRING, nlen %zd>", nlen1);
        }
        if (namp2 && nlen2) {
                strncpy(nams2, namp2, nlen2);
                if (nlen2 != sizeof(nams2))
                        nams2[nlen2] = '\0';
        } else {
-               sprintf(nams2, "<BAD STRING, nlen %d>", nlen2);
+               sprintf(nams2, "<BAD STRING, nlen %zd>", nlen2);
        }
 
        printf(VALS VALS VALS VALS VALS VALD,
@@ -475,6 +475,10 @@ handle_message(
                case DM_EVENT_TRUNCATE:
                        rgflag = DM_REGION_TRUNCATE;
                        break;
+               default:
+                       err_msg("unexpected event type (%d)\n", msg->ev_type);
+                       rgflag = DM_REGION_NOEVENT;
+                       break;
              }
              clear_region_event(sid, rgflag, hanp1, hlen1, hans1);
       }
@@ -488,24 +492,24 @@ handle_message(
       printf(HDR VALS VALLLD VALLLD,
             "read", msg->ev_token, msg->ev_sequence,
             "file handle",     hans1,
-            "offset",          msg_de->de_offset,
-            "length",          msg_de->de_length);
+            "offset",          (long long) msg_de->de_offset,
+            "length",          (long long) msg_de->de_length);
       break;
 
     case DM_EVENT_WRITE:
       printf(HDR VALS VALLLD VALLLD,
             "write", msg->ev_token, msg->ev_sequence,
             "file handle",     hans1,
-            "offset",          msg_de->de_offset,
-            "length",          msg_de->de_length);
+            "offset",          (long long) msg_de->de_offset,
+            "length",          (long long) msg_de->de_length);
       break;
 
     case DM_EVENT_TRUNCATE:
       printf(HDR VALS VALLLD VALLLD,
             "truncate", msg->ev_token, msg->ev_sequence,
             "file handle",     hans1,
-            "offset",          msg_de->de_offset,
-            "length",          msg_de->de_length);
+            "offset",          (long long) msg_de->de_offset,
+            "length",          (long long) msg_de->de_length);
       break;
     default: break;
     }
@@ -578,7 +582,7 @@ handle_message(
   /***** NAMESPACE EVENTS *****/
 
   else {
-    char       *type;
+    char       *type = NULL;
 
     msg_ne = DM_GET_VALUE(msg, ev_data, dm_namesp_event_t *);
     hanp1  = DM_GET_VALUE(msg_ne, ne_handle1, void *);
@@ -1150,7 +1154,7 @@ finish_responding(
   int          error = 0;
   u_int                nbytes, ntokens = 0, ret_ntokens, i;
   dm_token_t   *tokenbuf = NULL, *tokenptr;
-  size_t       buflen, ret_buflen;
+  size_t       buflen = 0, ret_buflen;
   char         *msgbuf = NULL;
   dm_eventmsg_t        *msg;
 
index e0e4a5b267bdc34ce7b3462b91f6255ee11afc11..c5f945a83191772f04adc1583785c9658721f321 100644 (file)
@@ -70,7 +70,8 @@ main(
        char            *name;
        int             opt;
 
-       if (Progname = strrchr(argv[0], '/')) {
+       Progname = strrchr(argv[0], '/');
+       if (Progname) {
                Progname++;
        } else {
                Progname = argv[0];
@@ -117,7 +118,8 @@ main(
                        strerror(errno));
                exit(1);
        }
-       fprintf(stdout, "roffp is %lld, rlenp is %lld\n", roffp, rlenp);
+       fprintf(stdout, "roffp is %lld, rlenp is %llu\n",
+               (long long) roffp, (unsigned long long) rlenp);
        dm_handle_free(hanp, hlen);
        exit(0);
 }
index effa98a45a2fe8a74593c0eba8c31aa8b1af4ee2..6b4d0b6cb5efa1b66025205474221206b7824195 100644 (file)
@@ -108,7 +108,8 @@ main(
        int             opt;
        int             method = METHOD_DMAPI_PROBE;
 
-       if (Progname = strrchr(argv[0], '/')) {
+       Progname = strrchr(argv[0], '/');
+       if (Progname) {
                Progname++;
        } else {
                Progname = argv[0];
@@ -149,7 +150,8 @@ main(
                usage();
 
        printf("Running %s on %s with settings:\n", methodmap[method], pathname);
-       printf("  offset = '%lld', length = '%lld', sid = '%lld'\n", offset, length, sid);
+       printf("  offset = '%lld', length = '%lld', sid = '%lld'\n",
+               (long long) offset, (unsigned long long) length, (long long) sid);
        
        if (method ==  METHOD_XFSCTL) 
                return xfsctl_punch_hole(pathname, offset, length);
@@ -176,7 +178,8 @@ main(
                                strerror(errno));
                        exit(1);
                }
-               fprintf(stdout, "roffp is %lld, rlenp is %lld\n", roffp, rlenp);
+               fprintf(stdout, "roffp is %lld, rlenp is %llu\n",
+                       (long long) roffp, (unsigned long long) rlenp);
                break;
        case METHOD_DMAPI_PUNCH:
                if (dm_punch_hole(sid, hanp, hlen, DM_NO_TOKEN, offset, length)) {
index e1d1a6762efbddfabc257d9b0a5a0af15991ada0..68439a755955c3c465452de7ad283da9099e95c0 100644 (file)
@@ -68,7 +68,8 @@ main(
        char            *name;
        int             opt;
 
-       if (Progname = strrchr(argv[0], '/')) {
+       Progname = strrchr(argv[0], '/');
+       if (Progname) {
                Progname++;
        } else {
                Progname = argv[0];
index a8e219adc2a7561cdd403f59f8add85c95f42f87..aab4de5b9ed833a5595570f7d733d028ec60e365 100644 (file)
@@ -77,7 +77,8 @@ main(
        char            *name;
        int             opt;
 
-       if (Progname = strrchr(argv[0], '/')) {
+       Progname = strrchr(argv[0], '/');
+       if (Progname) {
                Progname++;
        } else {
                Progname = argv[0];
index a45b2a39bc4e3fc19bac2df12d018711a04900e9..0c272fe587f7c58017cfc20b065161363d2fa7ee 100644 (file)
@@ -43,7 +43,7 @@ main(
 {
        FILE    *infile;
        FILE    *tmpfile;
-       char    *path;
+       char    path[] = "file_XXXXXX";
        int     line_count = 0;
        int     i;
        int     j;
@@ -77,7 +77,7 @@ main(
           to each line copied.
        */
 
-       path = tmpnam(NULL);
+       (void) mkstemp(path);
        if ((tmpfile = fopen(path, "w")) == NULL) {
                fprintf(stderr, "error opening temp file %s\n", path);
                exit(1);
index 648619c5e4870387f707e018f3581943f29a737b..e6a92d37a23f7331dbb541c260dcba02f66880c1 100644 (file)
@@ -76,7 +76,8 @@ main(
        char            *name;
        int             opt;
 
-       if (Progname = strrchr(argv[0], '/')) {
+       Progname = strrchr(argv[0], '/');
+       if (Progname) {
                Progname++;
        } else {
                Progname = argv[0];
index 3a82b5115c4e66b16cfaa011aaf5012c9e82de6d..865898d489beb3ba0e3dc189310bf7bee14c681f 100644 (file)
@@ -65,7 +65,8 @@ main(
        char            *name;
        int             opt;
 
-       if (Progname = strrchr(argv[0], '/')) {
+       Progname = strrchr(argv[0], '/');
+       if (Progname) {
                Progname++;
        } else {
                Progname = argv[0];
index 11ff944d4a0e58d8c3cbf57936a9f3ebca134dfa..fff3e78b68e019d8980e00f3319ec72ee79de0b5 100644 (file)
@@ -91,7 +91,8 @@ main(
        char            *name;
        int             opt;
 
-       if (Progname = strrchr(argv[0], '/')) {
+       Progname = strrchr(argv[0], '/');
+       if (Progname) {
                Progname++;
        } else {
                Progname = argv[0];
index 9fd297c631eeafb4e702e35658aaf556e0514d28..bf1bb94883b2fe7afd2b3b9848566e7a77b4ba69 100644 (file)
@@ -65,7 +65,8 @@ main(
        dm_response_t   response;
        int             reterror;
 
-       if (Progname = strrchr(argv[0], '/')) {
+       Progname = strrchr(argv[0], '/');
+       if (Progname) {
                Progname++;
        } else {
                Progname = argv[0];
index b5bd9037ff43b66c03997d3901d05710cdb2f828..c1489bdd4da2ef008486037a3aa3b5456f3d3bb4 100644 (file)
@@ -89,7 +89,8 @@ main(
        ssize_t         rc;
        int             opt;
 
-       if (Progname = strrchr(argv[0], '/')) {
+       Progname = strrchr(argv[0], '/');
+       if (Progname) {
                Progname++;
        } else {
                Progname = argv[0];
@@ -131,7 +132,7 @@ main(
        }
        if (length > 0) {
                if ((bufp = malloc(length)) == NULL) {
-                       fprintf(stderr, "malloc of %d bytes failed\n", length);
+                       fprintf(stderr, "malloc of %zd bytes failed\n", length);
                        exit(1);
                }
                if (wflag)
@@ -147,7 +148,7 @@ main(
                        fprintf(stderr,
                                "seeked to offset %lld, actually "
                                "arrived at %lld\n",
-                               (int64_t)offset, (int64_t)seek_off);
+                               (long long) offset, (long long) seek_off);
                        exit(1);
                }
        }
@@ -158,8 +159,8 @@ main(
                        exit(1);
                }
                if (rc != length) {
-                       fprintf(stderr, "expected to write %d bytes, actually "
-                               "wrote %d bytes\n", length, rc);
+                       fprintf(stderr, "expected to write %zd bytes, actually "
+                               "wrote %zd bytes\n", length, rc);
                        exit(1);
                }
        } else if (tflag) {
@@ -174,8 +175,8 @@ main(
                        exit(1);
                }
                if (rc != length) {
-                       fprintf(stderr, "expected to read %d bytes, actually "
-                               "read %d bytes\n", length, rc);
+                       fprintf(stderr, "expected to read %zd bytes, actually "
+                               "read %zd bytes\n", length, rc);
                        exit(1);
                }
        }
index c7c3db35b456e6c34ea5f81f6252983f22fdc2f6..d34026da7d4b8f15d9688f7ebc78ba3e81c86a36 100644 (file)
@@ -63,7 +63,8 @@ main(
        int     opt;
        int     fd;
 
-       if (Progname = strrchr(argv[0], '/')) {
+       Progname = strrchr(argv[0], '/');
+       if (Progname) {
                Progname++;
        } else {
                Progname = argv[0];
index c40801cff1189335f54ee2b5f8b83594ce6b5069..3af3395a6b2b77396cb5192ca59e1dddf0febc1d 100644 (file)
@@ -59,7 +59,8 @@ main(
        int     opt;
        int     fd;
 
-       if (Progname = strrchr(argv[0], '/')) {
+       Progname = strrchr(argv[0], '/');
+       if (Progname) {
                Progname++;
        } else {
                Progname = argv[0];
index b04306d3edf1d6f3464f545cc2568d0d13bcc426..fee1dcedeb52786ae000bbb30996e311d4c8f765 100644 (file)
@@ -89,7 +89,8 @@ main(
        char            *name;
        int             opt;
 
-       if (Progname = strrchr(argv[0], '/')) {
+       Progname = strrchr(argv[0], '/');
+       if (Progname) {
                Progname++;
        } else {
                Progname = argv[0];
index df4f0800fae1daff578de02640e4d7e0ee173f7d..4753cf445c0de94d33c706a91dd4fae115e19c8e 100644 (file)
@@ -63,7 +63,7 @@ main(
        char            *object;
        dm_attrname_t   *attrnamep;
        char            *bufp;
-       size_t          buflen;
+       size_t          buflen = 0;
        int             bflag = 0;
        int             setdtime = 0;
        void            *hanp;
@@ -71,7 +71,8 @@ main(
        char            *name;
        int             opt;
 
-       if (Progname = strrchr(argv[0], '/')) {
+       Progname = strrchr(argv[0], '/');
+       if (Progname) {
                Progname++;
        } else {
                Progname = argv[0];
index 44d62b3510e9cad7baef6c721713079bf1ecfe7a..857523d33dc267c809ca6d43deb918199260a344 100644 (file)
@@ -85,7 +85,8 @@ main(
        char            *name;
        int             opt;
 
-       if (Progname = strrchr(argv[0], '/')) {
+       Progname = strrchr(argv[0], '/');
+       if (Progname) {
                Progname++;
        } else {
                Progname = argv[0];
index eabb8263f0c2f4454c79db41c8c7f825e82beb05..70ae52f18f0c06ed3c2bcb234382c26b0ffe4a51 100644 (file)
@@ -323,7 +323,8 @@ main(
        char            *name;
        int             opt;
 
-       if (Progname = strrchr(argv[0], '/')) {
+       Progname = strrchr(argv[0], '/');
+       if (Progname) {
                Progname++;
        } else {
                Progname = argv[0];
index 4d0b31f07793991e7e59f507af5dbc79c1107972..7a0dd3580738a95de0a41d2a6496896d3b71ec92 100644 (file)
@@ -51,7 +51,7 @@
        if (offsetof(struct_name, field_name) != offset) { \
                printf("ERROR: field %s should be %d\n", \
                        #struct_name "." #field_name, \
-                       offsetof(struct_name, field_name)); \
+                       (int) offsetof(struct_name, field_name)); \
        } \
        offset = offsetof(struct_name, field_name) + sizeof(X.field_name); \
 }
@@ -60,7 +60,7 @@
 { \
        printf("struct %s size is %d\n", s_name, offset); \
        if (sizeof(struct_name) != offset) { \
-               printf("ERROR: struct %s should be %d\n", \
+               printf("ERROR: struct %s should be %zd\n", \
                        s_name, sizeof(struct_name)); \
        } \
 }
index a00ee0fc178d2a39a3e630413ea298c9e089c474..ee2276731e72ec97e853163eca9486de88f3e246 100644 (file)
@@ -65,7 +65,8 @@ main(
        char            *name;
        int             opt;
 
-       if (Progname = strrchr(argv[0], '/')) {
+       Progname = strrchr(argv[0], '/');
+       if (Progname) {
                Progname++;
        } else {
                Progname = argv[0];
index 6760f34d769bd5281ef2d58116791e82b2393434..98dea4c15f4564204d20a73df8ff207719d89247 100644 (file)
@@ -88,7 +88,7 @@ main(
                 exit(1);
         }
        fprintf(stdout, "Initial session ID: %d\n", sid);
-       fprintf(stdout, "Initial session message length: '%d'\n", rlen);
+       fprintf(stdout, "Initial session message length: '%zd'\n", rlen);
        if (rlen > 0) {
                fprintf(stdout, "Initial session message: '%s'\n", buffer);
        }
@@ -108,7 +108,7 @@ main(
                 exit(1);
         }
        fprintf(stdout, "Assumed session ID: %d\n", newsid);
-       fprintf(stdout, "Assumed session message length: '%d'\n", rlen);
+       fprintf(stdout, "Assumed session message length: '%zd'\n", rlen);
        if (rlen > 0) {
                fprintf(stdout, "Assumed session message: '%s'\n", buffer);
        }
index 5c4a50e73d6a6bc8cad61154fe26bfcc4fbefd2a..d2493912259248b41034115bcfd3e3922f440b6a 100644 (file)
@@ -76,7 +76,8 @@ main(
        char            *name;
        int             opt;
 
-       if (Progname = strrchr(argv[0], '/')) {
+       Progname = strrchr(argv[0], '/');
+       if (Progname) {
                Progname++;
        } else {
                Progname = argv[0];