Fix wrong null terminated list deallocation

Fixed wrong iteration on null terminated list in deallocating function.

Change-Id: I1f344cbfdcf775aedd3f7e0eb00518cfec423bc2

diff --git a/src/admin/api/admin-api.cpp b/src/admin/api/admin-api.cpp
index c2a5212..55fc4db 100644 (file)
--- a/src/admin/api/admin-api.cpp
+++ b/src/admin/api/admin-api.cpp
@@ -233,14 +233,13 @@ static int createPoliciesArray(const char *bucket,
         return CYNARA_API_OUT_OF_MEMORY;
     std::unique_ptr<Elem*, std::function<void(Elem**)>> plumber(tab,
         [](Elem **tab) {
-            Elem *elem = *tab;
-            while (elem) {
-                free(elem->bucket);
-                free(elem->client);
-                free(elem->user);
-                free(elem->privilege);
-                free(elem->result_extra);
-                free(elem++);
+            for (int i = 0; tab[i] != nullptr; i++) {
+                free(tab[i]->bucket);
+                free(tab[i]->client);
+                free(tab[i]->user);
+                free(tab[i]->privilege);
+                free(tab[i]->result_extra);
+                free(tab[i]);
             }
             free(tab);
         });