* [PATCH 0/5] xfstests: dmapi: eliminate all build warnings
@ 2011-04-21 19:11 Alex Elder
2011-04-21 19:11 ` [PATCH 1/5] xfstests: eliminate some warnings under dmapi/src Alex Elder
2011-04-28 2:15 ` [PATCH 0/5] xfstests: dmapi: eliminate all build warnings Dave Chinner
0 siblings, 2 replies; 7+ messages in thread
From: Alex Elder @ 2011-04-21 19:11 UTC (permalink / raw)
To: xfs
I'm hoping the Nth time is a charm...
This series eliminates all build warnings that come out of building
things under the dmapi/ subdirectory. I'm using gcc 4.x (x < 6) on
several platforms, including i386, x86_64, and ia64 architectures.
I have sent these a couple times before; they are unmodified from
the last time I sent them out. I am omitting the summary of the
types of warnings and how they got resolved this time around.
-Alex
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/5] xfstests: eliminate some warnings under dmapi/src
2011-04-21 19:11 [PATCH 0/5] xfstests: dmapi: eliminate all build warnings Alex Elder
@ 2011-04-21 19:11 ` Alex Elder
2011-04-21 19:11 ` [PATCH 2/5] xfstests: eliminate warnings under dmapi/src/suite1/cmd (1) Alex Elder
` (3 more replies)
2011-04-28 2:15 ` [PATCH 0/5] xfstests: dmapi: eliminate all build warnings Dave Chinner
1 sibling, 4 replies; 7+ messages in thread
From: Alex Elder @ 2011-04-21 19:11 UTC (permalink / raw)
To: xfs; +Cc: Alex Elder
Eliminate build warnings reported on files located under these
subdirectories:
dmapi/src/common/cmd/
dmapi/src/common/lib/
dmapi/src/sample_hsm/
dmapi/src/simple/
Signed-off-by: Alex Elder <aelder@sgi.com>
---
dmapi/src/common/cmd/read_invis.c | 21 +++++---
dmapi/src/common/cmd/set_region.c | 3 +-
dmapi/src/common/cmd/set_return_on_destroy.c | 3 +-
dmapi/src/common/cmd/write_invis.c | 19 +++++--
dmapi/src/common/lib/find_session.c | 8 ++--
dmapi/src/common/lib/print.c | 16 +++---
dmapi/src/common/lib/util.c | 68 ++++++++++++--------------
dmapi/src/sample_hsm/migfind.c | 6 +-
dmapi/src/sample_hsm/mls.c | 8 ++--
dmapi/src/simple/dm_find_eventmsg.c | 4 +-
dmapi/src/simple/dm_query_session.c | 2 +-
11 files changed, 85 insertions(+), 73 deletions(-)
diff --git a/dmapi/src/common/cmd/read_invis.c b/dmapi/src/common/cmd/read_invis.c
index afa12c9..53fd3d3 100644
--- a/dmapi/src/common/cmd/read_invis.c
+++ b/dmapi/src/common/cmd/read_invis.c
@@ -73,7 +73,9 @@ main(
dm_sessid_t sid = DM_NO_SESSION;
char *object = NULL;
dm_off_t offset = 0;
+ long long lltemp;
dm_size_t length = 1;
+ unsigned long long ulltemp;
char *bufp = NULL;
void *hanp;
size_t hlen;
@@ -82,10 +84,11 @@ main(
int opt;
int i;
char *storefile = NULL;
- int storefd;
+ int storefd = 0;
int exit_status = 0;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
@@ -96,10 +99,12 @@ main(
while ((opt = getopt(argc, argv, "o:l:s:c:S:")) != EOF) {
switch (opt) {
case 'o':
- sscanf(optarg, "%lld", &offset);
+ sscanf(optarg, "%lld", &lltemp);
+ offset = (dm_off_t) lltemp;
break;
case 'l':
- sscanf(optarg, "%llu", &length);
+ sscanf(optarg, "%llu", &ulltemp);
+ length = (dm_size_t) ulltemp;
break;
case 's':
sid = atol(optarg);
@@ -141,7 +146,8 @@ main(
sufficiently big boundary.
*/
if ((bufp = memalign(4096, length)) == NULL) {
- fprintf(stderr, "malloc of %llu bytes failed\n", length);
+ fprintf(stderr, "malloc of %llu bytes failed\n",
+ (unsigned long long) length);
exit(1);
}
memset(bufp, '\0', length);
@@ -168,8 +174,9 @@ main(
fprintf(stderr, "dm_read_invis failed, %s\n", strerror(errno));
exit_status++;
} else if (rc != length) {
- fprintf(stderr, "dm_read_invis expected to read %lld bytes, actually "
- "read %lld\n", length, rc);
+ fprintf(stderr, "dm_read_invis expected to read %llu bytes, actually "
+ "read %lld\n", (unsigned long long) length,
+ (long long) rc);
exit_status++;
}
diff --git a/dmapi/src/common/cmd/set_region.c b/dmapi/src/common/cmd/set_region.c
index 4c5c5ac..cfe5429 100644
--- a/dmapi/src/common/cmd/set_region.c
+++ b/dmapi/src/common/cmd/set_region.c
@@ -86,7 +86,8 @@ main(
int opt;
int i;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
diff --git a/dmapi/src/common/cmd/set_return_on_destroy.c b/dmapi/src/common/cmd/set_return_on_destroy.c
index c18cc59..a85e920 100644
--- a/dmapi/src/common/cmd/set_return_on_destroy.c
+++ b/dmapi/src/common/cmd/set_return_on_destroy.c
@@ -70,7 +70,8 @@ main(
int opt;
int Fflag = 0;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
diff --git a/dmapi/src/common/cmd/write_invis.c b/dmapi/src/common/cmd/write_invis.c
index 8c78a55..28744ac 100644
--- a/dmapi/src/common/cmd/write_invis.c
+++ b/dmapi/src/common/cmd/write_invis.c
@@ -71,7 +71,9 @@ main(
dm_sessid_t sid = DM_NO_SESSION;
char *object = NULL;
dm_off_t offset = 0;
+ long long lltemp;
dm_size_t length = 1;
+ unsigned long long ulltemp;
u_char ch = 'X';
void *bufp = NULL;
void *hanp;
@@ -83,7 +85,8 @@ main(
int storefd;
int exit_status = 0;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
@@ -97,10 +100,12 @@ main(
ch = *optarg;
break;
case 'o':
- sscanf(optarg, "%lld", &offset);
+ sscanf(optarg, "%lld", &lltemp);
+ offset = (dm_off_t) offset;
break;
case 'l':
- sscanf(optarg, "%llu", &length);
+ sscanf(optarg, "%llu", &ulltemp);
+ length = (dm_size_t) ulltemp;
break;
case 's':
sid = atol(optarg);
@@ -135,7 +140,8 @@ main(
sufficiently big boundary.
*/
if ((bufp = memalign(4096, length)) == NULL) {
- fprintf(stderr, "malloc of %llu bytes failed\n", length);
+ fprintf(stderr, "malloc of %llu bytes failed\n",
+ (unsigned long long) length);
exit(1);
}
memset(bufp, ch, length);
@@ -170,8 +176,9 @@ main(
fprintf(stderr, "dm_write_invis failed, %s\n", strerror(errno));
exit_status++;
} else if (rc != length) {
- fprintf(stderr, "dm_write_invis expected to write %lld bytes, actually "
- "wrote %lld\n", length, rc);
+ fprintf(stderr, "dm_write_invis expected to write %llu bytes, "
+ "actually wrote %lld\n",
+ (unsigned long long) length, (long long)rc);
exit_status++;
}
dm_handle_free(hanp, hlen);
diff --git a/dmapi/src/common/lib/find_session.c b/dmapi/src/common/lib/find_session.c
index dc1c519..e15fe7e 100644
--- a/dmapi/src/common/lib/find_session.c
+++ b/dmapi/src/common/lib/find_session.c
@@ -81,8 +81,8 @@ find_test_session(
allocelem = nelem;
sidbuf = realloc(sidbuf, nelem * sizeof(*sidbuf));
if (sidbuf == NULL) {
- fprintf(stderr, "realloc of %d bytes failed\n",
- nelem * sizeof(*sidbuf));
+ fprintf(stderr, "realloc of %lu bytes failed\n",
+ (unsigned long) nelem * sizeof(*sidbuf));
exit(1);
}
}
@@ -142,8 +142,8 @@ find_test_session(
allocelem = nelem;
sidbuf = realloc(sidbuf, nelem * sizeof(*sidbuf));
if (sidbuf == NULL) {
- fprintf(stderr, "realloc of %d bytes failed\n",
- nelem * sizeof(*sidbuf));
+ fprintf(stderr, "realloc of %lu bytes failed\n",
+ (unsigned long) nelem * sizeof(*sidbuf));
exit(1);
}
}
diff --git a/dmapi/src/common/lib/print.c b/dmapi/src/common/lib/print.c
index bbed8f3..384ba9c 100644
--- a/dmapi/src/common/lib/print.c
+++ b/dmapi/src/common/lib/print.c
@@ -132,31 +132,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,
@@ -183,13 +183,13 @@ print_one_data_event(
if (hanp && hlen) {
hantoa(hanp, hlen, handle);
} else {
- sprintf(handle, "<BAD HANDLE, hlen %d>", hlen);
+ sprintf(handle, "<BAD HANDLE, hlen %zd>", hlen);
}
printf(VALS VALLLD VALLLD,
"file_handle", handle,
- "offset", msg_de->de_offset,
- "length", msg_de->de_length);
+ "offset", (long long) msg_de->de_offset,
+ "length", (long long) msg_de->de_length);
return(0);
}
diff --git a/dmapi/src/common/lib/util.c b/dmapi/src/common/lib/util.c
index 958b7e4..f6a3335 100644
--- a/dmapi/src/common/lib/util.c
+++ b/dmapi/src/common/lib/util.c
@@ -234,12 +234,12 @@ print_handle(
char handle_str[HANDLE_STR];
if (hlen > HANDLE_LEN) {
- printf("-- invalid hlen length %d --\n", hlen);
+ printf("-- invalid hlen length %zd --\n", hlen);
return;
}
printf("print_handle: ");
- printf("%d\t", hlen);
+ printf("%zd\t", hlen);
hantoa(hanp, hlen, handle_str);
printf("%s\n ", handle_str);
}
@@ -257,10 +257,10 @@ print_victim(
return;
}
- printf("%d\t", hlen);
+ printf("%zd\t", hlen);
hantoa(hanp, hlen, handle_str);
printf("%s ", handle_str);
- printf("\t%lld \n", fsize);
+ printf("\t%lld \n", (long long) fsize);
}
@@ -583,20 +583,17 @@ validate_state(
if (report_errors) {
fprintf(stdout, "ERROR:dmstat->dt_dev 0x%llx, "
"statb.st_dev 0x%llx\n",
- (uint64_t)dmstat->dt_dev,
- (uint64_t)statb.st_dev);
+ (unsigned long long) dmstat->dt_dev,
+ (unsigned long long) statb.st_dev);
}
errors++;
}
if (dmstat->dt_ino != statb.st_ino) {
if (report_errors) {
fprintf(stdout, "ERROR:dmstat->dt_ino %llx, "
-#if defined(linux) || (defined(__sgi) && (_MIPS_SIM != _MIPS_SIM_ABI32))
"statb.st_ino %llx\n",
-#else
- "statb.st_ino %x\n",
-#endif
- dmstat->dt_ino, statb.st_ino);
+ (unsigned long long) dmstat->dt_ino,
+ (unsigned long long) statb.st_ino);
}
errors++;
}
@@ -620,9 +617,10 @@ validate_state(
}
if (dmstat->dt_nlink != statb.st_nlink) {
if (report_errors) {
- fprintf(stdout, "ERROR:dmstat->dt_nlink %d, "
- "statb.st_nlink %d\n", dmstat->dt_nlink,
- statb.st_nlink);
+ fprintf(stdout, "ERROR:dmstat->dt_nlink %u, "
+ "statb.st_nlink %u\n",
+ (unsigned int) dmstat->dt_nlink,
+ (unsigned int) statb.st_nlink);
}
errors++;
}
@@ -646,20 +644,17 @@ validate_state(
if (report_errors) {
fprintf(stdout, "ERROR:dmstat->dt_rdev 0x%llx, "
"statb.st_rdev 0x%llx\n",
- (uint64_t)dmstat->dt_rdev,
- (uint64_t)statb.st_rdev);
+ (unsigned long long) dmstat->dt_rdev,
+ (unsigned long long) statb.st_rdev);
}
errors++;
}
if (dmstat->dt_size != statb.st_size) {
if (report_errors) {
fprintf(stdout, "ERROR:dmstat->dt_size %lld, "
-#if defined(linux) || (defined(__sgi) && (_MIPS_SIM != _MIPS_SIM_ABI32))
"statb.st_size %lld\n",
-#else
- "statb.st_size %d\n",
-#endif
- dmstat->dt_size, statb.st_size);
+ (long long) dmstat->dt_size,
+ (long long) statb.st_size);
}
errors++;
}
@@ -706,12 +701,9 @@ validate_state(
if (dmstat->dt_blocks != statb.st_blocks) {
if (report_errors) {
fprintf(stdout, "ERROR:dmstat->dt_blocks %lld, "
-#if defined(linux) || (defined(__sgi) && (_MIPS_SIM != _MIPS_SIM_ABI32))
"statb.st_blocks %lld\n",
-#else
- "statb.st_blocks %d\n",
-#endif
- dmstat->dt_blocks, statb.st_blocks);
+ (long long) dmstat->dt_blocks,
+ (long long) statb.st_blocks);
}
errors++;
}
@@ -896,16 +888,20 @@ print_state(
{
/* Print all the stat block fields. */
- fprintf(stdout, "dt_dev 0x%llx\n", (uint64_t)dmstat->dt_dev);
- fprintf(stdout, "dt_ino %llx\n", dmstat->dt_ino);
+ fprintf(stdout, "dt_dev 0x%llx\n",
+ (unsigned long long) dmstat->dt_dev);
+ fprintf(stdout, "dt_ino %llx\n",
+ (unsigned long long) dmstat->dt_ino);
fprintf(stdout, "dt_mode (type) %s\n",
mode_to_string(dmstat->dt_mode));
fprintf(stdout, "dt_mode (perm) 0%o\n", dmstat->dt_mode & S_MASK);
fprintf(stdout, "dt_nlink %d\n", dmstat->dt_nlink);
fprintf(stdout, "dt_uid %d\n", dmstat->dt_uid);
fprintf(stdout, "dt_gid %d\n", dmstat->dt_gid);
- fprintf(stdout, "dt_rdev 0x%llx\n", (uint64_t)dmstat->dt_rdev);
- fprintf(stdout, "dt_size %lld\n", dmstat->dt_size);
+ fprintf(stdout, "dt_rdev 0x%llx\n",
+ (unsigned long long) dmstat->dt_rdev);
+ fprintf(stdout, "dt_size %lld\n",
+ (unsigned long long) dmstat->dt_size);
fprintf(stdout, "dt_atime %s\n",
date_to_string(dmstat->dt_atime));
@@ -915,7 +911,7 @@ print_state(
date_to_string(dmstat->dt_ctime));
fprintf(stdout, "dt_blksize %d\n", dmstat->dt_blksize);
- fprintf(stdout, "dt_blocks %lld\n", dmstat->dt_blocks);
+ fprintf(stdout, "dt_blocks %lld\n", (long long) dmstat->dt_blocks);
#if defined(__sgi) || defined(linux)
fprintf(stdout, "dt_xfs_igen %d\n", dmstat->dt_xfs_igen);
@@ -945,22 +941,22 @@ extern void
print_line(
dm_stat_t *dmstat)
{
- fprintf(stdout, "0x%llx|", (uint64_t)dmstat->dt_dev);
- fprintf(stdout, "%llx|", dmstat->dt_ino);
+ fprintf(stdout, "0x%llx|", (unsigned long long) dmstat->dt_dev);
+ fprintf(stdout, "%llx|", (unsigned long long) dmstat->dt_ino);
fprintf(stdout, "%s|", mode_to_string(dmstat->dt_mode));
fprintf(stdout, "0%o|", dmstat->dt_mode & S_MASK);
fprintf(stdout, "%d|", dmstat->dt_nlink);
fprintf(stdout, "%d|", dmstat->dt_uid);
fprintf(stdout, "%d|", dmstat->dt_gid);
- fprintf(stdout, "0x%llx|", (uint64_t)dmstat->dt_rdev);
- fprintf(stdout, "%lld|", dmstat->dt_size);
+ fprintf(stdout, "0x%llx|", (unsigned long long) dmstat->dt_rdev);
+ fprintf(stdout, "%lld|", (long long) dmstat->dt_size);
fprintf(stdout, "%s|", date_to_string(dmstat->dt_atime));
fprintf(stdout, "%s|", date_to_string(dmstat->dt_mtime));
fprintf(stdout, "%s|", date_to_string(dmstat->dt_ctime));
fprintf(stdout, "%d|", dmstat->dt_blksize);
- fprintf(stdout, "%lld|", dmstat->dt_blocks);
+ fprintf(stdout, "%lld|", (long long) dmstat->dt_blocks);
fprintf(stdout, "%d|", dmstat->dt_xfs_igen);
fprintf(stdout, "%s|", xflags_to_string(dmstat->dt_xfs_xflags));
diff --git a/dmapi/src/sample_hsm/migfind.c b/dmapi/src/sample_hsm/migfind.c
index be86761..95b6d6d 100644
--- a/dmapi/src/sample_hsm/migfind.c
+++ b/dmapi/src/sample_hsm/migfind.c
@@ -78,7 +78,7 @@ verify_size(
size = strtol(str,0,0);
if (size < 0 || size >= LONG_MAX ) {
- printf("Size %lld is invalid \n", size);
+ printf("Size %lld is invalid \n", (long long) size);
return(1);
}
@@ -87,13 +87,13 @@ verify_size(
cp++;
if (*cp == 'k' || *cp == 'K') {
if ( size >= (u_int) MAX_K) {
- printf("Size %lld is invalid\n", size);
+ printf("Size %lld is invalid\n", (long long) size);
return(1);
}
size *= 1024;
} else if (*cp == 'm' || *cp == 'M') {
if ( size >= (u_int) MAX_M) {
- printf("Size %lld is invalid\n", size);
+ printf("Size %lld is invalid\n", (long long) size);
return(1);
}
size *= (1024*1024);
diff --git a/dmapi/src/sample_hsm/mls.c b/dmapi/src/sample_hsm/mls.c
index 2afdbe2..10d8995 100644
--- a/dmapi/src/sample_hsm/mls.c
+++ b/dmapi/src/sample_hsm/mls.c
@@ -183,8 +183,8 @@ mr_info(
printf("\n");
for (i=0; i<ret; i++) {
printf("\tRegion %d:\n", i);
- printf("\t\toffset %lld, ", rgn[i].rg_offset);
- printf("size %lld, ", rgn[i].rg_size);
+ printf("\t\toffset %lld, ", (long long) rgn[i].rg_offset);
+ printf("size %llu, ", (unsigned long long) rgn[i].rg_size);
printf("flags 0x%x", rgn[i].rg_flags);
printf(" ( ");
if (rgn[i].rg_flags & DM_REGION_NOEVENT)
@@ -312,8 +312,8 @@ alloc_info(
printf("(resident): ");
if (ext[i].ex_type == DM_EXTENT_HOLE)
printf("(hole): ");
- printf("offset %lld, ", ext[i].ex_offset);
- printf("len %lld\n", ext[i].ex_length);
+ printf("offset %lld, ", (long long) ext[i].ex_offset);
+ printf("len %llu\n", (unsigned long long) ext[i].ex_length);
}
} while (more == 1);
return(0);
diff --git a/dmapi/src/simple/dm_find_eventmsg.c b/dmapi/src/simple/dm_find_eventmsg.c
index 20c4cb3..053614a 100644
--- a/dmapi/src/simple/dm_find_eventmsg.c
+++ b/dmapi/src/simple/dm_find_eventmsg.c
@@ -51,7 +51,7 @@ main( int argc, char **argv )
buflen = atoi( optarg );
break;
case 'q':
- printf("dm_eventmsg_t=%d\n", sizeof(dm_eventmsg_t) );
+ printf("dm_eventmsg_t=%zd\n", sizeof(dm_eventmsg_t) );
exit(0);
case 'h':
fprintf(stderr, "Usage: %s <-s sid> <-t token> [-l buflen]\n", argv[0]);
@@ -75,6 +75,6 @@ main( int argc, char **argv )
ret = dm_find_eventmsg( sid, token, buflen, &msg, &rlen );
printf( "ret=%d\n", ret );
- printf( "rlen=%d\n", rlen );
+ printf( "rlen=%zd\n", rlen );
exit(0);
}
diff --git a/dmapi/src/simple/dm_query_session.c b/dmapi/src/simple/dm_query_session.c
index cde754a..2b6a269 100644
--- a/dmapi/src/simple/dm_query_session.c
+++ b/dmapi/src/simple/dm_query_session.c
@@ -67,7 +67,7 @@ main( int argc, char **argv )
ret = dm_query_session( sid, buflen, sessinfo, &rlen );
printf( "ret=%d\n", ret );
- printf( "rlen=%d\n", rlen );
+ printf( "rlen=%zd\n", rlen );
if( ret != -1 )
printf( "sessinfo=%s\n", sessinfo );
exit(0);
--
1.7.4.4
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/5] xfstests: eliminate warnings under dmapi/src/suite1/cmd (1)
2011-04-21 19:11 ` [PATCH 1/5] xfstests: eliminate some warnings under dmapi/src Alex Elder
@ 2011-04-21 19:11 ` Alex Elder
2011-04-21 19:11 ` [PATCH 3/5] xfstests: eliminate warnings under dmapi/src/suite1/cmd (2) Alex Elder
` (2 subsequent siblings)
3 siblings, 0 replies; 7+ messages in thread
From: Alex Elder @ 2011-04-21 19:11 UTC (permalink / raw)
To: xfs; +Cc: Alex Elder
Eliminate build warnings reported on files located under
dmapi/src/suite1/cmd/ (first of three).
Signed-off-by: Alex Elder <aelder@sgi.com>
---
dmapi/src/suite1/cmd/create_userevent.c | 3 ++-
dmapi/src/suite1/cmd/dm_handle.c | 3 ++-
dmapi/src/suite1/cmd/downgrade_right.c | 3 ++-
dmapi/src/suite1/cmd/get_allocinfo.c | 25 ++++++++++++++-----------
dmapi/src/suite1/cmd/get_config_events.c | 5 +++--
dmapi/src/suite1/cmd/get_dirattrs.c | 10 ++++------
dmapi/src/suite1/cmd/get_dmattr.c | 9 +++++----
dmapi/src/suite1/cmd/get_eventlist.c | 9 +++------
dmapi/src/suite1/cmd/get_events.c | 9 +++++----
dmapi/src/suite1/cmd/get_fileattr.c | 3 ++-
dmapi/src/suite1/cmd/get_mountinfo.c | 9 +++++----
dmapi/src/suite1/cmd/get_region.c | 12 +++++-------
dmapi/src/suite1/cmd/getall_disp.c | 13 +++++++------
13 files changed, 59 insertions(+), 54 deletions(-)
diff --git a/dmapi/src/suite1/cmd/create_userevent.c b/dmapi/src/suite1/cmd/create_userevent.c
index 3edb083..35cb2f2 100644
--- a/dmapi/src/suite1/cmd/create_userevent.c
+++ b/dmapi/src/suite1/cmd/create_userevent.c
@@ -61,7 +61,8 @@ main(
char *name;
int opt;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
diff --git a/dmapi/src/suite1/cmd/dm_handle.c b/dmapi/src/suite1/cmd/dm_handle.c
index ba177d1..6ad9a92 100644
--- a/dmapi/src/suite1/cmd/dm_handle.c
+++ b/dmapi/src/suite1/cmd/dm_handle.c
@@ -80,7 +80,8 @@ main(
char fsbuffer3[100];
int fd;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
diff --git a/dmapi/src/suite1/cmd/downgrade_right.c b/dmapi/src/suite1/cmd/downgrade_right.c
index 840e797..96a4bb7 100644
--- a/dmapi/src/suite1/cmd/downgrade_right.c
+++ b/dmapi/src/suite1/cmd/downgrade_right.c
@@ -75,7 +75,8 @@ main(
char *name;
int opt;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
diff --git a/dmapi/src/suite1/cmd/get_allocinfo.c b/dmapi/src/suite1/cmd/get_allocinfo.c
index c568e0e..3583964 100644
--- a/dmapi/src/suite1/cmd/get_allocinfo.c
+++ b/dmapi/src/suite1/cmd/get_allocinfo.c
@@ -155,7 +155,8 @@ main(
char *name;
int opt;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
@@ -236,7 +237,8 @@ print_alloc(
char *type = NULL;
int rc;
- fprintf(stdout, "%s: starting offset %lld\n", pathname, startoff);
+ fprintf(stdout, "%s: starting offset %lld\n", pathname,
+ (long long) startoff);
/* Allocate space for the number of extents requested by the user. */
@@ -283,21 +285,21 @@ print_alloc(
if (!Dflag) {
fprintf(stdout, "\t%d: [%lld..%lld]: %s", num,
- extent[i].ex_offset / 512,
- (extent[i].ex_offset +
+ (long long) extent[i].ex_offset / 512,
+ (long long) (extent[i].ex_offset +
extent[i].ex_length - 1) / 512, type);
if ((extent[i].ex_offset % 512 != 0) ||
(endoff % 512 != 0)) {
fprintf(stdout, "\t[%lld..%lld]\n",
- extent[i].ex_offset % 512,
- (endoff-1) % 512);
+ (long long) extent[i].ex_offset % 512,
+ (long long) (endoff-1) % 512);
} else {
fprintf(stdout, "\n");
}
} else {
fprintf(stdout, "%5s %13lld %13lld\n",
- type, extent[i].ex_offset,
- extent[i].ex_length);
+ type, (long long) extent[i].ex_offset,
+ (long long) extent[i].ex_length);
}
/* The ex_offset of the first extent should match the
@@ -310,13 +312,14 @@ print_alloc(
if (extent[i].ex_offset != endoff) {
fprintf(stderr, "new extent (%lld)is not "
"adjacent to previous one (%lld)\n",
- extent[i].ex_offset, endoff);
+ (long long) extent[i].ex_offset,
+ (long long) endoff);
return(1);
}
if (num && (extent[i].ex_offset % 512) != 0) {
fprintf(stderr, "non-initial ex_offset (%lld) "
"is not a 512-byte multiple\n",
- extent[i].ex_offset);
+ (long long) extent[i].ex_offset);
return(1);
}
@@ -343,7 +346,7 @@ print_alloc(
if (rc && startoff != endoff) {
fprintf(stderr, "startoff is %lld, should be %lld\n",
- startoff, endoff);
+ (long long) startoff, (long long) endoff);
return(1);
}
diff --git a/dmapi/src/suite1/cmd/get_config_events.c b/dmapi/src/suite1/cmd/get_config_events.c
index cdaf7c2..d50eaa7 100644
--- a/dmapi/src/suite1/cmd/get_config_events.c
+++ b/dmapi/src/suite1/cmd/get_config_events.c
@@ -65,7 +65,8 @@ main(
int opt;
int i;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
@@ -104,7 +105,7 @@ main(
}
fprintf(stdout, "Events supported (0x%llx), nelemp %d:\n",
- eventset, nelemp);
+ (unsigned long long) eventset, nelemp);
for (i = 0; i < nelemp; i++) {
if (!DMEV_ISSET(i, eventset))
diff --git a/dmapi/src/suite1/cmd/get_dirattrs.c b/dmapi/src/suite1/cmd/get_dirattrs.c
index 8e2f4cb..49ac356 100644
--- a/dmapi/src/suite1/cmd/get_dirattrs.c
+++ b/dmapi/src/suite1/cmd/get_dirattrs.c
@@ -43,8 +43,6 @@ char *Progname;
static void
usage(void)
{
- int i;
-
fprintf(stderr, "usage:\t%s [-b buflen] [-l loc] [-s sid] [-1] [-q] dirpath\n",
Progname);
exit(1);
@@ -68,12 +66,12 @@ main(
size_t hlen;
char *name;
int opt;
- int i;
int ret;
int oneline = 0;
int quiet = 0;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
@@ -137,8 +135,8 @@ main(
exit(1);
}
if (!quiet) {
- fprintf(stdout, "ret = %d, rlenp is %d, loc is %lld\n", ret,
- rlenp, loc);
+ fprintf(stdout, "ret = %d, rlenp is %zd, loc is %lld\n",
+ ret, rlenp, (long long) loc);
}
if (rlenp > 0) {
dm_stat_t *statp;
diff --git a/dmapi/src/suite1/cmd/get_dmattr.c b/dmapi/src/suite1/cmd/get_dmattr.c
index 15fa6e2..9c7eabb 100644
--- a/dmapi/src/suite1/cmd/get_dmattr.c
+++ b/dmapi/src/suite1/cmd/get_dmattr.c
@@ -61,7 +61,7 @@ main(
dm_token_t token = DM_NO_TOKEN;
char *object;
dm_attrname_t *attrnamep;
- void *bufp;
+ void *bufp = NULL;
size_t buflen = 10000;
size_t rlenp;
void *hanp;
@@ -69,7 +69,8 @@ main(
char *name;
int opt;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
@@ -122,14 +123,14 @@ main(
bufp, &rlenp)) {
if (errno == E2BIG) {
fprintf(stderr, "dm_get_dmattr buffer too small, "
- "should be %d bytes\n", rlenp);
+ "should be %zd bytes\n", rlenp);
} else {
fprintf(stderr, "dm_get_dmattr failed, %s\n",
strerror(errno));
}
exit(1);
}
- fprintf(stdout, "rlenp is %d, value is '%s'\n", rlenp, (char*)bufp);
+ fprintf(stdout, "rlenp is %zd, value is '%s'\n", rlenp, (char*)bufp);
dm_handle_free(hanp, hlen);
exit(0);
diff --git a/dmapi/src/suite1/cmd/get_eventlist.c b/dmapi/src/suite1/cmd/get_eventlist.c
index 420024d..37ed8c4 100644
--- a/dmapi/src/suite1/cmd/get_eventlist.c
+++ b/dmapi/src/suite1/cmd/get_eventlist.c
@@ -80,7 +80,8 @@ main(
int opt;
int i;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
@@ -147,12 +148,8 @@ main(
return(1);
}
-#ifdef VERITAS_21
- fprintf(stdout, "Events on object %s (0x%x), nelemp %d:\n",
-#else
fprintf(stdout, "Events on object %s (0x%llx), nelemp %d:\n",
-#endif
- object, eventset, nelemp);
+ object, (unsigned long long) eventset, nelemp);
for (i = 0; i < nelemp; i++) {
if (!DMEV_ISSET(i, eventset))
diff --git a/dmapi/src/suite1/cmd/get_events.c b/dmapi/src/suite1/cmd/get_events.c
index e518e11..55ae09b 100644
--- a/dmapi/src/suite1/cmd/get_events.c
+++ b/dmapi/src/suite1/cmd/get_events.c
@@ -60,14 +60,15 @@ main(
dm_eventmsg_t *msg;
dm_sessid_t sid;
u_int flags = 0;
- void *bufp;
+ void *bufp = NULL;
size_t buflen = 10000;
u_int maxmsgs = 1;
size_t rlenp;
char *name;
int opt;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
@@ -109,14 +110,14 @@ main(
if (dm_get_events(sid, maxmsgs, flags, buflen, bufp, &rlenp)) {
if (errno == E2BIG) {
fprintf(stderr, "dm_get_events buffer too small, "
- "should be %d bytes\n", rlenp);
+ "should be %zd bytes\n", rlenp);
} else {
fprintf(stderr, "dm_get_events failed, (%d)%s\n",
errno, strerror(errno));
}
exit(1);
}
- fprintf(stdout, "rlenp=%d\n", rlenp);
+ fprintf(stdout, "rlenp=%zd\n", rlenp);
if (rlenp == 0)
return(0);
diff --git a/dmapi/src/suite1/cmd/get_fileattr.c b/dmapi/src/suite1/cmd/get_fileattr.c
index ff03924..e3ab024 100644
--- a/dmapi/src/suite1/cmd/get_fileattr.c
+++ b/dmapi/src/suite1/cmd/get_fileattr.c
@@ -59,7 +59,8 @@ main(
int opt;
int validate = 0;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
diff --git a/dmapi/src/suite1/cmd/get_mountinfo.c b/dmapi/src/suite1/cmd/get_mountinfo.c
index 052b6ea..bd0dc92 100644
--- a/dmapi/src/suite1/cmd/get_mountinfo.c
+++ b/dmapi/src/suite1/cmd/get_mountinfo.c
@@ -76,7 +76,7 @@ main(
{
dm_sessid_t sid = DM_NO_SESSION;
char *pathname;
- void *bufp;
+ void *bufp = NULL;
size_t buflen = 10000;
size_t rlenp;
void *fshanp;
@@ -84,7 +84,8 @@ main(
char *name;
int opt;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
@@ -134,14 +135,14 @@ main(
bufp, &rlenp)) {
if (errno == E2BIG) {
fprintf(stderr, "dm_get_mountinfo buffer too small, "
- "should be %d bytes\n", rlenp);
+ "should be %zd bytes\n", rlenp);
} else {
fprintf(stderr, "dm_get_mountinfo failed, %s\n",
strerror(errno));
}
exit(1);
}
- fprintf(stdout, "rlenp is %d\n", rlenp);
+ fprintf(stdout, "rlenp is %zd\n", rlenp);
print_one_mount_event(bufp);
dm_handle_free(fshanp, fshlen);
diff --git a/dmapi/src/suite1/cmd/get_region.c b/dmapi/src/suite1/cmd/get_region.c
index 6cafbd1..997faab 100644
--- a/dmapi/src/suite1/cmd/get_region.c
+++ b/dmapi/src/suite1/cmd/get_region.c
@@ -67,7 +67,8 @@ main(
int opt;
int i;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
@@ -121,12 +122,9 @@ main(
fprintf(stdout, "%d regions\n", nelemp);
for (i = 0; i < nelemp; i++) {
-#ifdef VERITAS_21
- fprintf(stdout, "offset %d, size %d, flags 0x%x\n",
-#else
- fprintf(stdout, "offset %lld, size %lld, flags 0x%x\n",
-#endif
- regbufp[i].rg_offset, regbufp[i].rg_size,
+ fprintf(stdout, "offset %lld, size %llu, flags 0x%x\n",
+ (long long) regbufp[i].rg_offset,
+ (unsigned long long) regbufp[i].rg_size,
regbufp[i].rg_flags);
}
diff --git a/dmapi/src/suite1/cmd/getall_disp.c b/dmapi/src/suite1/cmd/getall_disp.c
index a68fdfd..ef676b2 100644
--- a/dmapi/src/suite1/cmd/getall_disp.c
+++ b/dmapi/src/suite1/cmd/getall_disp.c
@@ -58,7 +58,7 @@ main(
{
dm_dispinfo_t *disp;
dm_sessid_t sid;
- void *bufp;
+ void *bufp = NULL;
size_t buflen = 10000;
void *hanp;
size_t hlen;
@@ -67,7 +67,8 @@ main(
char *name;
int opt;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
@@ -103,14 +104,14 @@ main(
if (dm_getall_disp(sid, buflen, bufp, &rlenp)) {
if (errno == E2BIG) {
fprintf(stderr, "dm_getall_disp buffer too small, "
- "should be %d bytes\n", rlenp);
+ "should be %zd bytes\n", rlenp);
} else {
fprintf(stderr, "dm_getall_disp failed, %s\n",
strerror(errno));
}
exit(1);
}
- fprintf(stdout, "rlenp is %d\n", rlenp);
+ fprintf(stdout, "rlenp is %zd\n", rlenp);
if (rlenp == 0)
return(0);
@@ -121,11 +122,11 @@ main(
if (hanp && hlen) {
hantoa(hanp, hlen, hans1);
} else {
- sprintf(hans1, "<BAD HANDLE, hlen %d>", hlen);
+ sprintf(hans1, "<BAD HANDLE, hlen %zd>", hlen);
}
printf("%-15s %s dm_eventset_t 0%llo\n",
"fshandle", hans1,
- disp->di_eventset);
+ (unsigned long long) disp->di_eventset);
disp = DM_STEP_TO_NEXT(disp, dm_dispinfo_t *);
}
--
1.7.4.4
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 3/5] xfstests: eliminate warnings under dmapi/src/suite1/cmd (2)
2011-04-21 19:11 ` [PATCH 1/5] xfstests: eliminate some warnings under dmapi/src Alex Elder
2011-04-21 19:11 ` [PATCH 2/5] xfstests: eliminate warnings under dmapi/src/suite1/cmd (1) Alex Elder
@ 2011-04-21 19:11 ` Alex Elder
2011-04-21 19:11 ` [PATCH 4/5] xfstests: eliminate warnings under dmapi/src/suite1/cmd (3) Alex Elder
2011-04-21 19:11 ` [PATCH 5/5] xfstests: eliminate warnings under dmapi/src/suite2/src Alex Elder
3 siblings, 0 replies; 7+ messages in thread
From: Alex Elder @ 2011-04-21 19:11 UTC (permalink / raw)
To: xfs; +Cc: Alex Elder
Eliminate build warnings reported on files located under
dmapi/src/suite1/cmd/ (second of three).
Signed-off-by: Alex Elder <aelder@sgi.com>
---
dmapi/src/suite1/cmd/getall_dmattr.c | 7 ++-
dmapi/src/suite1/cmd/handle_to_fshandle.c | 3 +-
dmapi/src/suite1/cmd/handle_to_path.c | 10 +++--
dmapi/src/suite1/cmd/init_service.c | 3 +-
dmapi/src/suite1/cmd/link_test.c | 69 ++++++++++++++++-------------
dmapi/src/suite1/cmd/make_sparse.c | 6 +-
dmapi/src/suite1/cmd/obj_ref_hold.c | 3 +-
dmapi/src/suite1/cmd/obj_ref_query.c | 3 +-
dmapi/src/suite1/cmd/obj_ref_rele.c | 3 +-
dmapi/src/suite1/cmd/path_to_fshandle.c | 3 +-
dmapi/src/suite1/cmd/path_to_handle.c | 3 +-
dmapi/src/suite1/cmd/pending.c | 3 +-
12 files changed, 67 insertions(+), 49 deletions(-)
diff --git a/dmapi/src/suite1/cmd/getall_dmattr.c b/dmapi/src/suite1/cmd/getall_dmattr.c
index 4637c27..56f2248 100644
--- a/dmapi/src/suite1/cmd/getall_dmattr.c
+++ b/dmapi/src/suite1/cmd/getall_dmattr.c
@@ -66,7 +66,8 @@ main(
char *name;
int opt;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
@@ -114,14 +115,14 @@ main(
bufp, &rlenp)) {
if (errno == E2BIG) {
fprintf(stderr, "dm_getall_dmattr buffer too small, "
- "should be %d bytes\n", rlenp);
+ "should be %zd bytes\n", rlenp);
} else {
fprintf(stderr, "dm_getall_dmattr failed, %s\n",
strerror(errno));
}
exit(1);
}
- fprintf(stdout, "rlenp is %d\n", rlenp);
+ fprintf(stdout, "rlenp is %zd\n", rlenp);
if (rlenp > 0) {
dm_attrlist_t *attrlist;
diff --git a/dmapi/src/suite1/cmd/handle_to_fshandle.c b/dmapi/src/suite1/cmd/handle_to_fshandle.c
index 8608c74..93f101d 100644
--- a/dmapi/src/suite1/cmd/handle_to_fshandle.c
+++ b/dmapi/src/suite1/cmd/handle_to_fshandle.c
@@ -56,7 +56,8 @@ main(
char buffer[100];
int error;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
diff --git a/dmapi/src/suite1/cmd/handle_to_path.c b/dmapi/src/suite1/cmd/handle_to_path.c
index 3a4c226..b3759e1 100644
--- a/dmapi/src/suite1/cmd/handle_to_path.c
+++ b/dmapi/src/suite1/cmd/handle_to_path.c
@@ -22,6 +22,7 @@
#include <string.h>
#include <lib/dmport.h>
+#include <lib/hsm.h>
#include <getopt.h>
#ifdef linux
@@ -84,7 +85,8 @@ main(
char *name;
int opt;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
@@ -130,16 +132,16 @@ main(
buflen, pathbufp, &rlenp)) {
if (errno == E2BIG) {
fprintf(stderr, "dm_handle_to_path buffer too small, "
- "should be %d bytes\n", rlenp);
+ "should be %zd bytes\n", rlenp);
} else {
fprintf(stderr, "dm_handle_to_path failed, (%d) %s\n",
errno, strerror(errno));
}
return(1);
}
- fprintf(stderr, "rlenp is %d, pathbufp is %s\n", rlenp, (char*)pathbufp);
+ fprintf(stderr, "rlenp is %zd, pathbufp is %s\n", rlenp, (char*)pathbufp);
if (strlen(pathbufp) + 1 != rlenp) {
- fprintf(stderr, "rlenp is %d, should be %d\n", rlenp,
+ fprintf(stderr, "rlenp is %zd, should be %zd\n", rlenp,
strlen(pathbufp) + 1);
return(1);
}
diff --git a/dmapi/src/suite1/cmd/init_service.c b/dmapi/src/suite1/cmd/init_service.c
index 1673f4c..6c7afa4 100644
--- a/dmapi/src/suite1/cmd/init_service.c
+++ b/dmapi/src/suite1/cmd/init_service.c
@@ -47,7 +47,8 @@ main(
{
char *name;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
diff --git a/dmapi/src/suite1/cmd/link_test.c b/dmapi/src/suite1/cmd/link_test.c
index 3d18de8..2d35da8 100644
--- a/dmapi/src/suite1/cmd/link_test.c
+++ b/dmapi/src/suite1/cmd/link_test.c
@@ -36,37 +36,44 @@ main(
int argc,
char **argv)
{
- dm_sessid_t sid, oldsid, targetsid, *newsidp, *sidbufp;
- dm_token_t token, *tokenp, *rtokenp, *tokenbufp;
- dm_attrname_t *attrnamep;
- dm_off_t off, *offp, *roffp;
- dm_extent_t *extentp;
- dm_inherit_t *inheritbufp;
- dm_stat_t *statp;
- dm_size_t len, *dmrlenp, *retvalp;
- dm_attrloc_t *locp;
- dm_eventset_t *eventsetp;
- dm_config_t flagname;
- dm_region_t *regbufp;
- dm_response_t response;
- dm_right_t right, *rightp;
- dm_igen_t igen, *igenp;
- dm_msgtype_t msgtype;
- dm_fileattr_t *attrp;
- dm_boolean_t enable, *exactflagp;
- dm_timestruct_t *delay;
- mode_t mode;
- size_t hlen, dirhlen, hlen1, hlen2, targhlen, *fshlenp, *hlenp;
- size_t msglen, buflen, *rlenp;
- u_int nelem, mask, maxmsgs, uflags, *nelemp, maxevent;
- void *hanp, *dirhanp, *hanp1, *hanp2, *targhanp;
- void *msgdatap, *bufp, **hanpp, *respbufp, **fshanpp;
- dm_fsid_t fsid, *fsidp;
- dm_ino_t ino, *inop;
- char *cname, *sessinfop, *path, *pathbufp, **versionstrpp;
- int flags, fd, setdtime, reterror;
- u_int urc;
- int rc;
+ dm_sessid_t sid = 0, oldsid = 0, targetsid = 0;
+ dm_sessid_t *newsidp = NULL, *sidbufp = NULL;
+ dm_token_t token = 0, *tokenp = NULL;
+ dm_token_t *rtokenp = NULL, *tokenbufp = NULL;
+ dm_attrname_t *attrnamep = NULL;
+ dm_off_t off = 0, *offp = NULL, *roffp = NULL;
+ dm_extent_t *extentp = NULL;
+ dm_inherit_t *inheritbufp = NULL;
+ dm_stat_t *statp = NULL;
+ dm_size_t len = 0, *dmrlenp = NULL, *retvalp = NULL;
+ dm_attrloc_t *locp = NULL;
+ dm_eventset_t *eventsetp = NULL;
+ dm_config_t flagname = DM_CONFIG_INVALID;
+ dm_region_t *regbufp = NULL;
+ dm_response_t response = DM_RESP_INVALID;
+ dm_right_t right = DM_RIGHT_NULL, *rightp = NULL;
+ dm_igen_t igen, *igenp = NULL;
+ dm_msgtype_t msgtype = DM_MSGTYPE_INVALID;
+ dm_fileattr_t *attrp = NULL;
+ dm_boolean_t enable = 0, *exactflagp = NULL;
+ dm_timestruct_t *delay = NULL;
+ mode_t mode = 0;
+ size_t hlen = 0, dirhlen = 0, hlen1 = 0, hlen2 = 0;
+ size_t targhlen = 0, *fshlenp = NULL, *hlenp = NULL;
+ size_t msglen = 0, buflen = 0, *rlenp = NULL;
+ u_int nelem = 0, mask = 0, maxmsgs = 0, uflags = 0;
+ u_int *nelemp = NULL, maxevent = 0;
+ void *hanp = NULL, *dirhanp = NULL;
+ void *hanp1 = NULL, *hanp2 = NULL, *targhanp = NULL;
+ void *msgdatap = NULL, *bufp = NULL, **hanpp = NULL;
+ void *respbufp = NULL, **fshanpp = NULL;
+ dm_fsid_t fsid, *fsidp = NULL;
+ dm_ino_t ino, *inop = NULL;
+ char *cname = NULL, *sessinfop = NULL;
+ char *path = NULL, *pathbufp = NULL, **versionstrpp = NULL;
+ int flags = 0, fd = 0, setdtime = 0, reterror = 0;
+ u_int urc;
+ int rc;
dm_ssize_t ssrc;
/* Definitions per the prototypes in dmport.h, in the same order. */
diff --git a/dmapi/src/suite1/cmd/make_sparse.c b/dmapi/src/suite1/cmd/make_sparse.c
index d16aeba..ead7390 100644
--- a/dmapi/src/suite1/cmd/make_sparse.c
+++ b/dmapi/src/suite1/cmd/make_sparse.c
@@ -82,18 +82,18 @@ main(
for (i = 0; i < 200; i += 2) {
offset = i * 65536;
if (lseek(fd, offset, SEEK_SET) < 0) {
- fprintf(stderr, "seek to %d failed, %s\n", offset,
+ fprintf(stderr, "seek to %zd failed, %s\n", offset,
strerror(errno));
exit(1);
}
if ((count = write(fd, buf, buflen)) < 0) {
fprintf(stderr, "write of %d bytes failed at offset "
- "%d, , %s\n", buflen, offset, strerror(errno));
+ "%zd, , %s\n", buflen, offset, strerror(errno));
exit(1);
}
if (count != buflen) {
fprintf(stderr, "expected to write %d bytes at offset "
- "%d, actually wrote %d\n", buflen, offset,
+ "%zd, actually wrote %zd\n", buflen, offset,
count);
exit(1);
}
diff --git a/dmapi/src/suite1/cmd/obj_ref_hold.c b/dmapi/src/suite1/cmd/obj_ref_hold.c
index c099547..771deb4 100644
--- a/dmapi/src/suite1/cmd/obj_ref_hold.c
+++ b/dmapi/src/suite1/cmd/obj_ref_hold.c
@@ -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];
diff --git a/dmapi/src/suite1/cmd/obj_ref_query.c b/dmapi/src/suite1/cmd/obj_ref_query.c
index a2150e9..f85716f 100644
--- a/dmapi/src/suite1/cmd/obj_ref_query.c
+++ b/dmapi/src/suite1/cmd/obj_ref_query.c
@@ -77,7 +77,8 @@ main(
int error;
int opt;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
diff --git a/dmapi/src/suite1/cmd/obj_ref_rele.c b/dmapi/src/suite1/cmd/obj_ref_rele.c
index cff486f..1c39b90 100644
--- a/dmapi/src/suite1/cmd/obj_ref_rele.c
+++ b/dmapi/src/suite1/cmd/obj_ref_rele.c
@@ -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];
diff --git a/dmapi/src/suite1/cmd/path_to_fshandle.c b/dmapi/src/suite1/cmd/path_to_fshandle.c
index 4c625ad..16563d9 100644
--- a/dmapi/src/suite1/cmd/path_to_fshandle.c
+++ b/dmapi/src/suite1/cmd/path_to_fshandle.c
@@ -55,7 +55,8 @@ main(
size_t fshlen;
char buffer[100];
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
diff --git a/dmapi/src/suite1/cmd/path_to_handle.c b/dmapi/src/suite1/cmd/path_to_handle.c
index 2263cb5..546415c 100644
--- a/dmapi/src/suite1/cmd/path_to_handle.c
+++ b/dmapi/src/suite1/cmd/path_to_handle.c
@@ -55,7 +55,8 @@ main(
size_t hlen;
char buffer[100];
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
diff --git a/dmapi/src/suite1/cmd/pending.c b/dmapi/src/suite1/cmd/pending.c
index 9a81500..827b61e 100644
--- a/dmapi/src/suite1/cmd/pending.c
+++ b/dmapi/src/suite1/cmd/pending.c
@@ -58,7 +58,8 @@ main(
char *name;
dm_token_t token;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
--
1.7.4.4
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 4/5] xfstests: eliminate warnings under dmapi/src/suite1/cmd (3)
2011-04-21 19:11 ` [PATCH 1/5] xfstests: eliminate some warnings under dmapi/src Alex Elder
2011-04-21 19:11 ` [PATCH 2/5] xfstests: eliminate warnings under dmapi/src/suite1/cmd (1) Alex Elder
2011-04-21 19:11 ` [PATCH 3/5] xfstests: eliminate warnings under dmapi/src/suite1/cmd (2) Alex Elder
@ 2011-04-21 19:11 ` Alex Elder
2011-04-21 19:11 ` [PATCH 5/5] xfstests: eliminate warnings under dmapi/src/suite2/src Alex Elder
3 siblings, 0 replies; 7+ messages in thread
From: Alex Elder @ 2011-04-21 19:11 UTC (permalink / raw)
To: xfs; +Cc: Alex Elder
Eliminate build warnings reported on files located under
dmapi/src/suite1/cmd/ (third of three).
Signed-off-by: Alex Elder <aelder@sgi.com>
---
dmapi/src/suite1/cmd/print_event.c | 30 +++++++++++++----------
dmapi/src/suite1/cmd/probe_hole.c | 6 +++-
dmapi/src/suite1/cmd/probe_punch_xfsctl_hole.c | 9 ++++--
dmapi/src/suite1/cmd/punch_hole.c | 3 +-
dmapi/src/suite1/cmd/query_right.c | 3 +-
dmapi/src/suite1/cmd/randomize_file.c | 4 +-
dmapi/src/suite1/cmd/release_right.c | 3 +-
dmapi/src/suite1/cmd/remove_dmattr.c | 3 +-
dmapi/src/suite1/cmd/request_right.c | 3 +-
dmapi/src/suite1/cmd/respond_event.c | 3 +-
dmapi/src/suite1/cmd/rwt.c | 15 ++++++-----
dmapi/src/suite1/cmd/security_hole.c | 3 +-
dmapi/src/suite1/cmd/security_hole2.c | 3 +-
dmapi/src/suite1/cmd/set_disp.c | 3 +-
dmapi/src/suite1/cmd/set_dmattr.c | 5 ++-
dmapi/src/suite1/cmd/set_eventlist.c | 3 +-
dmapi/src/suite1/cmd/set_fileattr.c | 3 +-
dmapi/src/suite1/cmd/struct_test.c | 4 +-
dmapi/src/suite1/cmd/sync_by_handle.c | 3 +-
dmapi/src/suite1/cmd/test_assumption.c | 4 +-
dmapi/src/suite1/cmd/upgrade_right.c | 3 +-
21 files changed, 70 insertions(+), 46 deletions(-)
diff --git a/dmapi/src/suite1/cmd/print_event.c b/dmapi/src/suite1/cmd/print_event.c
index e34cafe..afe2c27 100644
--- a/dmapi/src/suite1/cmd/print_event.c
+++ b/dmapi/src/suite1/cmd/print_event.c
@@ -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;
diff --git a/dmapi/src/suite1/cmd/probe_hole.c b/dmapi/src/suite1/cmd/probe_hole.c
index e0e4a5b..c5f945a 100644
--- a/dmapi/src/suite1/cmd/probe_hole.c
+++ b/dmapi/src/suite1/cmd/probe_hole.c
@@ -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);
}
diff --git a/dmapi/src/suite1/cmd/probe_punch_xfsctl_hole.c b/dmapi/src/suite1/cmd/probe_punch_xfsctl_hole.c
index effa98a..6b4d0b6 100644
--- a/dmapi/src/suite1/cmd/probe_punch_xfsctl_hole.c
+++ b/dmapi/src/suite1/cmd/probe_punch_xfsctl_hole.c
@@ -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)) {
diff --git a/dmapi/src/suite1/cmd/punch_hole.c b/dmapi/src/suite1/cmd/punch_hole.c
index e1d1a67..68439a7 100644
--- a/dmapi/src/suite1/cmd/punch_hole.c
+++ b/dmapi/src/suite1/cmd/punch_hole.c
@@ -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];
diff --git a/dmapi/src/suite1/cmd/query_right.c b/dmapi/src/suite1/cmd/query_right.c
index a8e219a..aab4de5 100644
--- a/dmapi/src/suite1/cmd/query_right.c
+++ b/dmapi/src/suite1/cmd/query_right.c
@@ -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];
diff --git a/dmapi/src/suite1/cmd/randomize_file.c b/dmapi/src/suite1/cmd/randomize_file.c
index a45b2a3..0c272fe 100644
--- a/dmapi/src/suite1/cmd/randomize_file.c
+++ b/dmapi/src/suite1/cmd/randomize_file.c
@@ -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);
diff --git a/dmapi/src/suite1/cmd/release_right.c b/dmapi/src/suite1/cmd/release_right.c
index 648619c..e6a92d3 100644
--- a/dmapi/src/suite1/cmd/release_right.c
+++ b/dmapi/src/suite1/cmd/release_right.c
@@ -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];
diff --git a/dmapi/src/suite1/cmd/remove_dmattr.c b/dmapi/src/suite1/cmd/remove_dmattr.c
index 3a82b51..865898d 100644
--- a/dmapi/src/suite1/cmd/remove_dmattr.c
+++ b/dmapi/src/suite1/cmd/remove_dmattr.c
@@ -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];
diff --git a/dmapi/src/suite1/cmd/request_right.c b/dmapi/src/suite1/cmd/request_right.c
index 11ff944..fff3e78 100644
--- a/dmapi/src/suite1/cmd/request_right.c
+++ b/dmapi/src/suite1/cmd/request_right.c
@@ -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];
diff --git a/dmapi/src/suite1/cmd/respond_event.c b/dmapi/src/suite1/cmd/respond_event.c
index 9fd297c..bf1bb94 100644
--- a/dmapi/src/suite1/cmd/respond_event.c
+++ b/dmapi/src/suite1/cmd/respond_event.c
@@ -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];
diff --git a/dmapi/src/suite1/cmd/rwt.c b/dmapi/src/suite1/cmd/rwt.c
index b5bd903..c1489bd 100644
--- a/dmapi/src/suite1/cmd/rwt.c
+++ b/dmapi/src/suite1/cmd/rwt.c
@@ -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);
}
}
diff --git a/dmapi/src/suite1/cmd/security_hole.c b/dmapi/src/suite1/cmd/security_hole.c
index c7c3db3..d34026d 100644
--- a/dmapi/src/suite1/cmd/security_hole.c
+++ b/dmapi/src/suite1/cmd/security_hole.c
@@ -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];
diff --git a/dmapi/src/suite1/cmd/security_hole2.c b/dmapi/src/suite1/cmd/security_hole2.c
index c40801c..3af3395 100644
--- a/dmapi/src/suite1/cmd/security_hole2.c
+++ b/dmapi/src/suite1/cmd/security_hole2.c
@@ -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];
diff --git a/dmapi/src/suite1/cmd/set_disp.c b/dmapi/src/suite1/cmd/set_disp.c
index b04306d..fee1dce 100644
--- a/dmapi/src/suite1/cmd/set_disp.c
+++ b/dmapi/src/suite1/cmd/set_disp.c
@@ -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];
diff --git a/dmapi/src/suite1/cmd/set_dmattr.c b/dmapi/src/suite1/cmd/set_dmattr.c
index df4f080..4753cf4 100644
--- a/dmapi/src/suite1/cmd/set_dmattr.c
+++ b/dmapi/src/suite1/cmd/set_dmattr.c
@@ -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];
diff --git a/dmapi/src/suite1/cmd/set_eventlist.c b/dmapi/src/suite1/cmd/set_eventlist.c
index 44d62b3..857523d 100644
--- a/dmapi/src/suite1/cmd/set_eventlist.c
+++ b/dmapi/src/suite1/cmd/set_eventlist.c
@@ -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];
diff --git a/dmapi/src/suite1/cmd/set_fileattr.c b/dmapi/src/suite1/cmd/set_fileattr.c
index eabb826..70ae52f 100644
--- a/dmapi/src/suite1/cmd/set_fileattr.c
+++ b/dmapi/src/suite1/cmd/set_fileattr.c
@@ -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];
diff --git a/dmapi/src/suite1/cmd/struct_test.c b/dmapi/src/suite1/cmd/struct_test.c
index 4d0b31f..7a0dd35 100644
--- a/dmapi/src/suite1/cmd/struct_test.c
+++ b/dmapi/src/suite1/cmd/struct_test.c
@@ -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)); \
} \
}
diff --git a/dmapi/src/suite1/cmd/sync_by_handle.c b/dmapi/src/suite1/cmd/sync_by_handle.c
index a00ee0f..ee22767 100644
--- a/dmapi/src/suite1/cmd/sync_by_handle.c
+++ b/dmapi/src/suite1/cmd/sync_by_handle.c
@@ -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];
diff --git a/dmapi/src/suite1/cmd/test_assumption.c b/dmapi/src/suite1/cmd/test_assumption.c
index 6760f34..98dea4c 100644
--- a/dmapi/src/suite1/cmd/test_assumption.c
+++ b/dmapi/src/suite1/cmd/test_assumption.c
@@ -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);
}
diff --git a/dmapi/src/suite1/cmd/upgrade_right.c b/dmapi/src/suite1/cmd/upgrade_right.c
index 5c4a50e..d249391 100644
--- a/dmapi/src/suite1/cmd/upgrade_right.c
+++ b/dmapi/src/suite1/cmd/upgrade_right.c
@@ -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];
--
1.7.4.4
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 5/5] xfstests: eliminate warnings under dmapi/src/suite2/src
2011-04-21 19:11 ` [PATCH 1/5] xfstests: eliminate some warnings under dmapi/src Alex Elder
` (2 preceding siblings ...)
2011-04-21 19:11 ` [PATCH 4/5] xfstests: eliminate warnings under dmapi/src/suite1/cmd (3) Alex Elder
@ 2011-04-21 19:11 ` Alex Elder
3 siblings, 0 replies; 7+ messages in thread
From: Alex Elder @ 2011-04-21 19:11 UTC (permalink / raw)
To: xfs; +Cc: Alex Elder
Eliminate build warnings reported on files located under
dmapi/src/suite2/src/.
Signed-off-by: Alex Elder <aelder@sgi.com>
---
dmapi/src/suite2/src/dm_test_daemon.c | 30 +++++++++++++++---------------
dmapi/src/suite2/src/invis_test.c | 32 +++++++++++++++++++++-----------
dmapi/src/suite2/src/mmap.c | 4 +++-
dmapi/src/suite2/src/region_test.c | 3 ++-
dmapi/src/suite2/src/send_msg.c | 3 ++-
dmapi/src/suite2/src/test_bulkall.c | 9 +++++----
dmapi/src/suite2/src/test_bulkattr.c | 7 ++++---
dmapi/src/suite2/src/test_dmattr.c | 5 +++--
dmapi/src/suite2/src/test_efault.c | 7 ++++---
dmapi/src/suite2/src/test_eventlist.c | 3 ++-
dmapi/src/suite2/src/test_fileattr.c | 10 ++++++----
dmapi/src/suite2/src/test_hole.c | 3 ++-
dmapi/src/suite2/src/test_invis.c | 10 ++++++----
dmapi/src/suite2/src/test_region.c | 3 ++-
dmapi/src/suite2/src/test_rights.c | 3 ++-
15 files changed, 79 insertions(+), 53 deletions(-)
diff --git a/dmapi/src/suite2/src/dm_test_daemon.c b/dmapi/src/suite2/src/dm_test_daemon.c
index 02f5d0e..c3ecf92 100644
--- a/dmapi/src/suite2/src/dm_test_daemon.c
+++ b/dmapi/src/suite2/src/dm_test_daemon.c
@@ -324,31 +324,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,
@@ -373,8 +373,8 @@ handle_message(
{
int pkt_error = 0;
int error;
- dm_response_t response;
- int respond, respcode;
+ dm_response_t response = DM_RESP_INVALID;
+ int respond, respcode = 0;
dm_namesp_event_t *msg_ne;
#if !VERITAS
dm_mount_event_t *msg_me;
@@ -578,24 +578,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;
}
@@ -658,7 +658,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 *);
@@ -1168,7 +1168,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;
diff --git a/dmapi/src/suite2/src/invis_test.c b/dmapi/src/suite2/src/invis_test.c
index 0cd8f81..6946b05 100644
--- a/dmapi/src/suite2/src/invis_test.c
+++ b/dmapi/src/suite2/src/invis_test.c
@@ -100,7 +100,8 @@ main(
int i;
int reuse_file = 0;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
@@ -157,9 +158,10 @@ main(
}
if( Vflag )
- printf("using length = %llu\n", length );
+ printf("using length = %llu\n", (unsigned long long) length);
if( length > BUFSZ ){
- fprintf(stderr, "length(%llu) > BUFSZ(%d)\n", length, BUFSZ);
+ fprintf(stderr, "length(%llu) > BUFSZ(%d)\n",
+ (unsigned long long) length, BUFSZ);
exit(1);
}
@@ -174,13 +176,15 @@ main(
exit(1);
}
if( rc != length ){
- fprintf(stderr, "dm_read_invis read %lld bytes, wanted to write %lld bytes\n",
- rc, length );
+ fprintf(stderr, "dm_read_invis read %llu bytes, "
+ "wanted to write %lld bytes\n",
+ (long long) rc, (unsigned long long) length);
dm_handle_free(hanp, hlen);
exitstat++;
}
else {
- printf("dm_read_invis read %lld bytes\n", rc);
+ printf("dm_read_invis read %lld bytes\n",
+ (long long) rc);
}
in_err_block = 0;
@@ -193,7 +197,10 @@ main(
}
else {
/* end of bad block */
- fprintf(stderr, "read err block: byte %lld to %lld\n", errblockstart, errblockend);
+ fprintf(stderr, "read err block: "
+ "byte %lld to %lld\n",
+ (long long) errblockstart,
+ (long long) errblockend);
in_err_block = 0;
}
}
@@ -205,7 +212,9 @@ main(
}
if( in_err_block ){
/* end of bad block */
- fprintf(stderr, "read err block: byte %lld to %lld\n", errblockstart, errblockend);
+ fprintf(stderr, "read err block: byte %lld to %lld\n",
+ (long long) errblockstart,
+ (long long) errblockend);
in_err_block = 0;
}
}
@@ -221,12 +230,13 @@ main(
exit(1);
}
if( rc != length ){
- fprintf(stderr, "dm_write_invis wrote %lld bytes, wanted to write %lld bytes\n",
- rc, length );
+ fprintf(stderr, "dm_write_invis wrote %lld bytes, "
+ "wanted to write %lld bytes\n",
+ (long long) rc, (long long) length );
dm_handle_free(hanp, hlen);
exit(1);
}
- printf("dm_write_invis wrote %lld bytes\n", rc);
+ printf("dm_write_invis wrote %lld bytes\n", (long long) rc);
}
dm_handle_free(hanp, hlen);
diff --git a/dmapi/src/suite2/src/mmap.c b/dmapi/src/suite2/src/mmap.c
index 983fe7d..a0fc4cc 100644
--- a/dmapi/src/suite2/src/mmap.c
+++ b/dmapi/src/suite2/src/mmap.c
@@ -121,11 +121,13 @@ static int mfile_opt(char * s, mfile_t * f);
static void print_flags(char *s, mfile_t *f);
static void Usage(void);
+int
main(int argc, char * argv[])
{
int opt;
- if ((Progname = strrchr(argv[0], '/')) == NULL)
+ Progname = strrchr(argv[0], '/');
+ if (Progname == NULL)
Progname = argv[0];
else
Progname++;
diff --git a/dmapi/src/suite2/src/region_test.c b/dmapi/src/suite2/src/region_test.c
index b6e94c8..f8b2868 100644
--- a/dmapi/src/suite2/src/region_test.c
+++ b/dmapi/src/suite2/src/region_test.c
@@ -83,7 +83,8 @@ main(
size_t hlen;
dm_boolean_t exactflag;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
diff --git a/dmapi/src/suite2/src/send_msg.c b/dmapi/src/suite2/src/send_msg.c
index b853b44..3f07d06 100644
--- a/dmapi/src/suite2/src/send_msg.c
+++ b/dmapi/src/suite2/src/send_msg.c
@@ -70,7 +70,8 @@ main(
int opt;
dm_msgtype_t msgtype = DM_MSGTYPE_SYNC;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
diff --git a/dmapi/src/suite2/src/test_bulkall.c b/dmapi/src/suite2/src/test_bulkall.c
index 427eeb3..0055cdc 100644
--- a/dmapi/src/suite2/src/test_bulkall.c
+++ b/dmapi/src/suite2/src/test_bulkall.c
@@ -172,14 +172,14 @@ my_print_victim(
else {
char handle_str[HANDLE_STR];
if (verbose & V_PRINT) {
- printf("%d\t", hlen);
+ printf("%zd\t", hlen);
hantoa(hanp, hlen, handle_str);
printf("%s ", handle_str);
if (extras) {
printf("size=%lld ",
- (int64_t)sbuf->dt_size);
+ (long long) sbuf->dt_size);
printf("ino=%lld ",
- (int64_t)sbuf->dt_ino);
+ (long long) sbuf->dt_ino);
}
}
@@ -271,7 +271,8 @@ scan_fs(
mask, dmattr, &locp, buflen,
dm_xstatbuf, &rlenp);
if (verbose & V_VERBOSE)
- fprintf(stderr, "BULKALL more=%d, rlen=%d\n", more, rlenp);
+ fprintf(stderr, "BULKALL more=%d, rlen=%zd\n",
+ more, rlenp);
if (more == -1) {
errno_msg("%s/%d: Can't get bulkall for filesystem", __FILE__, __LINE__, errno);
break;
diff --git a/dmapi/src/suite2/src/test_bulkattr.c b/dmapi/src/suite2/src/test_bulkattr.c
index 8b19d54..4ed5c64 100644
--- a/dmapi/src/suite2/src/test_bulkattr.c
+++ b/dmapi/src/suite2/src/test_bulkattr.c
@@ -164,7 +164,7 @@ my_print_victim(
else {
char handle_str[HANDLE_STR];
if (verbose & V_PRINT) {
- printf("%d\t", hlen);
+ printf("%zd\t", hlen);
hantoa(hanp, hlen, handle_str);
printf("%s ", handle_str);
}
@@ -187,7 +187,7 @@ my_print_victim(
if( isalpha(buf[0]) )
printf("(%s)\n", buf);
else
- printf("<len=%d>\n",rlen);
+ printf("<len=%zd>\n",rlen);
}
}
}
@@ -251,7 +251,8 @@ scan_fs(
more = dm_get_bulkattr(sid, fs_hanp, fs_hlen, DM_NO_TOKEN,
mask, &locp, buflen, dm_statbuf, &rlenp);
if (verbose & V_VERBOSE)
- fprintf(stderr, "BULKATTR more=%d, rlen=%d\n", more, rlenp);
+ fprintf(stderr, "BULKATTR more=%d, rlen=%zd\n",
+ more, rlenp);
if (more == -1) {
errno_msg("%s/%d: Can't get bulkattr for filesystem", __FILE__, __LINE__, errno);
break;
diff --git a/dmapi/src/suite2/src/test_dmattr.c b/dmapi/src/suite2/src/test_dmattr.c
index 7609800..cf87077 100644
--- a/dmapi/src/suite2/src/test_dmattr.c
+++ b/dmapi/src/suite2/src/test_dmattr.c
@@ -101,7 +101,8 @@ main(
struct stat *statbuf;
struct stat *checkbuf;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
@@ -224,7 +225,7 @@ main(
bufp, &rlenp)) {
if (errno == E2BIG) {
fprintf(stderr, "dm_get_dmattr buffer too small, "
- "should be %d bytes\n", rlenp);
+ "should be %zd bytes\n", rlenp);
} else {
fprintf(stderr, "dm_get_dmattr failed (%s) for test file %d\n",
ERR_NAME, i);
diff --git a/dmapi/src/suite2/src/test_efault.c b/dmapi/src/suite2/src/test_efault.c
index 19cb0a3..83f69b2 100644
--- a/dmapi/src/suite2/src/test_efault.c
+++ b/dmapi/src/suite2/src/test_efault.c
@@ -84,7 +84,8 @@ main(int argc, char **argv) {
char command[100];
int opt;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
@@ -170,7 +171,7 @@ main(int argc, char **argv) {
*/
{
u_int nelem=5;
- u_int *nelemp;
+ u_int *nelemp = NULL;
dm_eventset_t *eventsetp;
eventsetp = (dm_eventset_t *)malloc(nelem*sizeof(dm_eventset_t));
if (eventsetp == NULL) {
@@ -198,7 +199,7 @@ main(int argc, char **argv) {
{
size_t buflen = 5;
void *bufp = (void *)malloc(buflen*sizeof(dm_attrlist_t));
- size_t *rlenp;
+ size_t *rlenp = NULL;
ERRTEST(EFAULT, "getall_dmattr (NULL handle)",
dm_getall_dmattr(sid, NULL, hlen, DM_NO_TOKEN,
buflen, bufp, rlenp))
diff --git a/dmapi/src/suite2/src/test_eventlist.c b/dmapi/src/suite2/src/test_eventlist.c
index e633736..2257f44 100644
--- a/dmapi/src/suite2/src/test_eventlist.c
+++ b/dmapi/src/suite2/src/test_eventlist.c
@@ -93,7 +93,8 @@ main(
void *test_vp;
u_int nelemp;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
diff --git a/dmapi/src/suite2/src/test_fileattr.c b/dmapi/src/suite2/src/test_fileattr.c
index 520fc8d..ff8da83 100644
--- a/dmapi/src/suite2/src/test_fileattr.c
+++ b/dmapi/src/suite2/src/test_fileattr.c
@@ -123,7 +123,7 @@ comp_stat ( dm_stat_t expected,
if (found.dt_size != expected.dt_size) {
fprintf(stderr,
"ERROR: get #%d, expected size %lld, but found %lld\n",
- i, expected.dt_size, found.dt_size);
+ i, (long long) expected.dt_size, (long long) found.dt_size);
}
else good++;
if (Vflag){
@@ -183,7 +183,8 @@ main(
char *chk_name_p;
int chk_num;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
@@ -405,7 +406,8 @@ main(
}
else {
/* Put JUST name (not path) from check_name into chk_name_p */
- if (chk_name_p = strrchr(check_name, '/')) chk_name_p++;
+ chk_name_p = strrchr(check_name, '/');
+ if (chk_name_p) chk_name_p++;
else chk_name_p = check_name;
/* Verify that check_name_p holds a testfile name */
if (strncmp(chk_name_p, "DMAPI_fileattr_test.",20)==0) {
@@ -679,7 +681,7 @@ main(
ERRTEST(EFAULT, "get_bulk (bad rlenp)",
dm_get_bulkattr(sid, fs_hanp, fs_hlen, DM_NO_TOKEN,
GET_MASK, &loc, buflen, bufp,
- (u_int*)(-1000)))
+ (size_t *) (-1000)))
ERRTEST(EFAULT, "get_bulk (bad bufp)",
dm_get_bulkattr(sid, fs_hanp, fs_hlen, DM_NO_TOKEN,
GET_MASK, &loc, buflen, p, &rlen))
diff --git a/dmapi/src/suite2/src/test_hole.c b/dmapi/src/suite2/src/test_hole.c
index a000f26..3c2c4fc 100644
--- a/dmapi/src/suite2/src/test_hole.c
+++ b/dmapi/src/suite2/src/test_hole.c
@@ -90,7 +90,8 @@ main(
struct stat checkbuf;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
diff --git a/dmapi/src/suite2/src/test_invis.c b/dmapi/src/suite2/src/test_invis.c
index 115ffb7..766d767 100644
--- a/dmapi/src/suite2/src/test_invis.c
+++ b/dmapi/src/suite2/src/test_invis.c
@@ -102,7 +102,8 @@ main(
int cont;
int error_reported;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
@@ -180,7 +181,8 @@ main(
if(curlength>0)
free(bufp);
if ((bufp = malloc(length)) == NULL) {
- fprintf(stderr, "malloc of %llu bytes failed\n", length);
+ fprintf(stderr, "malloc of %llu bytes failed\n",
+ (unsigned long long) length);
continue;
}
curlength = length;
@@ -195,7 +197,7 @@ main(
cont=1;
} else if (rc != length) {
fprintf(stderr, "expected to write %lld bytes, actually "
- "wrote %lld\n", length, rc);
+ "wrote %lld\n", (long long) length, (long long) rc);
cont=1;
}
if(cont)
@@ -238,7 +240,7 @@ main(
}
else if (rc != length) {
fprintf(stderr, "expected to read %lld bytes, actually "
- "wrote %lld\n", length, rc);
+ "wrote %lld\n", (long long) length, (long long) rc);
continue;
}
else {
diff --git a/dmapi/src/suite2/src/test_region.c b/dmapi/src/suite2/src/test_region.c
index 455fbb4..7bdb0bf 100644
--- a/dmapi/src/suite2/src/test_region.c
+++ b/dmapi/src/suite2/src/test_region.c
@@ -95,7 +95,8 @@ main(
int Vflag = 0;
dm_token_t test_token = DM_NO_TOKEN;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
diff --git a/dmapi/src/suite2/src/test_rights.c b/dmapi/src/suite2/src/test_rights.c
index a245f65..1198e0a 100644
--- a/dmapi/src/suite2/src/test_rights.c
+++ b/dmapi/src/suite2/src/test_rights.c
@@ -91,7 +91,8 @@ main(int argc, char **argv) {
int opt;
int i=0;
- if (Progname = strrchr(argv[0], '/')) {
+ Progname = strrchr(argv[0], '/');
+ if (Progname) {
Progname++;
} else {
Progname = argv[0];
--
1.7.4.4
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 0/5] xfstests: dmapi: eliminate all build warnings
2011-04-21 19:11 [PATCH 0/5] xfstests: dmapi: eliminate all build warnings Alex Elder
2011-04-21 19:11 ` [PATCH 1/5] xfstests: eliminate some warnings under dmapi/src Alex Elder
@ 2011-04-28 2:15 ` Dave Chinner
1 sibling, 0 replies; 7+ messages in thread
From: Dave Chinner @ 2011-04-28 2:15 UTC (permalink / raw)
To: Alex Elder; +Cc: xfs
On Thu, Apr 21, 2011 at 02:11:08PM -0500, Alex Elder wrote:
> I'm hoping the Nth time is a charm...
>
> This series eliminates all build warnings that come out of building
> things under the dmapi/ subdirectory. I'm using gcc 4.x (x < 6) on
> several platforms, including i386, x86_64, and ia64 architectures.
>
> I have sent these a couple times before; they are unmodified from
> the last time I sent them out. I am omitting the summary of the
> types of warnings and how they got resolved this time around.
Nothing jumps out as wrong to me. Looks good.
Reviewed-by: Dave Chinner <dchinner@redhat.com>
--
Dave Chinner
david@fromorbit.com
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2011-04-28 2:12 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-04-21 19:11 [PATCH 0/5] xfstests: dmapi: eliminate all build warnings Alex Elder
2011-04-21 19:11 ` [PATCH 1/5] xfstests: eliminate some warnings under dmapi/src Alex Elder
2011-04-21 19:11 ` [PATCH 2/5] xfstests: eliminate warnings under dmapi/src/suite1/cmd (1) Alex Elder
2011-04-21 19:11 ` [PATCH 3/5] xfstests: eliminate warnings under dmapi/src/suite1/cmd (2) Alex Elder
2011-04-21 19:11 ` [PATCH 4/5] xfstests: eliminate warnings under dmapi/src/suite1/cmd (3) Alex Elder
2011-04-21 19:11 ` [PATCH 5/5] xfstests: eliminate warnings under dmapi/src/suite2/src Alex Elder
2011-04-28 2:15 ` [PATCH 0/5] xfstests: dmapi: eliminate all build warnings Dave Chinner
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox