Skip to content

Commit

Permalink
crypto: spacc - Use crypto_authenc_extractkeys
Browse files Browse the repository at this point in the history
Use the crypto_authenc_extractkeys helper rather than ad-hoc parsing.

Signed-off-by: Herbert Xu <[email protected]>
  • Loading branch information
herbertx committed Aug 24, 2024
1 parent c76c9ec commit f036dd5
Showing 1 changed file with 9 additions and 24 deletions.
33 changes: 9 additions & 24 deletions drivers/crypto/dwc-spacc/spacc_aead.c
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
#include <crypto/gcm.h>
#include <crypto/aead.h>
#include <crypto/authenc.h>
#include <linux/rtnetlink.h>
#include <crypto/scatterwalk.h>
#include <crypto/internal/aead.h>
#include <linux/platform_device.h>
Expand Down Expand Up @@ -540,15 +539,13 @@ static int spacc_aead_setkey(struct crypto_aead *tfm, const u8 *key,
{
struct spacc_crypto_ctx *ctx = crypto_aead_ctx(tfm);
const struct spacc_alg *salg = spacc_tfm_aead(&tfm->base);
struct crypto_authenc_keys authenc_keys;
struct spacc_priv *priv;
struct rtattr *rta = (void *)key;
struct crypto_authenc_key_param *param;
unsigned int authkeylen, enckeylen;
const unsigned char *authkey, *enckey;
unsigned char xcbc[64];

int err = -EINVAL;
int singlekey = 0;
int err;

/* are keylens valid? */
ctx->ctx_valid = false;
Expand All @@ -569,26 +566,14 @@ static int spacc_aead_setkey(struct crypto_aead *tfm, const u8 *key,
goto skipover;
}

if (!RTA_OK(rta, keylen) ||
rta->rta_type != CRYPTO_AUTHENC_KEYA_PARAM ||
RTA_PAYLOAD(rta) < sizeof(*param))
return -EINVAL;

param = RTA_DATA(rta);
enckeylen = be32_to_cpu(param->enckeylen);
key += RTA_ALIGN(rta->rta_len);
keylen -= RTA_ALIGN(rta->rta_len);
err = crypto_authenc_extractkeys(&authenc_keys, key, keylen);
if (err)
return err;

if (keylen < enckeylen)
return -EINVAL;

authkeylen = keylen - enckeylen;

/* enckey is at &key[authkeylen] and
* authkey is at &key[0]
*/
authkey = &key[0];
enckey = &key[authkeylen];
authkeylen = authenc_keys.authkeylen;
authkey = authenc_keys.authkey;
enckeylen = authenc_keys.enckeylen;
enckey = authenc_keys.enckey;

skipover:
/* detect RFC3686/4106 and trim from enckeylen(and copy salt..) */
Expand Down

0 comments on commit f036dd5

Please sign in to comment.