diff options
| author | Jim McDonough <jmcd@samba.org> | 2006-03-10 14:20:09 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 11:15:14 -0500 |
| commit | c0d4100517bc50d7f81c3e5816a08c1a8284efa4 (patch) | |
| tree | 98223ef7f5abc5214299be591fa0f0aba12e1c11 | |
| parent | 49dec1cac6defdfb0ab9a15ab274a4a233b44e45 (diff) | |
| download | samba-c0d4100517bc50d7f81c3e5816a08c1a8284efa4.tar.gz samba-c0d4100517bc50d7f81c3e5816a08c1a8284efa4.tar.bz2 samba-c0d4100517bc50d7f81c3e5816a08c1a8284efa4.zip | |
r14155: Fix coverity #115: free storage alloc'ed by sstring_sub()
(This used to be commit a197b8c5cb02c8a5fac3882e7b76bcd7abb0279c)
| -rw-r--r-- | source3/utils/net_rpc_samsync.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/source3/utils/net_rpc_samsync.c b/source3/utils/net_rpc_samsync.c index a171782c48..691e28af8e 100644 --- a/source3/utils/net_rpc_samsync.c +++ b/source3/utils/net_rpc_samsync.c @@ -1636,6 +1636,7 @@ static NTSTATUS fetch_alias_info_to_ldif(SAM_DELTA_CTR *delta, GROUPMAP *groupma strcmp(aliasname, "Print Operators") == 0 || strcmp(aliasname, "Backup Operators") == 0 || strcmp(aliasname, "Replicator") == 0) { + SAFE_FREE(group_attr); return NT_STATUS_OK; } else { /* Increment the gid for the new group */ @@ -1663,6 +1664,7 @@ static NTSTATUS fetch_alias_info_to_ldif(SAM_DELTA_CTR *delta, GROUPMAP *groupma fprintf(add_fd, "\n"); fflush(add_fd); + SAFE_FREE(group_attr); /* Return */ return NT_STATUS_OK; } |