Guidelines about fixing bugs

[Matthieu Patou]

Hello All,

We have been witnessing some issues related to the way Winbindd do DNS
lookups of DC for various services (ldap, kdc, ...)

It mainly boils down to this bug:
https://bugzilla.samba.org/show_bug.cgi?id=12533, that is to say DNS
resolution of names from the SRV records are done in a sequential way;
when there is a combination of slow DNS server and huge network of DC
the resolution can take so long (and is made worse by other bugs) that
clients timeout.

While looking at this issue I've found a couple of in-efficiencies like

https://bugzilla.samba.org/show_bug.cgi?id=12532, where the refresh_usn
function would be called multiple time in parallel, or

https://bugzilla.samba.org/show_bug.cgi?id=12548, where the get_dc_name
is called twice almost back to back, given the fact that this function
cause the SRV records for _kerberos to be looked up and the names to be
resolved it's basically doubling the time it takes to do refresh_usn
which is kind of a big deal when DNS is slow and DC is huge.

I'm not as familiar as Volker, Metze of Guenther with this part of the
code base so I would appreciate if one of you (or a bit more) could
chime in on the high level solution that proposed in those bugs so that
me or pradeep or ravindra are doing the work in the right direction.


While at it I have also one bug about net ads changetrustpw picking a
random DC rather on in the site
(https://bugzilla.samba.org/show_bug.cgi?id=12515) and if there as well
I could have feedback it would be great.


Thanks.

-- 
Matthieu Patou
Samba Team
http://samba.org