Security: The f_iki allocation functions are calling f_memory_destroy() and f_memory_...

The private_f_iki_datas_adjust() should be called instead of f_memory_destroy().
The private_f_iki_datas_resize() should be called instead of f_memory_delete().

This is because the iki_datass needs the individual sets within to be deallocated.
De-allocating at the top level via the f_memory_destroy() and f_memory_delete() is very probably a memory leak.

diff --git a/level_0/f_iki/c/iki/private-data.c b/level_0/f_iki/c/iki/private-data.c
index 9be4bc31b2b7f4cf26dcf952cf01a8067b27e0fe..0a16fc6ecf006ccfb8b92e340433efe7cbd08ef4 100644 (file)
--- a/level_0/f_iki/c/iki/private-data.c
+++ b/level_0/f_iki/c/iki/private-data.c
@@ -169,11 +169,8 @@ extern "C" {
 
     for (f_array_length_t i = length; i < datass->size; ++i) {
 
-      status = f_memory_destroy(datass->array[i].size, sizeof(f_iki_datas_t), (void **) & datass->array[i].array);
+      status = private_f_iki_datas_adjust(0, &datass->array[i]);
       if (F_status_is_error(status)) return status;
-
-      datass->array[i].size = 0;
-      datass->array[i].used = 0;
     } // for
 
     status = f_memory_adjust(datass->size, length, sizeof(f_iki_datas_t), (void **) & datass->array);
@@ -196,11 +193,8 @@ extern "C" {
 
     for (f_array_length_t i = length; i < datass->size; ++i) {
 
-      status = f_memory_delete(datass->array[i].size, sizeof(f_iki_datas_t), (void **) & datass->array[i].array);
+      status = private_f_iki_datas_resize(0, &datass->array[i]);
       if (F_status_is_error(status)) return status;
-
-      datass->array[i].size = 0;
-      datass->array[i].used = 0;
     } // for
 
     status = f_memory_resize(datass->size, length, sizeof(f_iki_datas_t), (void **) & datass->array);