Hi Joy, On Wed, Apr 13, 2005 at 11:30:29PM +0200, Josip Rodin wrote: > Apparently the upstream authors of OpenLDAP decided that > > limits user size=100 > > is no longer accepted, meaning slapd fails to start after upgrade.
It was never acceptable but the parser had a bug which caused undefined
behaviour in that case. They forgot an else branch in a instruction of
type
if (strcmp(type, "...") == 0)
...
else if (strcmp(...))
...
else
...
> It evades me why their parser in 2.1 didn't see this as a problem, yet the
Because it was a bug?
> new one suddenly breaks shit. I mean, if the old one ignored the line for
> invalid syntax, why does he new one have to *die* when seeing it?!
The same argument would apply to gcc which is much more strict in newer
versions what syntax checking is concerned. I can't see how that is a
bug.
> Please verify the integrity of ^limits lines in postinst upon upgrade,
> like you do with other changes. TIA.
I'll see if I find time to implement this. Currently I am only trying to
fix things the old slapd.conf generated by debconf has as well and which
really changed. I don't see why I should reimplement the syntax checker
of the config parser in places where it works just fine now.
Greetings
Torsten
signature.asc
Description: Digital signature
