diff options
author | Mikulas Patocka <mpatocka@redhat.com> | 2013-07-10 22:41:17 (GMT) |
---|---|---|
committer | Alasdair G Kergon <agk@redhat.com> | 2013-07-10 22:41:17 (GMT) |
commit | 5d8be84397618a2c04ec2b74d5d5a3d852a2ee5a (patch) | |
tree | 47b6d39cec72b8c9a18d9c38853e1a20a0fd254e | |
parent | 220cd058d9b66b940105e0a32324f06d557deba7 (diff) | |
download | linux-5d8be84397618a2c04ec2b74d5d5a3d852a2ee5a.tar.xz |
dm verity: remove pointless comparison
Remove num < 0 test in verity_ctr because num is unsigned.
(Found by Coverity.)
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Signed-off-by: Alasdair G Kergon <agk@redhat.com>
-rw-r--r-- | drivers/md/dm-verity.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/md/dm-verity.c b/drivers/md/dm-verity.c index 0d2e812..9804da1 100644 --- a/drivers/md/dm-verity.c +++ b/drivers/md/dm-verity.c @@ -695,8 +695,8 @@ static int verity_ctr(struct dm_target *ti, unsigned argc, char **argv) goto bad; } - if (sscanf(argv[0], "%d%c", &num, &dummy) != 1 || - num < 0 || num > 1) { + if (sscanf(argv[0], "%u%c", &num, &dummy) != 1 || + num > 1) { ti->error = "Invalid version"; r = -EINVAL; goto bad; |