aboutsummaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authormoneromooo-monero <moneromooo-monero@users.noreply.github.com>2018-11-05 11:15:02 +0000
committermoneromooo-monero <moneromooo-monero@users.noreply.github.com>2018-11-05 11:15:39 +0000
commitfa3f756bfb4f0595bd2dbe0e22a9d2f00f4a0ef5 (patch)
tree901dd758a1781125c87019267b1b0aa2c6798bbc /tests
parentMerge pull request #4702 (diff)
downloadmonero-fa3f756bfb4f0595bd2dbe0e22a9d2f00f4a0ef5.tar.xz
unit_tests: fix leak when the test check fails
Coverity 189527
Diffstat (limited to 'tests')
-rw-r--r--tests/unit_tests/notify.cpp4
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/unit_tests/notify.cpp b/tests/unit_tests/notify.cpp
index 4daeeddee..edc4eabdf 100644
--- a/tests/unit_tests/notify.cpp
+++ b/tests/unit_tests/notify.cpp
@@ -49,7 +49,8 @@ TEST(notify, works)
tmp = "/tmp";
static const char *filename = "monero-notify-unit-test-XXXXXX";
const size_t len = strlen(tmp) + 1 + strlen(filename);
- char *name_template = (char*)malloc(len + 1);
+ std::unique_ptr<char[]> name_template_((char*)malloc(len + 1));
+ char *name_template = name_template_.get();
ASSERT_TRUE(name_template != NULL);
snprintf(name_template, len + 1, "%s/%s", tmp, filename);
int fd = mkstemp(name_template);
@@ -75,5 +76,4 @@ TEST(notify, works)
ASSERT_TRUE(s == "1111111111111111111111111111111111111111111111111111111111111111");
boost::filesystem::remove(name_template);
- free(name_template);
}