c/mirror_zfs
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_zfs.git synced 2025-01-13 11:40:25 +03:00
Code Issues Packages Projects Releases Wiki Activity

cppcheck: (error) Uninitialized variable

Browse Source 
Resolve the following uninitialized variable warnings.  In practice
these were unreachable due to the goto.  Replacing the goto with a
return resolves the warning and yields more readable code.

[module/icp/algs/modes/ccm.c:892]: (error) Uninitialized variable: ccm_param
[module/icp/algs/modes/ccm.c:893]: (error) Uninitialized variable: ccm_param
[module/icp/algs/modes/gcm.c:564]: (error) Uninitialized variable: gcm_param
[module/icp/algs/modes/gcm.c:565]: (error) Uninitialized variable: gcm_param
[module/icp/algs/modes/gcm.c:599]: (error) Uninitialized variable: gmac_param
[module/icp/algs/modes/gcm.c:600]: (error) Uninitialized variable: gmac_param

Reviewed-by: Tony Hutter <hutter2@llnl.gov>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #9732
This commit is contained in:
Ubuntu 2019-12-13 23:56:37 +00:00 committed by Brian Behlendorf
parent b8a899e94f
commit 5215fdd43c
2 changed files with 6 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
module/icp/algs/modes/ccm.c
View File

@ -885,15 +885,13 @@ ccm_init_ctx(ccm_ctx_t *ccm_ctx, char *param, int kmflag,
ccm_ctx->ccm_flags |= CCM_MODE; ccm_ctx->ccm_flags |= CCM_MODE;
} else { } else {
rv = CRYPTO_MECHANISM_PARAM_INVALID; return (CRYPTO_MECHANISM_PARAM_INVALID);
goto out;
} }
if (ccm_init(ccm_ctx, ccm_param->nonce, ccm_param->ulNonceSize, if (ccm_init(ccm_ctx, ccm_param->nonce, ccm_param->ulNonceSize,
ccm_param->authData, ccm_param->ulAuthDataSize, block_size, ccm_param->authData, ccm_param->ulAuthDataSize, block_size,
encrypt_block, xor_block) != 0) { encrypt_block, xor_block) != 0) {
rv = CRYPTO_MECHANISM_PARAM_INVALID; return (CRYPTO_MECHANISM_PARAM_INVALID);
goto out;
} }
if (!is_encrypt_init) { if (!is_encrypt_init) {
/* allocate buffer for storing decrypted plaintext */ /* allocate buffer for storing decrypted plaintext */
@ -903,7 +901,6 @@ ccm_init_ctx(ccm_ctx_t *ccm_ctx, char *param, int kmflag,
rv = CRYPTO_HOST_MEMORY; rv = CRYPTO_HOST_MEMORY;
} }
} }
out:
return (rv); return (rv);
} }

10
module/icp/algs/modes/gcm.c
View File

@ -557,8 +557,7 @@ gcm_init_ctx(gcm_ctx_t *gcm_ctx, char *param, size_t block_size,
rv = CRYPTO_SUCCESS; rv = CRYPTO_SUCCESS;
gcm_ctx->gcm_flags |= GCM_MODE; gcm_ctx->gcm_flags |= GCM_MODE;
} else { } else {
rv = CRYPTO_MECHANISM_PARAM_INVALID; return (CRYPTO_MECHANISM_PARAM_INVALID);
goto out;
} }
if (gcm_init(gcm_ctx, gcm_param->pIv, gcm_param->ulIvLen, if (gcm_init(gcm_ctx, gcm_param->pIv, gcm_param->ulIvLen,
@ -566,7 +565,7 @@ gcm_init_ctx(gcm_ctx_t *gcm_ctx, char *param, size_t block_size,
encrypt_block, copy_block, xor_block) != 0) { encrypt_block, copy_block, xor_block) != 0) {
rv = CRYPTO_MECHANISM_PARAM_INVALID; rv = CRYPTO_MECHANISM_PARAM_INVALID;
} }
out:
return (rv); return (rv);
} }
@ -592,8 +591,7 @@ gmac_init_ctx(gcm_ctx_t *gcm_ctx, char *param, size_t block_size,
rv = CRYPTO_SUCCESS; rv = CRYPTO_SUCCESS;
gcm_ctx->gcm_flags |= GMAC_MODE; gcm_ctx->gcm_flags |= GMAC_MODE;
} else { } else {
rv = CRYPTO_MECHANISM_PARAM_INVALID; return (CRYPTO_MECHANISM_PARAM_INVALID);
goto out;
} }
if (gcm_init(gcm_ctx, gmac_param->pIv, AES_GMAC_IV_LEN, if (gcm_init(gcm_ctx, gmac_param->pIv, AES_GMAC_IV_LEN,
@ -601,7 +599,7 @@ gmac_init_ctx(gcm_ctx_t *gcm_ctx, char *param, size_t block_size,
encrypt_block, copy_block, xor_block) != 0) { encrypt_block, copy_block, xor_block) != 0) {
rv = CRYPTO_MECHANISM_PARAM_INVALID; rv = CRYPTO_MECHANISM_PARAM_INVALID;
} }
out:
return (rv); return (rv);
} }