core: tee_svc_cryp: report RSAES_PKCS1_OAEP_MGF1 bad hash ID

Fixes syscall_asymm_operate() to report inconsistent hash algorithm specified as attribute for TEE_ALG_RSAES_PKCS1_OAEP_MGF1_* operations as OP-TEE only supports the hash predefined for the request algorithm TEE_ALG_RSAES_PKCS1_OAEP_MGF1_xxx. Link: #6143 Reviewed-by: Jens Wiklander <[email protected]> Signed-off-by: Etienne Carriere <[email protected]>
OP-TEE · Jun 28, 2023 · 9ab92de · 9ab92de
1 parent 9216643
commit 9ab92de
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/core/tee/tee_svc_cryp.c b/core/tee/tee_svc_cryp.c
@@ -4257,14 +4257,14 @@ TEE_Result syscall_asymm_operate(unsigned long state,
 				if (params[n].content.ref.length !=
 				    sizeof(hash)) {
 					res = TEE_ERROR_BAD_PARAMETERS;
-					break;
+					goto out;
 				}
 				memcpy(&hash, params[n].content.ref.buffer,
 				       sizeof(hash));
 				if (hash !=
 				    TEE_INTERNAL_HASH_TO_ALGO(cs->algo)) {
 					res = TEE_ERROR_NOT_SUPPORTED;
-					break;
+					goto out;
 				}
 			}
 		}