* [PATCH 18/18] keyctl_pkey: Add pkey parameter slen to pass in PSS salt length [not found] <20210330202829.4825-1-varad.gautam@suse.com> @ 2021-03-30 20:28 ` Varad Gautam 2021-03-31 23:13 ` Jarkko Sakkinen 0 siblings, 1 reply; 2+ messages in thread From: Varad Gautam @ 2021-03-30 20:28 UTC (permalink / raw) To: linux-crypto Cc: Varad Gautam, David Howells, Herbert Xu, David S. Miller, Jarkko Sakkinen, James Morris, Serge E. Hallyn, open list:ASYMMETRIC KEYS, open list, open list:SECURITY SUBSYSTEM keyctl pkey_* operations accept enc and hash parameters at present. RSASSA-PSS signatures also require passing in the signature salt length. Add another parameter 'slen' to feed in salt length of a PSS signature. Signed-off-by: Varad Gautam <varad.gautam@suse.com> --- crypto/asymmetric_keys/asymmetric_type.c | 1 + include/linux/keyctl.h | 1 + security/keys/keyctl_pkey.c | 6 ++++++ 3 files changed, 8 insertions(+) diff --git a/crypto/asymmetric_keys/asymmetric_type.c b/crypto/asymmetric_keys/asymmetric_type.c index ad8af3d70ac0..eb2ef4a07f8e 100644 --- a/crypto/asymmetric_keys/asymmetric_type.c +++ b/crypto/asymmetric_keys/asymmetric_type.c @@ -571,6 +571,7 @@ static int asymmetric_key_verify_signature(struct kernel_pkey_params *params, .hash_algo = params->hash_algo, .digest = (void *)in, .s = (void *)in2, + .salt_length = params->slen, }; return verify_signature(params->key, &sig); diff --git a/include/linux/keyctl.h b/include/linux/keyctl.h index 5b79847207ef..970c7bed3082 100644 --- a/include/linux/keyctl.h +++ b/include/linux/keyctl.h @@ -37,6 +37,7 @@ struct kernel_pkey_params { __u32 in2_len; /* 2nd input data size (verify) */ }; enum kernel_pkey_operation op : 8; + __u32 slen; }; #endif /* __LINUX_KEYCTL_H */ diff --git a/security/keys/keyctl_pkey.c b/security/keys/keyctl_pkey.c index 5de0d599a274..b54a021e16b1 100644 --- a/security/keys/keyctl_pkey.c +++ b/security/keys/keyctl_pkey.c @@ -24,11 +24,13 @@ enum { Opt_err, Opt_enc, /* "enc=<encoding>" eg. "enc=oaep" */ Opt_hash, /* "hash=<digest-name>" eg. "hash=sha1" */ + Opt_slen, /* "slen=<salt-length>" eg. "slen=32" */ }; static const match_table_t param_keys = { { Opt_enc, "enc=%s" }, { Opt_hash, "hash=%s" }, + { Opt_slen, "slen=%u" }, { Opt_err, NULL } }; @@ -63,6 +65,10 @@ static int keyctl_pkey_params_parse(struct kernel_pkey_params *params) params->hash_algo = q; break; + case Opt_slen: + if (kstrtouint(q, 0, ¶ms->slen)) + return -EINVAL; + break; default: return -EINVAL; } -- 2.30.2 ^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH 18/18] keyctl_pkey: Add pkey parameter slen to pass in PSS salt length 2021-03-30 20:28 ` [PATCH 18/18] keyctl_pkey: Add pkey parameter slen to pass in PSS salt length Varad Gautam @ 2021-03-31 23:13 ` Jarkko Sakkinen 0 siblings, 0 replies; 2+ messages in thread From: Jarkko Sakkinen @ 2021-03-31 23:13 UTC (permalink / raw) To: Varad Gautam Cc: linux-crypto, David Howells, Herbert Xu, David S. Miller, James Morris, Serge E. Hallyn, open list:ASYMMETRIC KEYS, open list, open list:SECURITY SUBSYSTEM On Tue, Mar 30, 2021 at 10:28:29PM +0200, Varad Gautam wrote: > keyctl pkey_* operations accept enc and hash parameters at present. > RSASSA-PSS signatures also require passing in the signature salt > length. > > Add another parameter 'slen' to feed in salt length of a PSS > signature. > > Signed-off-by: Varad Gautam <varad.gautam@suse.com> > --- Reviewed-by: Jarkko Sakkinen <jarkko@kernel.org> /Jarkko > crypto/asymmetric_keys/asymmetric_type.c | 1 + > include/linux/keyctl.h | 1 + > security/keys/keyctl_pkey.c | 6 ++++++ > 3 files changed, 8 insertions(+) > > diff --git a/crypto/asymmetric_keys/asymmetric_type.c b/crypto/asymmetric_keys/asymmetric_type.c > index ad8af3d70ac0..eb2ef4a07f8e 100644 > --- a/crypto/asymmetric_keys/asymmetric_type.c > +++ b/crypto/asymmetric_keys/asymmetric_type.c > @@ -571,6 +571,7 @@ static int asymmetric_key_verify_signature(struct kernel_pkey_params *params, > .hash_algo = params->hash_algo, > .digest = (void *)in, > .s = (void *)in2, > + .salt_length = params->slen, > }; > > return verify_signature(params->key, &sig); > diff --git a/include/linux/keyctl.h b/include/linux/keyctl.h > index 5b79847207ef..970c7bed3082 100644 > --- a/include/linux/keyctl.h > +++ b/include/linux/keyctl.h > @@ -37,6 +37,7 @@ struct kernel_pkey_params { > __u32 in2_len; /* 2nd input data size (verify) */ > }; > enum kernel_pkey_operation op : 8; > + __u32 slen; > }; > > #endif /* __LINUX_KEYCTL_H */ > diff --git a/security/keys/keyctl_pkey.c b/security/keys/keyctl_pkey.c > index 5de0d599a274..b54a021e16b1 100644 > --- a/security/keys/keyctl_pkey.c > +++ b/security/keys/keyctl_pkey.c > @@ -24,11 +24,13 @@ enum { > Opt_err, > Opt_enc, /* "enc=<encoding>" eg. "enc=oaep" */ > Opt_hash, /* "hash=<digest-name>" eg. "hash=sha1" */ > + Opt_slen, /* "slen=<salt-length>" eg. "slen=32" */ > }; > > static const match_table_t param_keys = { > { Opt_enc, "enc=%s" }, > { Opt_hash, "hash=%s" }, > + { Opt_slen, "slen=%u" }, > { Opt_err, NULL } > }; > > @@ -63,6 +65,10 @@ static int keyctl_pkey_params_parse(struct kernel_pkey_params *params) > params->hash_algo = q; > break; > > + case Opt_slen: > + if (kstrtouint(q, 0, ¶ms->slen)) > + return -EINVAL; > + break; > default: > return -EINVAL; > } > -- > 2.30.2 > > ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-03-31 23:14 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20210330202829.4825-1-varad.gautam@suse.com>
2021-03-30 20:28 ` [PATCH 18/18] keyctl_pkey: Add pkey parameter slen to pass in PSS salt length Varad Gautam
2021-03-31 23:13 ` Jarkko Sakkinen
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).