From b5f12bc0f8717f5c91070a6615b9597d675a3a6c Mon Sep 17 00:00:00 2001
From: Jim McDonough <jmcd@samba.org>
Date: Fri, 10 Mar 2006 14:09:34 +0000
Subject: r14150: Fix coverity #118: not freeing alloc'ed storage returned from
 sstring_sub(). (This used to be commit
 6ff849f35ae3394d6557f79c73b0fe54fbb86d0f)

---
 source3/utils/net_rpc_samsync.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

(limited to 'source3/utils')

diff --git a/source3/utils/net_rpc_samsync.c b/source3/utils/net_rpc_samsync.c
index 5ee0b70278..2c2ffbe6f0 100644
--- a/source3/utils/net_rpc_samsync.c
+++ b/source3/utils/net_rpc_samsync.c
@@ -1309,7 +1309,9 @@ static NTSTATUS populate_ldap_for_ldif(fstring sid, const char *suffix, const ch
 	fflush(add_fd);
 
 	/* Deallocate memory, and return */
-	if (suffix_attr != NULL) SAFE_FREE(suffix_attr);
+	SAFE_FREE(suffix_attr);
+	SAFE_FREE(user_attr);
+	SAFE_FREE(group_attr);
 	return NT_STATUS_OK;
 }
 
-- 
cgit