From ade05a08c4fbe7ad9b5220eeafa5aa6a2dcf7e5e Mon Sep 17 00:00:00 2001
From: Morris Jette <jette@schedmd.com>
Date: Thu, 29 Sep 2016 13:51:44 -0600
Subject: [PATCH] Fix memory leak reported by CLANG
---
src/slurmd/common/slurmstepd_init.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/slurmd/common/slurmstepd_init.c b/src/slurmd/common/slurmstepd_init.c
index 5a72093081b..6dffb83a733 100644
--- a/src/slurmd/common/slurmstepd_init.c
+++ b/src/slurmd/common/slurmstepd_init.c
@@ -86,11 +86,12 @@ extern int unpack_slurmd_conf_lite_no_alloc(slurmd_conf_t *conf, Buf buffer)
{
uint32_t uint32_tmp;
uint16_t protocol_version;
- char *ver_str;
+ char *ver_str = NULL;
safe_unpackstr_xmalloc(&ver_str, &uint32_tmp, buffer);
if (ver_str && !xstrcmp(ver_str, PROTOCOL_VERSION)) {
safe_unpack16(&protocol_version, buffer);
+ xfree(ver_str);
} else {
/* Kludge to add protocol version in existing data structure:
* use "hostname" as a version marker. Eliminate this in
--
GitLab