any value in configuring minimum password length?

Robert P. J. Day rpjday at mindspring.com
Mon Jul 3 17:02:04 UTC 2006


On Mon, 3 Jul 2006, Rob Landley wrote:

> On Sunday 02 July 2006 3:47 pm, Tito wrote:
> > -	/* Add 1 for each type of characters to the minlen of password */
> > +	/* Add 2 for each type of characters to the minlen of password */
> > 	int size = MINLEN + 8;
>
> Actually, changing the code to match the comment might be better.
>
> And if you're making MINLEN configurable, why aren't you making the
> increment size be configurable?  (Or the number of different types
> of characters required before it increments the required length?)
> What's the rationale for exposing one but not the other?
>
> You see why I didn't want to go there?

but, luckily, you're going to overwrite all that stuff with your
massive rewrite anyway, right?  :-)

rday



More information about the busybox mailing list