[BugFix]: Add ruturn value check for fs_unlink

[Desc]: 1. check_storage_available should check fs_unlink's return value; 2. ta_storage_cmd_unlink should't always return TEE_SUCCESS; Signed-off-by: pengyinjie <[email protected]>
OP-TEE · May 14, 2024 · a26fbfa · a26fbfa
1 parent 54263d9
commit a26fbfa
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/ta/storage/storage.c b/ta/storage/storage.c
@@ -244,9 +244,7 @@ TEE_Result ta_storage_cmd_unlink(uint32_t param_types, TEE_Param params[4])
 			  (TEE_PARAM_TYPE_VALUE_INPUT, TEE_PARAM_TYPE_NONE,
 			   TEE_PARAM_TYPE_NONE, TEE_PARAM_TYPE_NONE));
 
-    res = TEE_CloseAndDeletePersistentObject1(o);
-
-	return res;
+	return TEE_CloseAndDeletePersistentObject1(o);
 }
 
 TEE_Result ta_storage_cmd_rename(uint32_t command, uint32_t param_types,