diff options
| author | Swen Schillig <swen@vnet.ibm.com> | 2018-05-25 10:06:21 +0200 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2018-06-16 01:06:36 +0200 |
| commit | 4dac6e7cd08ab226eff9415b4924b6dcd8e5178d (patch) | |
| tree | b3492df2195236da4e175e896d475fca6c557ff3 /source3/rpc_client | |
| parent | 3ca1c09f686fbfa9257cd95710dba4a98c3eeb8f (diff) | |
| download | samba-4dac6e7cd08ab226eff9415b4924b6dcd8e5178d.tar.gz | |
Possible memory leak in map_info6_to_validation
If the call to copy_netr_SamInfo6 returns an error status,
the allocated memory for "validation" needs to be free'd before returning.
Signed-off-by: Swen Schillig <swen@vnet.ibm.com>
Reviewed-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Christof Schmitt <cs@samba.org>
Diffstat (limited to 'source3/rpc_client')
| -rw-r--r-- | source3/rpc_client/util_netlogon.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/source3/rpc_client/util_netlogon.c b/source3/rpc_client/util_netlogon.c index 2d73bc95cea..c57979a1265 100644 --- a/source3/rpc_client/util_netlogon.c +++ b/source3/rpc_client/util_netlogon.c @@ -378,6 +378,7 @@ NTSTATUS map_info6_to_validation(TALLOC_CTX *mem_ctx, info6, &validation->sam6); if (!NT_STATUS_IS_OK(status)) { + TALLOC_FREE(validation); return status; } |