From 9ae606c3535643fd9bd9daf7b7ea05dcc1fe0869 Mon Sep 17 00:00:00 2001
From: Cyrille Bagard <nocbos@gmail.com>
Date: Mon, 18 Apr 2016 22:14:03 +0200
Subject: Avoided memory corruption by skipping a call to free().
---
ChangeLog | 5 +++++
src/analysis/binary.c | 4 +++-
2 files changed, 8 insertions(+), 1 deletion(-)
diff --git a/ChangeLog b/ChangeLog
index aa1dfa4..f956e91 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+16-04-18 Cyrille Bagard <nocbos@gmail.com>
+
+ * src/analysis/binary.c:
+ Avoid memory corruption by skipping a call to free().
+
16-04-17 Cyrille Bagard <nocbos@gmail.com>
* autogen.sh:
diff --git a/src/analysis/binary.c b/src/analysis/binary.c
index c422e29..8957399 100644
--- a/src/analysis/binary.c
+++ b/src/analysis/binary.c
@@ -631,7 +631,7 @@ static bool g_loaded_binary_load_storage(GLoadedBinary *binary, xmlXPathContextP
storage_path = stradd(storage_path, "/Storage");
value = get_node_prop_value(context, storage_path, "local");
- if (value == NULL) goto glbls_features;
+ if (value == NULL) goto glbls_no_storage_config;
binary->local_storage = (strcmp(value, "true") == 0);
@@ -711,6 +711,8 @@ static bool g_loaded_binary_load_storage(GLoadedBinary *binary, xmlXPathContextP
}
+ glbls_no_storage_config:
+
free(storage_path);
return result;
--
cgit v0.11.2-87-g4458