mirror of
https://github.com/BobTheBlinker/android_kernel_motorola_sm6375.git
synced 2026-10-29 03:01:39 -04:00
KEYS: fix parsing invalid pkey info string
We need to check the return value of match_token() for Opt_err before doing anything with it. [ Not only did the old "-1" value for Opt_err cause problems for the __test_and_set_bit(), as fixed in commit94c13f66e1("security: don't use a negative Opt_err token index"), but accessing "args[0].from" is invalid for the Opt_err case, as pointed out by Eric later. - Linus ] Reported-by: [email protected] Fixes:00d60fd3b9("KEYS: Provide keyctls to drive the new key type ops for asymmetric keys [ver #2]") Signed-off-by: Eric Biggers <[email protected]> Cc: [email protected] # 4.20 Signed-off-by: Linus Torvalds <[email protected]>
This commit is contained in:
parent
e1ef035d27
commit
57b0e31453
1 changed files with 2 additions and 0 deletions
|
|
@ -50,6 +50,8 @@ static int keyctl_pkey_params_parse(struct kernel_pkey_params *params)
|
|||
if (*p == '\0' || *p == ' ' || *p == '\t')
|
||||
continue;
|
||||
token = match_token(p, param_keys, args);
|
||||
if (token == Opt_err)
|
||||
return -EINVAL;
|
||||
if (__test_and_set_bit(token, &token_mask))
|
||||
return -EINVAL;
|
||||
q = args[0].from;
|
||||
|
|
|
|||
Loading…
Reference in a new issue