Emailing: 0001-Add-IPv6-support-to-ADS-client-side-LDAP-connects.-C.patch

Ralph Böhme rb at sernet.de
Tue May 12 11:44:09 MDT 2015


On Tue, May 12, 2015 at 10:23:24AM -0700, Jeremy Allison wrote:
> On Tue, May 12, 2015 at 04:27:03PM +0100, David Holder wrote:
> > Hi All,
> > 
> > The attached patch fixes a bug in Samba whereby client ADS LDAP
> > connections fail to IPv6-only DCs.
> > 
> > The fault was that the code assumed that a URI containing a literal
> > IPv6 address has the same format as a URI containing a literal IPv4
> > address in it. This assumption is wrong. For IPv6 URIs, the address
> > must be enclosed in square brackets.
> > 
> > E.g:
> > 
> > ldap://192.168.1.1
> > ldap://[2045::1]
> > 
> > Please review - thanks!
> 
> Reviewed with minor change (attached)
> 
> strstr(name, ":") doesn't
> cope with utf8 names.
> 
> strchr_m(name, ':')
> 
> does. Also added your 'Signed-off-by:'
> 
> With that LGTM. Can I get a second
> Team reviewer please ?

rb: me.

-Ralph

-- 
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.sernet.de,mailto:kontakt@sernet.de


More information about the samba-technical mailing list