Skip to content

Commit 8185905

Browse files
committed
Fix cfstore_test_delete_all by removing swap
Remove the handle swap in cfstore_test_delete_all. This prevents a deleted handle from being used.
1 parent 6a31ffb commit 8185905

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

features/storage/FEATURE_STORAGE/cfstore/source/cfstore_test.c

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -374,7 +374,11 @@ int32_t cfstore_test_delete_all(void)
374374
CFSTORE_ERRLOG("%s:Error: failed to delete key_name=%s, len=%d\r\n", __func__, key_name, (int) len);
375375
return ret;
376376
}
377-
CFSTORE_HANDLE_SWAP(prev, next);
377+
ret = drv->Close(next);
378+
if(ret < ARM_DRIVER_OK){
379+
CFSTORE_ERRLOG("%s:Error: failed to close key_name=%s, len=%d\r\n", __func__, key_name, (int) len);
380+
return ret;
381+
}
378382
}
379383
if(ret == ARM_CFSTORE_DRIVER_ERROR_KEY_NOT_FOUND) {
380384
/* as expected, no more keys have been found by the Find()*/

0 commit comments

Comments
 (0)