Bugfix noticed by Herb. On using the again: tag as a
commit1db2dc1931d0a91aa6b8d9359fd395ec8a11cd44
authorJeremy Allison <jra@samba.org>
Sun, 8 Jun 2008 21:19:44 +0000 (8 14:19 -0700)
committerJeremy Allison <jra@samba.org>
Sun, 8 Jun 2008 21:19:44 +0000 (8 14:19 -0700)
tree67e142e468293c3ec2eb59cea9127d0b86aad53a
parentd1e0c7f0c1a62030e08fd3ae00ec8ed07c79aa72
Bugfix noticed by Herb. On using the again: tag as a
goto target we were not reinitializing the array counts.

From Herb:

This is in the file nsswitch/winbindd_cm.c (samba-3.0.30) line 1236

We have a label again: where we keep trying to find the name of the DC
from the list of IPs returned by get_dcs. If we fail to figure out the
name we do a goto again at the end of the function. The problem is we
don't reset the num_dcs, num_addrs, etc and free the memory in the
various arrays. This seems wrong to me. I have a winbindd core where
I have 9 IPs returned for the DCs but at the time of the crash num_dcs
is 87 and if I look through the array dcs it keeps repeating entries
from the same group of 9

Jerry, Volker and Guenther please check.

Jeremy.
(This used to be commit 15f464321a7c71a86b747918343746050d286655)
source3/winbindd/winbindd_cm.c