diff options
| author | Andrew Bartlett <abartlet@samba.org> | 2002-03-19 06:36:37 +0000 |
|---|---|---|
| committer | Andrew Bartlett <abartlet@samba.org> | 2002-03-19 06:36:37 +0000 |
| commit | 462402fc098a8d727d92dee8114a3b06d96e3c93 (patch) | |
| tree | 986d82e4ca390131d5df4948aa11715b6887dacd /source3/nsswitch/winbindd_cm.c | |
| parent | bf555a158a99becb3a6ac0c3b2b9502f01cd2bc2 (diff) | |
| download | samba-462402fc098a8d727d92dee8114a3b06d96e3c93.tar.gz samba-462402fc098a8d727d92dee8114a3b06d96e3c93.tar.bz2 samba-462402fc098a8d727d92dee8114a3b06d96e3c93.zip | |
Fix a double-free bug in wbinfo -t's call in winbindd.
I forgot to clean this up when netlogon move across to the connection cache
arrangement.
Also add some smb_panics to the connection_ok() code to try to catch this kind
of thing better in future.
Andrew Bartlett
(This used to be commit f4f23fad6099143ec26550afc67655390070ceb8)
Diffstat (limited to 'source3/nsswitch/winbindd_cm.c')
| -rw-r--r-- | source3/nsswitch/winbindd_cm.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/source3/nsswitch/winbindd_cm.c b/source3/nsswitch/winbindd_cm.c index dfae06ff00..03a8e3cdd1 100644 --- a/source3/nsswitch/winbindd_cm.c +++ b/source3/nsswitch/winbindd_cm.c @@ -379,14 +379,16 @@ static BOOL connection_ok(struct winbindd_cm_conn *conn) } if (!conn->cli) { - DEBUG(3, ("Connection to %s for domain %s (pipe %s) has NULL conn->cli!\n", + DEBUG(0, ("Connection to %s for domain %s (pipe %s) has NULL conn->cli!\n", conn->controller, conn->domain, conn->pipe_name)); + smb_panic("connection_ok: conn->cli was null!"); return False; } if (!conn->cli->initialised) { - DEBUG(3, ("Connection to %s for domain %s (pipe %s) was never initialised!\n", + DEBUG(0, ("Connection to %s for domain %s (pipe %s) was never initialised!\n", conn->controller, conn->domain, conn->pipe_name)); + smb_panic("connection_ok: conn->cli->initialised is False!"); return False; } |