1From c2431936b80e2064d52a380c715cd3260ce54d54 Mon Sep 17 00:00:00 2001
2From: Jerome Forissier <jerome@forissier.org>
3Date: Fri, 23 Apr 2021 09:25:55 +0200
4Subject: [PATCH] TTA_TCF: remove TEE_MemMove() from
5 CmdTEEGetPropertyAsXXX_fromEnum()
6
7Patch 0006-TTA_TCF-fix-CmdTEEGetPropertyA-_withoutEnum.patch introduced
8a TEE_MemMove() call in CmdTEEGetPropertyAsXXX_fromEnum(). This call is
9wrong, because in this function pParams[1] is always of type
10TEE_PARAM_TYPE_VALUE_INPUT (in other words, not a memory reference).
11
12Fixes: https://github.com/OP-TEE/optee_os/issues/4561
13Signed-off-by: Jerome Forissier <jerome@forissier.org>
14---
15 .../TTA_TCF/TTA_TCF/code_files/TTA_TCF.c | 3 ---
16 1 file changed, 3 deletions(-)
17
18diff --git a/TTAs_Internal_API_1_1_1/TTA_TCF/TTA_TCF/code_files/TTA_TCF.c b/TTAs_Internal_API_1_1_1/TTA_TCF/TTA_TCF/code_files/TTA_TCF.c
19index 14e642c..0c7c743 100644
20--- a/TTAs_Internal_API_1_1_1/TTA_TCF/TTA_TCF/code_files/TTA_TCF.c
21+++ b/TTAs_Internal_API_1_1_1/TTA_TCF/TTA_TCF/code_files/TTA_TCF.c
22@@ -666,9 +666,6 @@ TEE_Result CmdTEEGetPropertyAsXXX_fromEnum(
23 	TEE_PropSetHandle nPropSet;
24 	char pPropName[PROPERTY_NAME_MAX_SIZE];
25 	uint32_t nPropNameSize = 0;
26-	TEE_MemMove(pPropName, pParams[1].memref.buffer,
27-		    pParams[1].memref.size);
28-	pPropName[pParams[1].memref.size] = 0;
29 	char pOutputString1[PROPERTY_OUTPUT_STRING_MAX_SIZE], pOutputString2[PROPERTY_OUTPUT_STRING_MAX_SIZE];
30 	uint32_t nOutputString1Length = 0;
31 	uint32_t nOutputString2Length = 0;
32--
332.27.0
34
35