From b7ea69265dd945ff08a829c3f9918ea0190b148a Mon Sep 17 00:00:00 2001 From: Danny Al-Gaaf Date: Tue, 24 Feb 2015 09:21:17 +0100 Subject: [PATCH] libradosstriper/striping.cc: fix resource leak Fix for: CID 1221525 (#1 of 1): Resource leak (RESOURCE_LEAK) leaked_storage: Failing to save or free storage allocated by this->getObjName(soid, 0UL) leaks it. Signed-off-by: Danny Al-Gaaf --- src/test/libradosstriper/striping.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/test/libradosstriper/striping.cc b/src/test/libradosstriper/striping.cc index 9b26329c6fcc..404256637f7e 100644 --- a/src/test/libradosstriper/striping.cc +++ b/src/test/libradosstriper/striping.cc @@ -230,7 +230,7 @@ TEST_P(StriperTestRT, StripedRoundtrip) { // recreate object ASSERT_EQ(0, striper.write(soid, bl1, testData.size*2, 0)); // remove the object size attribute from the striped object - std::string firstOid = getObjName(soid, 0); + char* firstOid = getObjName(soid, 0); ASSERT_EQ(0, ioctx.rmxattr(firstOid, "striper.size")); // check that stat fails uint64_t size; @@ -244,6 +244,7 @@ TEST_P(StriperTestRT, StripedRoundtrip) { object_nb++) { ASSERT_EQ(-ENOENT, ioctx.stat(getObjName(soid, object_nb), &size, &mtime)); } + free(firstOid); } } -- 2.47.3