feedc0de/esp-idf
1
0
Fork 0
forked from espressif/esp-idf
Code Pull Requests Activity

Merge branch 'fix/coverity_use_after_free_partition.c' into 'master'

Browse Source 
fix(partition): Simplify macro loop in esp_partition_unload_all, should fix Coverity problem

Closes IDF-9837

See merge request espressif/esp-idf!30853
This commit is contained in:
Adam Múdry
2024-05-23 23:08:11 +08:00
parent 83d58f9b59 c2ac9bad62
commit 050db26419
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
components/esp_partition/partition.c
View File

@@ -238,7 +238,8 @@ void esp_partition_unload_all(void)
partition_list_item_t *it; partition_list_item_t *it;
partition_list_item_t *tmp; partition_list_item_t *tmp;
SLIST_FOREACH_SAFE(it, &s_partition_list, next, tmp) { SLIST_FOREACH_SAFE(it, &s_partition_list, next, tmp) {
SLIST_REMOVE(&s_partition_list, it, partition_list_item_, next); // Remove current head from the list and free it, new head is the next element
SLIST_REMOVE_HEAD(&s_partition_list, next);
free(it); free(it);
} }
_lock_release(&s_partition_list_lock); _lock_release(&s_partition_list_lock);