NULL-Pointer in passdb backends

Simo Sorce idra at samba.org
Fri Sep 3 13:40:45 GMT 2004


On Fri, 2004-09-03 at 15:10, Simo Sorce wrote:
> On Fri, 2004-09-03 at 14:55, Heinrich Mislik wrote:
> > Hi,
> > 
> > this is from examples/pdb/pdb_test.c :
> > 
> > 
> >         /* Functions your pdb module doesn't provide should be set
> >          * to NULL */
> > 
> > and is wrong. The interface does not check for NULL-Pointers and 
> > crashes the smbd, giving some strange results. It just took me a 
> > couple of hours, to find this out. So please remove this comment.
> 
> IIRC it should check, if it doesn't then we have a problem, I'll watch
> out which is the right way to handle this.

Check, you're right, the hint is wrong, you should just not set anything
as the make_pdb_methods() function already sets default functions for
you (default functions returns NT_STATUS_NOT_IMPLEMENTED).

Simo.
-- 
Simo Sorce    -  idra at samba.org
Samba Team    -  http://www.samba.org
Italian Site  -  http://samba.xsec.it



More information about the samba-technical mailing list