Commit 5319216d authored by Andrzej Zaborowski's avatar Andrzej Zaborowski Committed by Herbert Xu

crypto: rsa-pkcs1pad - don't allocate buffer on stack

Avoid the s390 compile "warning: 'pkcs1pad_encrypt_sign_complete'
uses dynamic stack allocation" reported by kbuild test robot.  Don't
use a flat zero-filled buffer, instead zero the contents of the SGL.
Signed-off-by: default avatarAndrew Zaborowski <andrew.zaborowski@intel.com>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent 16f080aa
...@@ -110,21 +110,32 @@ static int pkcs1pad_encrypt_sign_complete(struct akcipher_request *req, int err) ...@@ -110,21 +110,32 @@ static int pkcs1pad_encrypt_sign_complete(struct akcipher_request *req, int err)
struct crypto_akcipher *tfm = crypto_akcipher_reqtfm(req); struct crypto_akcipher *tfm = crypto_akcipher_reqtfm(req);
struct pkcs1pad_ctx *ctx = akcipher_tfm_ctx(tfm); struct pkcs1pad_ctx *ctx = akcipher_tfm_ctx(tfm);
struct pkcs1pad_request *req_ctx = akcipher_request_ctx(req); struct pkcs1pad_request *req_ctx = akcipher_request_ctx(req);
uint8_t zeros[ctx->key_size - req_ctx->child_req.dst_len]; size_t pad_len = ctx->key_size - req_ctx->child_req.dst_len;
size_t chunk_len, pad_left;
struct sg_mapping_iter miter;
if (!err) { if (!err) {
if (req_ctx->child_req.dst_len < ctx->key_size) { if (pad_len) {
memset(zeros, 0, sizeof(zeros)); sg_miter_start(&miter, req->dst,
sg_copy_from_buffer(req->dst, sg_nents_for_len(req->dst, pad_len),
sg_nents_for_len(req->dst, SG_MITER_ATOMIC | SG_MITER_TO_SG);
sizeof(zeros)),
zeros, sizeof(zeros)); pad_left = pad_len;
while (pad_left) {
sg_miter_next(&miter);
chunk_len = min(miter.length, pad_left);
memset(miter.addr, 0, chunk_len);
pad_left -= chunk_len;
}
sg_miter_stop(&miter);
} }
sg_pcopy_from_buffer(req->dst, sg_pcopy_from_buffer(req->dst,
sg_nents_for_len(req->dst, ctx->key_size), sg_nents_for_len(req->dst, ctx->key_size),
req_ctx->out_buf, req_ctx->child_req.dst_len, req_ctx->out_buf, req_ctx->child_req.dst_len,
sizeof(zeros)); pad_len);
} }
req->dst_len = ctx->key_size; req->dst_len = ctx->key_size;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment