ArmPkg: Fix Ecc error 3002 in OpteeLib
This patch fixes the following Ecc reported error: Non-Boolean comparisons should use a compare operator (==, !=, >, < >=, <=) Signed-off-by: Pierre Gondois <Pierre.Gondois@arm.com> Reviewed-by: Ard Biesheuvel <ard.biesheuvel@arm.com>
This commit is contained in:
		
				
					committed by
					
						![mergify[bot]](/avatar/e3df20cd7a67969c41a65f03bea54961?size=40) mergify[bot]
						mergify[bot]
					
				
			
			
				
	
			
			
			
						parent
						
							53aabb978e
						
					
				
				
					commit
					d61b2e28e6
				
			| @@ -3,6 +3,7 @@ | |||||||
|   secure monitor calls. |   secure monitor calls. | ||||||
|  |  | ||||||
|   Copyright (c) 2018, Linaro Ltd. All rights reserved.<BR> |   Copyright (c) 2018, Linaro Ltd. All rights reserved.<BR> | ||||||
|  |   Copyright (c) 2021, Arm Limited. All rights reserved.<BR> | ||||||
|  |  | ||||||
|   SPDX-License-Identifier: BSD-2-Clause-Patent |   SPDX-License-Identifier: BSD-2-Clause-Patent | ||||||
|  |  | ||||||
| @@ -227,7 +228,7 @@ OpteeOpenSession ( | |||||||
|  |  | ||||||
|   MessageArg->NumParams = 2; |   MessageArg->NumParams = 2; | ||||||
|  |  | ||||||
|   if (OpteeCallWithArg ((UINTN)MessageArg)) { |   if (OpteeCallWithArg ((UINTN)MessageArg) != 0) { | ||||||
|     MessageArg->Return = OPTEE_ERROR_COMMUNICATION; |     MessageArg->Return = OPTEE_ERROR_COMMUNICATION; | ||||||
|     MessageArg->ReturnOrigin = OPTEE_ORIGIN_COMMUNICATION; |     MessageArg->ReturnOrigin = OPTEE_ORIGIN_COMMUNICATION; | ||||||
|   } |   } | ||||||
| @@ -431,7 +432,7 @@ OpteeInvokeFunction ( | |||||||
|  |  | ||||||
|   MessageArg->NumParams = OPTEE_MAX_CALL_PARAMS; |   MessageArg->NumParams = OPTEE_MAX_CALL_PARAMS; | ||||||
|  |  | ||||||
|   if (OpteeCallWithArg ((UINTN)MessageArg)) { |   if (OpteeCallWithArg ((UINTN)MessageArg) != 0) { | ||||||
|     MessageArg->Return = OPTEE_ERROR_COMMUNICATION; |     MessageArg->Return = OPTEE_ERROR_COMMUNICATION; | ||||||
|     MessageArg->ReturnOrigin = OPTEE_ORIGIN_COMMUNICATION; |     MessageArg->ReturnOrigin = OPTEE_ORIGIN_COMMUNICATION; | ||||||
|   } |   } | ||||||
| @@ -440,7 +441,7 @@ OpteeInvokeFunction ( | |||||||
|         InvokeFunctionArg->Params, |         InvokeFunctionArg->Params, | ||||||
|         OPTEE_MAX_CALL_PARAMS, |         OPTEE_MAX_CALL_PARAMS, | ||||||
|         MessageArg->Params |         MessageArg->Params | ||||||
|         )) { |         ) != 0) { | ||||||
|     MessageArg->Return = OPTEE_ERROR_COMMUNICATION; |     MessageArg->Return = OPTEE_ERROR_COMMUNICATION; | ||||||
|     MessageArg->ReturnOrigin = OPTEE_ORIGIN_COMMUNICATION; |     MessageArg->ReturnOrigin = OPTEE_ORIGIN_COMMUNICATION; | ||||||
|   } |   } | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user