Skip to content

Commit

Permalink
Format UUID strings correctly
Browse files Browse the repository at this point in the history
Prior to this patch although GlobalPlatform specifies that TAs and such
are identified with UUIDs, we don't format them quite right when turning
them into strings. Per https://www.ietf.org/rfc/rfc4122.txt, there
should be another hyphen after the first two bytes of clockSeqAndNode.

Unfortunately, fixing this breaks compatibility between how TAs are
built, and when the OS loads them.

With this patch UUID string are formated with the additional hyphen as
for instance: f81d4fae-7dec-11d0-a765-00a0c91e6bf6

Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Fixes: OP-TEE/optee_os#857
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
  • Loading branch information
jenswi-linaro authored and Silvano di Ninno committed Dec 21, 2016
1 parent 93bbcc8 commit 2183835
Show file tree
Hide file tree
Showing 25 changed files with 25 additions and 25 deletions.
2 changes: 1 addition & 1 deletion host/xtest/xtest_1000.c
Original file line number Diff line number Diff line change
Expand Up @@ -610,7 +610,7 @@ static void uuid_to_full_name(char *buf, size_t blen, const TEEC_UUID *uuid,
bool for_write)
{
snprintf(buf, blen,
"%s/%08x-%04x-%04x-%02x%02x%02x%02x%02x%02x%02x%02x.ta",
"%s/%08x-%04x-%04x-%02x%02x-%02x%02x%02x%02x%02x%02x.ta",
for_write ? TA_TEST_DIR : TA_DIR,
uuid->timeLow, uuid->timeMid, uuid->timeHiAndVersion,
uuid->clockSeqAndNode[0], uuid->clockSeqAndNode[1],
Expand Down
2 changes: 1 addition & 1 deletion ta/aes_perf/Android.mk
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
LOCAL_PATH := $(call my-dir)

local_module := e626662e-c0e2-485c-b8c809fbce6edf3d.ta
local_module := e626662e-c0e2-485c-b8c8-09fbce6edf3d.ta
include $(BUILD_OPTEE_MK)
2 changes: 1 addition & 1 deletion ta/aes_perf/Makefile
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
BINARY = e626662e-c0e2-485c-b8c809fbce6edf3d
BINARY = e626662e-c0e2-485c-b8c8-09fbce6edf3d
include ../ta_common.mk

2 changes: 1 addition & 1 deletion ta/concurrent/Android.mk
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
LOCAL_PATH := $(call my-dir)

local_module := e13010e0-2ae1-11e5-896a0002a5d5c51b.ta
local_module := e13010e0-2ae1-11e5-896a-0002a5d5c51b.ta
include $(BUILD_OPTEE_MK)
2 changes: 1 addition & 1 deletion ta/concurrent/Makefile
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
BINARY = e13010e0-2ae1-11e5-896a0002a5d5c51b
BINARY = e13010e0-2ae1-11e5-896a-0002a5d5c51b
include ../ta_common.mk
2 changes: 1 addition & 1 deletion ta/concurrent_large/Android.mk
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
LOCAL_PATH := $(call my-dir)

local_module := 5ce0c432-0ab0-40e5-a056782ca0e6aba2.ta
local_module := 5ce0c432-0ab0-40e5-a056-782ca0e6aba2.ta
include $(BUILD_OPTEE_MK)
2 changes: 1 addition & 1 deletion ta/concurrent_large/Makefile
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
BINARY = 5ce0c432-0ab0-40e5-a056782ca0e6aba2
BINARY = 5ce0c432-0ab0-40e5-a056-782ca0e6aba2
include ../ta_common.mk
2 changes: 1 addition & 1 deletion ta/create_fail_test/Android.mk
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
LOCAL_PATH := $(call my-dir)

local_module := c3f6e2c0-3548-11e1-b86c0800200c9a66.ta
local_module := c3f6e2c0-3548-11e1-b86c-0800200c9a66.ta
include $(BUILD_OPTEE_MK)
2 changes: 1 addition & 1 deletion ta/create_fail_test/Makefile
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
BINARY = c3f6e2c0-3548-11e1-b86c0800200c9a66
BINARY = c3f6e2c0-3548-11e1-b86c-0800200c9a66
include ../ta_common.mk
2 changes: 1 addition & 1 deletion ta/crypt/Android.mk
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
LOCAL_PATH := $(call my-dir)

local_module := cb3e5ba0-adf1-11e0-998b0002a5d5c51b.ta
local_module := cb3e5ba0-adf1-11e0-998b-0002a5d5c51b.ta
include $(BUILD_OPTEE_MK)
2 changes: 1 addition & 1 deletion ta/crypt/Makefile
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
BINARY = cb3e5ba0-adf1-11e0-998b0002a5d5c51b
BINARY = cb3e5ba0-adf1-11e0-998b-0002a5d5c51b
include ../ta_common.mk
2 changes: 1 addition & 1 deletion ta/os_test/Android.mk
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
LOCAL_PATH := $(call my-dir)

local_module := 5b9e0e40-2636-11e1-ad9e0002a5d5c51b.ta
local_module := 5b9e0e40-2636-11e1-ad9e-0002a5d5c51b.ta
include $(BUILD_OPTEE_MK)
2 changes: 1 addition & 1 deletion ta/os_test/Makefile
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
BINARY = 5b9e0e40-2636-11e1-ad9e0002a5d5c51b
BINARY = 5b9e0e40-2636-11e1-ad9e-0002a5d5c51b
include ../ta_common.mk
2 changes: 1 addition & 1 deletion ta/rpc_test/Android.mk
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
LOCAL_PATH := $(call my-dir)

local_module := d17f73a0-36ef-11e1-984a0002a5d5c51b.ta
local_module := d17f73a0-36ef-11e1-984a-0002a5d5c51b.ta
include $(BUILD_OPTEE_MK)
2 changes: 1 addition & 1 deletion ta/rpc_test/Makefile
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
BINARY = d17f73a0-36ef-11e1-984a0002a5d5c51b
BINARY = d17f73a0-36ef-11e1-984a-0002a5d5c51b
include ../ta_common.mk
2 changes: 1 addition & 1 deletion ta/sha_perf/Android.mk
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
LOCAL_PATH := $(call my-dir)

local_module := 614789f2-39c0-4ebf-b23592b32ac107ed.ta
local_module := 614789f2-39c0-4ebf-b235-92b32ac107ed.ta
include $(BUILD_OPTEE_MK)
2 changes: 1 addition & 1 deletion ta/sha_perf/Makefile
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
BINARY = 614789f2-39c0-4ebf-b23592b32ac107ed
BINARY = 614789f2-39c0-4ebf-b235-92b32ac107ed
include ../ta_common.mk

2 changes: 1 addition & 1 deletion ta/sims/Android.mk
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
LOCAL_PATH := $(call my-dir)

local_module := e6a33ed4-562b-463a-bb7eff5e15a493c8.ta
local_module := e6a33ed4-562b-463a-bb7e-ff5e15a493c8.ta
include $(BUILD_OPTEE_MK)
2 changes: 1 addition & 1 deletion ta/sims/Makefile
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
BINARY = e6a33ed4-562b-463a-bb7eff5e15a493c8
BINARY = e6a33ed4-562b-463a-bb7e-ff5e15a493c8
include ../ta_common.mk
2 changes: 1 addition & 1 deletion ta/storage/Android.mk
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
LOCAL_PATH := $(call my-dir)

local_module := b689f2a7-8adf-477a-9f9932e90c0ad0a2.ta
local_module := b689f2a7-8adf-477a-9f99-32e90c0ad0a2.ta
include $(BUILD_OPTEE_MK)
2 changes: 1 addition & 1 deletion ta/storage/Makefile
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
BINARY = b689f2a7-8adf-477a-9f9932e90c0ad0a2
BINARY = b689f2a7-8adf-477a-9f99-32e90c0ad0a2
include ../ta_common.mk
2 changes: 1 addition & 1 deletion ta/storage2/Android.mk
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
LOCAL_PATH := $(call my-dir)

local_module := 731e279e-aafb-4575-a77138caa6f0cca6.ta
local_module := 731e279e-aafb-4575-a771-38caa6f0cca6.ta
include $(BUILD_OPTEE_MK)
2 changes: 1 addition & 1 deletion ta/storage2/Makefile
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
BINARY = 731e279e-aafb-4575-a77138caa6f0cca6
BINARY = 731e279e-aafb-4575-a771-38caa6f0cca6
include ../ta_common.mk
2 changes: 1 addition & 1 deletion ta/storage_benchmark/Android.mk
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
LOCAL_PATH := $(call my-dir)

local_module := f157cda0-550c-11e5-a6fa0002a5d5c51b.ta
local_module := f157cda0-550c-11e5-a6fa-0002a5d5c51b.ta
include $(BUILD_OPTEE_MK)
2 changes: 1 addition & 1 deletion ta/storage_benchmark/Makefile
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
BINARY = f157cda0-550c-11e5-a6fa0002a5d5c51b
BINARY = f157cda0-550c-11e5-a6fa-0002a5d5c51b
include ../ta_common.mk

0 comments on commit 2183835

Please sign in to comment.