Commit e36215d8 authored by Matthias Kaehlcke's avatar Matthias Kaehlcke Committed by Mike Snitzer

dm ioctl: remove double parentheses

The extra pair of parantheses is not needed and causes clang to generate
warnings about the DM_DEV_CREATE_CMD comparison in validate_params().

Also remove another double parentheses that doesn't cause a warning.
Signed-off-by: default avatarMatthias Kaehlcke <mka@chromium.org>
Signed-off-by: default avatarMike Snitzer <snitzer@redhat.com>
parent 9119fedd
...@@ -1770,12 +1770,12 @@ static int validate_params(uint cmd, struct dm_ioctl *param) ...@@ -1770,12 +1770,12 @@ static int validate_params(uint cmd, struct dm_ioctl *param)
cmd == DM_LIST_VERSIONS_CMD) cmd == DM_LIST_VERSIONS_CMD)
return 0; return 0;
if ((cmd == DM_DEV_CREATE_CMD)) { if (cmd == DM_DEV_CREATE_CMD) {
if (!*param->name) { if (!*param->name) {
DMWARN("name not supplied when creating device"); DMWARN("name not supplied when creating device");
return -EINVAL; return -EINVAL;
} }
} else if ((*param->uuid && *param->name)) { } else if (*param->uuid && *param->name) {
DMWARN("only supply one of name or uuid, cmd(%u)", cmd); DMWARN("only supply one of name or uuid, cmd(%u)", cmd);
return -EINVAL; return -EINVAL;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment