commit c52c23783d82311db2292a35422574e1e38515ed
author Ramesh Masavarapu <rameshm@codeaurora.org> Thu Oct 27 07:35:56 2011 -0700
committer Ramesh Masavarapu <rameshm@codeaurora.org> Fri Oct 28 13:19:14 2011 -0700
tree 62acc3898303b77c782314db06680f6e459b4f4d
parent 804c6b5a2672e2603d90ec6e1861a615b8ab0157

crypto: Added checks for initializations and buffer overflow.

Added checks for
 -Initializations of alg and mode values in qcedev.
 -Buffer overflow in qcrypto.

CRs-fixed: 313705
Change-Id: I20e285679af4967f989427ea50eb5a287df28cca
Signed-off-by: Ramesh Masavarapu <rameshm@codeaurora.org>

drivers/crypto/msm/qcedev.c

diff --git a/drivers/crypto/msm/qcedev.c b/drivers/crypto/msm/qcedev.c
index 405d021..3ae1647 100644
--- a/drivers/crypto/msm/qcedev.c
+++ b/drivers/crypto/msm/qcedev.c
@@ -439,7 +439,7 @@
 		creq.alg = CIPHER_ALG_AES;
 		break;
 	default:
-		break;
+		return -EINVAL;
 	};
 
 	switch (qcedev_areq->cipher_op_req.mode) {
@@ -458,7 +458,7 @@
 		creq.mode = QCE_MODE_XTS;
 		break;
 	default:
-		break;
+		return -EINVAL;
 	};
 
 	if ((creq.alg == CIPHER_ALG_AES) &&