diff options
author | kan <kan@FreeBSD.org> | 2003-07-11 05:47:05 +0000 |
---|---|---|
committer | kan <kan@FreeBSD.org> | 2003-07-11 05:47:05 +0000 |
commit | 8c2df19687b187984be9a4b27caba6d4bcf1b96b (patch) | |
tree | 84931f6828f4b8ccd41393c4be85ba28f8e3c42a /sbin/mdmfs/mdmfs.c | |
parent | e16ef3051fc93762c9e35dc8623f838d928e1d07 (diff) | |
download | FreeBSD-src-8c2df19687b187984be9a4b27caba6d4bcf1b96b.zip FreeBSD-src-8c2df19687b187984be9a4b27caba6d4bcf1b96b.tar.gz |
Do not compare unsigned int values with ULONG_MAX. The comparison is
always false on 64bit platforms and GCC 3.3.1 issues warning there.
Diffstat (limited to 'sbin/mdmfs/mdmfs.c')
-rw-r--r-- | sbin/mdmfs/mdmfs.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/sbin/mdmfs/mdmfs.c b/sbin/mdmfs/mdmfs.c index e741299..035f1fe 100644 --- a/sbin/mdmfs/mdmfs.c +++ b/sbin/mdmfs/mdmfs.c @@ -246,7 +246,7 @@ main(int argc, char **argv) unit = -1; } else { unit = strtoul(unitstr, &p, 10); - if ((unsigned)unit == ULONG_MAX || *p != '\0') + if (unit == (unsigned)ULONG_MAX || *p != '\0') errx(1, "bad device unit: %s", unitstr); } @@ -394,7 +394,7 @@ do_mdconfig_attach_au(const char *args, const enum md_types mdtype) strncpy(linebuf, linep + mdnamelen, linelen); linebuf[linelen] = '\0'; unit = strtoul(linebuf, &p, 10); - if ((unsigned)unit == ULONG_MAX || *p != '\n') + if (unit == (unsigned)ULONG_MAX || *p != '\n') errx(1, "unexpected output from mdconfig (attach)"); fclose(sfd); @@ -513,7 +513,7 @@ extract_ugid(const char *str, struct mtpt_info *mip) /* Derive uid. */ *uid = strtoul(user, &p, 10); - if ((unsigned)*uid == ULONG_MAX) + if (*uid == (uid_t)ULONG_MAX) usage(); if (*p != '\0') { pw = getpwnam(user); @@ -525,7 +525,7 @@ extract_ugid(const char *str, struct mtpt_info *mip) /* Derive gid. */ *gid = strtoul(group, &p, 10); - if ((unsigned)*gid == ULONG_MAX) + if (*gid == (gid_t)ULONG_MAX) usage(); if (*p != '\0') { gr = getgrnam(group); |