diff options
| author | Jim McDonough <jmcd@samba.org> | 2006-03-10 14:09:34 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 11:15:14 -0500 |
| commit | b5f12bc0f8717f5c91070a6615b9597d675a3a6c (patch) | |
| tree | 4e638c72223cd537189297fc80318425a3c3af0d /source3/utils | |
| parent | 14e904fe868eb61e4ce39db14d156f384d086743 (diff) | |
| download | samba-b5f12bc0f8717f5c91070a6615b9597d675a3a6c.tar.gz samba-b5f12bc0f8717f5c91070a6615b9597d675a3a6c.tar.bz2 samba-b5f12bc0f8717f5c91070a6615b9597d675a3a6c.zip | |
r14150: Fix coverity #118: not freeing alloc'ed storage returned from
sstring_sub().
(This used to be commit 6ff849f35ae3394d6557f79c73b0fe54fbb86d0f)
Diffstat (limited to 'source3/utils')
| -rw-r--r-- | source3/utils/net_rpc_samsync.c | 4 |
1 files changed, 3 insertions, 1 deletions
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; } |