[PATCH] small refactorings to smbXsrv_open

[Volker Lendecke]

On Fri, Feb 26, 2016 at 04:56:50PM +0100, Michael Adam wrote:
> > With these checks in place I immediately look for the circumstances when
> > "db" can be NULL.
> 
> Heh, my reflex is to start verifying that all callers pass db != NULL
> if I see a function withOUT such checks. ;-)
> 
> > If in the future this needs to be legitimately calleable
> > without a db, then we can add it. But for that we need to really make
> > sure we understand why we need a "db fetch" call without a db in the
> > first place.
> 
> Updated patchset attached.

r-b me.

Volker

-- 
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 at sernet.de