Fix resource leak on crash

Even though we're heading to abort(), it seems like some static
analysis checkers still think we're leaking. So.... well, let's
just make them happy and free the memory. It's harmless to do
that.

Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>

1 files changed, 5 insertions, 1 deletions

diff --git a/core/device.c b/core/device.c
index 483baec7..89e530f2 100644
--- a/core/device.c
+++ b/core/device.c
@@ -318,14 +318,18 @@ static struct dt_property *new_property(struct dt_node *node,
 					const char *name, size_t size)
 {
 	struct dt_property *p = malloc(sizeof(*p) + size);
+	char *path;
+
 	if (!p) {
 		prerror("Failed to allocate property \"%s\" for %s of %zu bytes\n",
 			name, dt_get_path(node), size);
 		abort();
 	}
 	if (dt_find_property(node, name)) {
+		path = dt_get_path(node);
 		prerror("Duplicate property \"%s\" in node %s\n",
-			name, dt_get_path(node));
+			name, path);
+		free(path);
 		abort();
 
 	}