dm crypt: fix get_key_size compiler warning if !CONFIG_KEYS
authorAashish Sharma <shraash@google.com>
Fri, 11 Feb 2022 12:15:38 +0000 (12:15 +0000)
committerMike Snitzer <snitzer@redhat.com>
Tue, 22 Feb 2022 16:27:56 +0000 (11:27 -0500)
Explicitly convert unsigned int in the right of the conditional
expression to int to match the left side operand and the return type,
fixing the following compiler warning:

drivers/md/dm-crypt.c:2593:43: warning: signed and unsigned
type in conditional expression [-Wsign-compare]

Fixes: c538f6ec9f56 ("dm crypt: add ability to use keys from the kernel key retention service")
Signed-off-by: Aashish Sharma <shraash@google.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
drivers/md/dm-crypt.c

index f310f2b..ec746cc 100644 (file)
@@ -2582,7 +2582,7 @@ static int crypt_set_keyring_key(struct crypt_config *cc, const char *key_string
 
 static int get_key_size(char **key_string)
 {
-       return (*key_string[0] == ':') ? -EINVAL : strlen(*key_string) >> 1;
+       return (*key_string[0] == ':') ? -EINVAL : (int)(strlen(*key_string) >> 1);
 }
 
 #endif /* CONFIG_KEYS */