diff options
| author | Mikulas Patocka <mpatocka@redhat.com> | 2013-07-11 00:41:17 +0200 |
|---|---|---|
| committer | Alasdair G Kergon <agk@redhat.com> | 2013-07-11 00:41:17 +0200 |
| commit | 5d8be84397618a2c04ec2b74d5d5a3d852a2ee5a (patch) | |
| tree | 47b6d39cec72b8c9a18d9c38853e1a20a0fd254e /drivers | |
| parent | dm: use __GFP_HIGHMEM in __vmalloc (diff) | |
| download | linux-5d8be84397618a2c04ec2b74d5d5a3d852a2ee5a.tar.xz linux-5d8be84397618a2c04ec2b74d5d5a3d852a2ee5a.zip | |
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>
Diffstat (limited to 'drivers')
| -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 0d2e812d424b..9804da16a476 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; |