regression 1014: fix return status condition
When passing an invalid reference to the TA, we expect the TEE to catch
the error and return either TEE_ERROR_GENERIC or
TEE_ERROR_BAD_PARAMETERS (not any other unrelated error). The condition
on the return code is not written correctly, fix it.
Signed-off-by: Jerome Forissier <jerome@forissier.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Sumit Garg <sumit.garg@linaro.org>
diff --git a/host/xtest/sdp_basic.c b/host/xtest/sdp_basic.c
index 71fd018..b539dc7 100644
--- a/host/xtest/sdp_basic.c
+++ b/host/xtest/sdp_basic.c
@@ -570,9 +570,9 @@
* result is.
*/
if ((valid_ref && orig != TEEC_ORIGIN_TRUSTED_APP) ||
- (!valid_ref && orig != TEEC_ORIGIN_COMMS &&
- (teerc != TEEC_ERROR_GENERIC ||
- teerc != TEEC_ERROR_BAD_PARAMETERS)))
+ (!valid_ref && ((orig != TEEC_ORIGIN_COMMS) ||
+ (teerc != TEEC_ERROR_GENERIC &&
+ teerc != TEEC_ERROR_BAD_PARAMETERS))))
goto error;
verbose("Out of bounds memref test successful:\n");