summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJenkins <jenkins@review.openstack.org>2016-12-07 21:51:15 +0000
committerGerrit Code Review <review@openstack.org>2016-12-07 21:51:15 +0000
commitb1156356baf8b6b011096a5d75dbc50570ee323b (patch)
treed8028ffc5cfb8e7344e115940bd544c1a4a98d67
parentca4d2dd7c3663ff675cd868ad57821f1c839c2cf (diff)
parent288d1e07de0f3b525ea7ae5b7c7fd67d6187756f (diff)
downloadpyeclib-b1156356baf8b6b011096a5d75dbc50570ee323b.tar.gz
Merge "Fix checksum memory leak"
-rw-r--r--src/c/pyeclib_c/pyeclib_c.c1
-rw-r--r--test/test_pyeclib_api.py2
2 files changed, 2 insertions, 1 deletions
diff --git a/src/c/pyeclib_c/pyeclib_c.c b/src/c/pyeclib_c/pyeclib_c.c
index 40acd39..3479733 100644
--- a/src/c/pyeclib_c/pyeclib_c.c
+++ b/src/c/pyeclib_c/pyeclib_c.c
@@ -1047,6 +1047,7 @@ fragment_metadata_to_dict(fragment_metadata_t *fragment_metadata)
"chksum_mismatch", fragment_metadata->chksum_mismatch,
"backend_id", backend_id_str,
"backend_version", fragment_metadata->backend_version);
+ encoded_chksum = check_and_free_buffer(encoded_chksum);
if (metadata_dict == NULL) {
pyeclib_c_seterr(-ENOMEM, "fragment_metadata_to_dict ERROR: ");
return NULL;
diff --git a/test/test_pyeclib_api.py b/test/test_pyeclib_api.py
index e50758a..6716ab1 100644
--- a/test/test_pyeclib_api.py
+++ b/test/test_pyeclib_api.py
@@ -715,7 +715,7 @@ class TestPyECLibDriver(unittest.TestCase):
# 1. Prepare the expected memory allocation
encoded = ec_driver.encode(b'aaa')
ec_driver.get_metadata(encoded[0], formatted=True)
- loop_range = range(1000)
+ loop_range = range(400000)
# 2. Get current memory usage
baseline_usage = resource.getrusage(resource.RUSAGE_SELF)[2]