vlserver: unsigned values are always >0

Maxvolidbump is defined as an unisgned int, and thus can never be less
than 0. Remove the pointless check, as it just makes clang sad.

Change-Id: I78a2c8f6fcaee17196e37183256e6935cdca183b
Reviewed-on: http://gerrit.openafs.org/7084
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementix.org>
This commit is contained in:
Simon Wilkinson 2012-03-30 19:33:55 +01:00 committed by Derrick Brashear
parent 408afc5690
commit 350b5d5877

View File

@ -683,7 +683,7 @@ SVL_GetNewVolumeId(struct rx_call *rxcall, afs_uint32 Maxvolidbump,
if (!afsconf_SuperUser(vldb_confdir, rxcall, NULL))
END(VL_PERM);
if (Maxvolidbump < 0 || Maxvolidbump > MAXBUMPCOUNT)
if (Maxvolidbump > MAXBUMPCOUNT)
END(VL_BADVOLIDBUMP);
if ((code = Init_VLdbase(&ctx, LOCKWRITE, this_op)))