commit 5d8be84397618a2c04ec2b74d5d5a3d852a2ee5a
author Mikulas Patocka <mpatocka@redhat.com> Wed Jul 10 23:41:17 2013 +0100
committer Alasdair G Kergon <agk@redhat.com> Wed Jul 10 23:41:17 2013 +0100
tree 47b6d39cec72b8c9a18d9c38853e1a20a0fd254e
parent 220cd058d9b66b940105e0a32324f06d557deba7

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>

drivers/md/dm-verity.c

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 @@
 		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;