Commit 373b8eeb authored by Alexey Dobriyan's avatar Alexey Dobriyan Committed by Steffen Klassert

xfrm: make aead_len() return unsigned int

Key lengths can't be negative.

Comparison with nla_len() is left signed just in case negative value
can sneak in there.
Signed-off-by: default avatarAlexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: default avatarSteffen Klassert <steffen.klassert@secunet.com>
parent 39e50d96
...@@ -1764,7 +1764,7 @@ static inline int xfrm_acquire_is_on(struct net *net) ...@@ -1764,7 +1764,7 @@ static inline int xfrm_acquire_is_on(struct net *net)
} }
#endif #endif
static inline int aead_len(struct xfrm_algo_aead *alg) static inline unsigned int aead_len(struct xfrm_algo_aead *alg)
{ {
return sizeof(*alg) + ((alg->alg_key_len + 7) / 8); return sizeof(*alg) + ((alg->alg_key_len + 7) / 8);
} }
......
...@@ -84,7 +84,7 @@ static int verify_aead(struct nlattr **attrs) ...@@ -84,7 +84,7 @@ static int verify_aead(struct nlattr **attrs)
return 0; return 0;
algp = nla_data(rt); algp = nla_data(rt);
if (nla_len(rt) < aead_len(algp)) if (nla_len(rt) < (int)aead_len(algp))
return -EINVAL; return -EINVAL;
algp->alg_name[sizeof(algp->alg_name) - 1] = '\0'; algp->alg_name[sizeof(algp->alg_name) - 1] = '\0';
......
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