diff options
| author | Gerald Carter <jerry@samba.org> | 2003-09-06 19:49:59 +0000 |
|---|---|---|
| committer | Gerald Carter <jerry@samba.org> | 2003-09-06 19:49:59 +0000 |
| commit | 5bd4af13023bd645e1bcd1fde826d286382b33e5 (patch) | |
| tree | 0637276d2cf18e39e1e2b26cbbcc74409df38fd7 | |
| parent | c913fc058113b3a3a193f7b98459679945afcf03 (diff) | |
| download | samba-5bd4af13023bd645e1bcd1fde826d286382b33e5.tar.gz samba-5bd4af13023bd645e1bcd1fde826d286382b33e5.tar.bz2 samba-5bd4af13023bd645e1bcd1fde826d286382b33e5.zip | |
fix bug 382: make sure to free and reset the cli_state pointer after flushing connections in winbindd_pam_auth[_crap]()
(This used to be commit 60c8796edd84249db5419329a0fea4bbc7f9bb98)
| -rw-r--r-- | source3/nsswitch/winbindd_pam.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/source3/nsswitch/winbindd_pam.c b/source3/nsswitch/winbindd_pam.c index a89c7ca065..79b8c1d92d 100644 --- a/source3/nsswitch/winbindd_pam.c +++ b/source3/nsswitch/winbindd_pam.c @@ -127,6 +127,9 @@ enum winbindd_result winbindd_pam_auth(struct winbindd_cli_state *state) do { ZERO_STRUCT(info3); ZERO_STRUCT(ret_creds); + + if ( cli ) + SAFE_FREE(cli); /* Don't shut this down - it belongs to the connection cache code */ result = cm_get_netlogon_cli(contact_domain, trust_passwd, @@ -301,6 +304,9 @@ enum winbindd_result winbindd_pam_auth_crap(struct winbindd_cli_state *state) ZERO_STRUCT(info3); ZERO_STRUCT(ret_creds); + if ( cli ) + SAFE_FREE( cli ); + /* Don't shut this down - it belongs to the connection cache code */ result = cm_get_netlogon_cli(contact_domain, trust_passwd, sec_channel_type, False, &cli); |