diff options
author | Andrew Bartlett <abartlet@samba.org> | 2007-04-02 05:53:34 +0000 |
---|---|---|
committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 12:19:03 -0500 |
commit | ccc06f84415a0496b8f3771e8888f81442acd808 (patch) | |
tree | ba6cdd8ef0b98126ff07f9658510ab87c4bf12ba /source3/auth/auth_sam.c | |
parent | fb3835846ef89a632230ff808259dad1cddc05c0 (diff) | |
download | samba-ccc06f84415a0496b8f3771e8888f81442acd808.tar.gz samba-ccc06f84415a0496b8f3771e8888f81442acd808.tar.bz2 samba-ccc06f84415a0496b8f3771e8888f81442acd808.zip |
r22022: - Clarify the comments
- make sure never to free an uninitialised variable
- ensure to free result on getpwnam_alloc failure
Andrew Bartlett
(This used to be commit 5fe3328e66661371182cc1c3b6e239797c3b4f93)
Diffstat (limited to 'source3/auth/auth_sam.c')
-rw-r--r-- | source3/auth/auth_sam.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/source3/auth/auth_sam.c b/source3/auth/auth_sam.c index 31178a761c..0fa68cbb87 100644 --- a/source3/auth/auth_sam.c +++ b/source3/auth/auth_sam.c @@ -263,8 +263,8 @@ static NTSTATUS check_sam_security(const struct auth_context *auth_context, return NT_STATUS_UNSUCCESSFUL; } - /* Can't use the talloc version here, because the returned struct gets - kept on the server_info */ + /* the returned struct gets kept on the server_info, by means + of a steal further down */ if ( !(sampass = samu_new( mem_ctx )) ) { return NT_STATUS_NO_MEMORY; |