[Pkg-samba-maint] Bug#410823: closed by Christian Perrier <bubulle@debian.org> (Re: Bug#410823: samba: smbd segfaults during startup)

[Steve Langasek]

On Tue, Feb 13, 2007 at 09:26:38PM +0100, Fabien COUTANT wrote:
> 1/ if "guest" has been disabled from "passdb backend", smbd should *not*
>    crash with a segfault when guest is still written inside, but instead
>    ignore it or show an explanatory error message;  testparm should also
>    report it (it still doesn't).

the value of "passdb backend" is a freeform text value, whose allowed values
are extended by the addition of plugins.  It's improbable that testparm will
vet these values.

> 2/ The changelog provided in the Debian package doesn't give any rationale,
>    only the fact it was removed.

Christian referred to the upstream samba changelog, not to the Debian
package's changelog.  The upstream changelog isn't included in the Debian
binary packages.

> 3/ by reading the changelog I see that some upgrade *should* have
>    *silently* removed the incriminated "guest" value. (a) it didn't, and
>    (b) it shouldn't be done silently but instead clearly advertised since
>    it's important functionality removal.

No, it's not important functionality removal.  The 'guest' backend was an
obsolete no-op before sarge's release.  So a silent upgrade is the correct
behavior.

If there are cases where the guest value is not being automatically removed
when it should be, we want to know about that so we can fix it.  What
version were you upgrading from?

> 4/ "passdb backend = tdbsam guest" was working fine for me.  Is the new
>    version supposed to behave the same, i.e. provide the same access right
>    policy, with "guest" removed ?

Yes, as I mentioned the 'guest' backend is long obsolete.

-- 
Steve Langasek                   Give me a lever long enough and a Free OS
Debian Developer                   to set it on, and I can move the world.
vorlon at debian.org                                   http://www.debian.org/