commit 9f483bb2076179dde3f1b328f2cb88427774093d
author Cedric Neveux <cedric.neveux@nxp.com> Mon Mar 04 08:58:06 2019 +0100
committer Jérôme Forissier <jerome@forissier.org> Fri Apr 03 14:49:42 2020 +0200
tree 11613602589c056d35aff087e112ab00bbfebe1d
parent 61f89d8fd89dce8b1bd430a71224d4508595cc82

Allow memrefs with size 0

  Add test 1025 verifying temporary memref of 0 bytes or NULL pointer
  buffer conforming with:
  GlobalPlatform Client API v1.0c (section 3.2.5 memory references), which
  can be found at [1].

  Link: [1] http://www.globalplatform.org/specificationsdevice.asp

Signed-off-by: Cedric Neveux <cedric.neveux@nxp.com>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Tested-by: Joakim Bech <joakim.bech@linaro.org> (QEMU)

ta/os_test/os_test.c

diff --git a/ta/os_test/os_test.c b/ta/os_test/os_test.c
index a2c745b..ccf16e3 100644
--- a/ta/os_test/os_test.c
+++ b/ta/os_test/os_test.c
@@ -1068,6 +1068,30 @@
 	return TEE_SUCCESS;
 }
 
+TEE_Result ta_entry_null_memref(uint32_t param_types, TEE_Param params[4])
+{
+	if (param_types != TEE_PARAM_TYPES(TEE_PARAM_TYPE_MEMREF_INPUT,
+					   TEE_PARAM_TYPE_MEMREF_INPUT,
+					   TEE_PARAM_TYPE_MEMREF_OUTPUT,
+					   TEE_PARAM_TYPE_MEMREF_OUTPUT))
+		return TEE_ERROR_BAD_PARAMETERS;
+
+	/*
+	 * Tests how client can provide null or non-null memref parameters
+	 * param[0] expected as a 0 byte input mapped memeref.
+	 * param[1] expected as a 0 byte input not-mapped memeref.
+	 * param[2] expected as a 0 byte output mapped memeref.
+	 * param[3] expected as a 0 byte output not-mapped memeref.
+	 */
+	if (!params[0].memref.buffer || params[0].memref.size ||
+	    params[1].memref.buffer || params[1].memref.size ||
+	    !params[2].memref.buffer || params[2].memref.size ||
+	    params[3].memref.buffer || params[3].memref.size)
+		return TEE_ERROR_BAD_PARAMETERS;
+
+	return TEE_SUCCESS;
+}
+
 TEE_Result ta_entry_call_lib(uint32_t param_types,
 			     TEE_Param params[4] __unused)
 {