Commit f2ac6774 authored by Horia Geantă's avatar Horia Geantă Committed by Herbert Xu
Browse files

crypto: caam - remove unneeded edesc zeroization 



Extended descriptor allocation has been changed by
commit dde20ae9 ("crypto: caam - Change kmalloc to kzalloc to avoid residual data")
to provide zeroized memory, meaning we no longer have to sanitize
its members - edesc->src_nents and edesc->dst_dma.

Signed-off-by: default avatarHoria Geantă <horia.geanta@nxp.com>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent 7a373fd7

drivers/crypto/caam/caamhash.c

+0 −4
Original line number Diff line number Diff line
@@ -871,7 +871,6 @@ static int ahash_final_ctx(struct ahash_request *req) 
	desc = edesc->hw_desc;



	edesc->sec4_sg_bytes = sec4_sg_bytes;

	edesc->src_nents = 0;



	ret = ctx_map_to_sec4_sg(desc, jrdev, state, ctx->ctx_len,

				 edesc->sec4_sg, DMA_TO_DEVICE);
@@ -1123,7 +1122,6 @@ static int ahash_final_no_ctx(struct ahash_request *req) 
		dev_err(jrdev, "unable to map dst\n");

		goto unmap;

	}

	edesc->src_nents = 0;



#ifdef DEBUG

	print_hex_dump(KERN_ERR, "jobdesc@"__stringify(__LINE__)": ",
@@ -1205,7 +1203,6 @@ static int ahash_update_no_ctx(struct ahash_request *req) 


		edesc->src_nents = src_nents;

		edesc->sec4_sg_bytes = sec4_sg_bytes;

		edesc->dst_dma = 0;



		ret = buf_map_to_sec4_sg(jrdev, edesc->sec4_sg, state);

		if (ret)
@@ -1417,7 +1414,6 @@ static int ahash_update_first(struct ahash_request *req) 
		}



		edesc->src_nents = src_nents;

		edesc->dst_dma = 0;



		ret = ahash_edesc_add_src(ctx, edesc, req, mapped_nents, 0, 0,

					  to_hash);