Skip to contentSkip to conversation

crypto: sa2ul - Remove unused fields

PATCH In review Eric Biggers 2 participants 2 messages Oct 2 – Oct 8 Crypto API

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 crypto: sa2ul - Remove unused fields

Remove the shash, authkey, and auth fields from sa_tfm_ctx since they have no purpose and create confusion with fields that actually matter.

---
 drivers/crypto/sa2ul.c | 9 ---------
 drivers/crypto/sa2ul.h | 3 ---
 2 files changed, 12 deletions(-)
drivers/crypto/sa2ul.c +0 -9
index b014350a3da0..4557eae94164 100644
@@ -1388,7 +1388,6 @@ static int sa_sha_run(struct ahash_request *req)
 
 static int sa_sha_setup(struct sa_tfm_ctx *ctx, struct  algo_data *ad)
 {
-	int bs = crypto_shash_blocksize(ctx->shash);
 	int cmdl_len;
 	struct sa_cmdl_cfg cfg;
 
@@ -1396,7 +1395,6 @@ static int sa_sha_setup(struct sa_tfm_ctx *ctx, struct  algo_data *ad)
 	ad->auth_eng.eng_id = SA_ENG_ID_AM1;
 	ad->auth_eng.sc_size = SA_CTX_AUTH_TYPE2_SZ;
 
-	memset(ctx->authkey, 0, bs);
 	memset(&cfg, 0, sizeof(cfg));
 	cfg.aalg = ad->aalg_id;
 	cfg.enc_eng_id = ad->enc_eng.eng_id;
@@ -1439,12 +1437,6 @@ static int sa_sha_cra_init_alg(struct crypto_tfm *tfm, const char *alg_base)
 		return ret;
 
 	if (alg_base) {
-		ctx->shash = crypto_alloc_shash(alg_base, 0, 0);
-		if (IS_ERR(ctx->shash)) {
-			dev_err(sa_k3_dev, "base driver %s couldn't be loaded\n",
-				alg_base);
-			return PTR_ERR(ctx->shash);
-		}
 		/* for fallback */
 		ctx->fallback.ahash =
 			crypto_alloc_ahash(alg_base, 0, CRYPTO_ALG_ASYNC);
@@ -1599,7 +1591,6 @@ static void sa_sha_cra_exit(struct crypto_tfm *tfm)
 	if (crypto_tfm_alg_type(tfm) == CRYPTO_ALG_TYPE_AHASH)
 		sa_free_ctx_info(&ctx->enc, data);
 
-	crypto_free_shash(ctx->shash);
 	crypto_free_ahash(ctx->fallback.ahash);
 }
 
drivers/crypto/sa2ul.h +0 -3
index 12c17a68d350..211ef4e40f22 100644
@@ -307,12 +307,9 @@ struct sa_tfm_ctx {
 	struct sa_crypto_data *dev_data;
 	struct sa_ctx_info enc;
 	struct sa_ctx_info dec;
-	struct sa_ctx_info auth;
 	int keylen;
 	int iv_idx;
 	u32 key[AES_KEYSIZE_256 / sizeof(u32)];
-	u8 authkey[SHA512_BLOCK_SIZE];
-	struct crypto_shash	*shash;
 	/* for fallback */
 	union {
 		struct crypto_skcipher		*skcipher;

base-commit: 4c3ca7c8c152278332590bfd3df1b0f086364759

Signature · 1 line
2.56.0
maintainer
#1 lore raw
Quoted 8 lines · Eric Biggers

Remove the shash, authkey, and auth fields from sa_tfm_ctx since they have no purpose and create confusion with fields that actually matter.

---
 drivers/crypto/sa2ul.c | 9 ---------
 drivers/crypto/sa2ul.h | 3 ---
 2 files changed, 12 deletions(-)

Patch applied. Thanks.

Signature · 3 lines
Email: Herbert Xu <[email protected]>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt