diff options
author | Cyrille Bagard <nocbos@gmail.com> | 2016-04-18 20:14:03 (GMT) |
---|---|---|
committer | Cyrille Bagard <nocbos@gmail.com> | 2016-04-18 20:14:03 (GMT) |
commit | 9ae606c3535643fd9bd9daf7b7ea05dcc1fe0869 (patch) | |
tree | 159eab714d5481614c20753926f73ad0a6c07880 /src | |
parent | 412d8e9df853ae3a6349d38163a1570dfa04434a (diff) |
Avoided memory corruption by skipping a call to free().
Diffstat (limited to 'src')
-rw-r--r-- | src/analysis/binary.c | 4 |
1 files changed, 3 insertions, 1 deletions
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; |