Skip to content

Commit 102d3dc

Browse files
bmolodanclaude
authored andcommitted
test: remove temporary TA-pointer access-rights negative test
The CONFIG_APPS_ACCESS_RIGHTS_TEST app was a throwaway negative test for the guard added in the previous commit. The guard is now verified (host logic tests + on-target mps2 AN505 QEMU run, all cases rejected/accepted as expected), so drop the app and its build/menu wiring. The core guard in tee_svc.c / tee_svc_cryp.c is unaffected. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 5a63e65 commit 102d3dc

4 files changed

Lines changed: 0 additions & 95 deletions

File tree

apps/Kconfig

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -34,16 +34,6 @@ config APPS_TEST
3434
---help---
3535
"GP TEE Client API and TEE Internal API test suite."
3636

37-
config APPS_ACCESS_RIGHTS_TEST
38-
bool "TA->secure-kernel pointer-access guard test (temporary)"
39-
default n
40-
---help---
41-
"Temporary negative test that drives tee_mmu_check_access_rights()
42-
with malformed TA-supplied pointer ranges (NULL+len, wrapping range)
43-
and checks that they are rejected with TEE_ERROR_ACCESS_DENIED while a
44-
valid buffer is accepted. Remove once the access-rights fix is
45-
verified."
46-
4737
comment "H/W Security exception:"
4838

4939
config APPS_HW_SECURITY_EXCEPTION_EXAMPLE

apps/access_rights_test/access_rights_test.c

Lines changed: 0 additions & 73 deletions
This file was deleted.

arch/cortex-m33/mps2/src/mps2_an505_qemu/secure/Make.defs

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,3 @@ endif
6868
ifeq ($(CONFIG_APPS_TEST),y)
6969
CHIP_CSRCS_S += ../../../../../../apps/test/ta/test_ta.c
7070
endif
71-
72-
ifeq ($(CONFIG_APPS_ACCESS_RIGHTS_TEST),y)
73-
CHIP_CSRCS_S += ../../../../../../apps/access_rights_test/access_rights_test.c
74-
endif

arch/cortex-m33/mps2/src/mps2_an505_qemu/secure/main.c

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -81,10 +81,6 @@ void menu_security_exception_example(void);
8181

8282
extern TEE_Result tee_cryp_init(void);
8383

84-
#ifdef CONFIG_APPS_ACCESS_RIGHTS_TEST
85-
extern void tee_access_rights_test(void);
86-
#endif
87-
8884
///* Private Functions. */
8985
//
9086
///**
@@ -689,10 +685,6 @@ int main(void)
689685
#endif
690686

691687
tee_cryp_init();
692-
693-
#ifdef CONFIG_APPS_ACCESS_RIGHTS_TEST
694-
tee_access_rights_test();
695-
#endif
696688
//
697689
// Boot_Init(CONFIG_START_ADDRESS_BL33);
698690

0 commit comments

Comments
 (0)