svn commit: samba r22342 - in branches/SAMBA_3_0_25/source: include nsswitch

simo idra at samba.org
Thu Apr 19 12:37:23 GMT 2007


I tested idmap_ad yesterday, while online there is no problem I could
get into. In offline mode I fond a segfault deep in the ad connection
code, going on with the debugging.

Simo.

On Wed, 2007-04-18 at 16:58 -0400, simo wrote:
> I have not fixed all the problems I found yet, but this patch helped
> a bit. I am investigating the Winbindd offline code some more,
> because I can reproduce a situation where if the DC is offline, on
> authentication I get a username consisting only of the domain portion
> and a gid of 0. I am also investigating the situation when our clock
> is not in sync with that of the DC.
> 
> Simo.
> 
> 
> On Wed, 2007-04-18 at 20:50 +0000, idra at samba.org wrote:
> > Author: idra
> > Date: 2007-04-18 20:49:59 +0000 (Wed, 18 Apr 2007)
> > New Revision: 22342
> > 
> > WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=22342
> > 
> > Log:
> > 
> > This patch tries to fix a number of problems with the offline code.
> > So far I have tested it only with idmap_tdb, idmap_ad testing is next.
> > 
> > Fixes:
> > - The code returned an error on any idmap call if winbindd
> > was put offline before the idmap code was initialized.
> > - The code denied consulting any backend, even local ones
> > like idmap_tdb when offline, correct the situation by
> > handling offline decisions in the backends
> > 
> > Simo.
> > 
> > 
> > Modified:
> >    branches/SAMBA_3_0_25/source/include/idmap.h
> >    branches/SAMBA_3_0_25/source/include/smb.h
> >    branches/SAMBA_3_0_25/source/nsswitch/idmap.c
> >    branches/SAMBA_3_0_25/source/nsswitch/idmap_ad.c
> >    branches/SAMBA_3_0_25/source/nsswitch/idmap_cache.c
> >    branches/SAMBA_3_0_25/source/nsswitch/idmap_ldap.c
> >    branches/SAMBA_3_0_25/source/nsswitch/idmap_nss.c
> >    branches/SAMBA_3_0_25/source/nsswitch/idmap_passdb.c
> >    branches/SAMBA_3_0_25/source/nsswitch/idmap_tdb.c
> > 
> > 
> > Changeset:
> > Sorry, the patch is too large (1193 lines) to include; please use WebSVN to see it!
> > WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=22342
-- 
Simo Sorce
Samba Team GPL Compliance Officer
email: idra at samba.org
http://samba.org



More information about the samba-technical mailing list