tree: Fix clearing application strings

Freeing strings without clearing to NULL may potentially lead
to double-free later when freeing the tree structs.

Signed-off-by: Tomas Bzatek <tbzatek@redhat.com>

diff --git a/src/nvme/tree.c b/src/nvme/tree.c
index 5636aa1809ec6a3358e527110deba6e67288e389..3c8e75253c2eaae6718c28732406e4f5aa25f53e 100644 (file)
--- a/src/nvme/tree.c
+++ b/src/nvme/tree.c
@@ -261,8 +261,10 @@ const char *nvme_root_get_application(nvme_root_t r)
 
 void nvme_root_set_application(nvme_root_t r, const char *a)
 {
-       if (r->application)
+       if (r->application) {
                free(r->application);
+               r->application = NULL;
+       }
        if (a)
                r->application = strdup(a);
 }
@@ -404,8 +406,10 @@ const char *nvme_subsystem_get_application(nvme_subsystem_t s)
 
 void nvme_subsystem_set_application(nvme_subsystem_t s, const char *a)
 {
-       if (s->application)
+       if (s->application) {
                free(s->application);
+               s->application = NULL;
+       }
        if (a)
                s->application = strdup(a);
 }