On Tue, 26 Sep 2006, Dag-Erling Sm?rgrav wrote:
DS> Dmitry Morozovsky <[EMAIL PROTECTED]> writes:
DS> > BTW, a question inspired by ru@: why pw_equal() skips password
DS> > field? Its interface is unpublished, and the only consumers are
DS> >
DS> > [EMAIL PROTECTED]:/lh/src.current> grep -Rl pw_eq
Dmitry Morozovsky <[EMAIL PROTECTED]> writes:
> BTW, a question inspired by ru@: why pw_equal() skips password
> field? Its interface is unpublished, and the only consumers are
>
> [EMAIL PROTECTED]:/lh/src.current> grep -Rl pw_equal .
> ./lib/libutil/libutil.h
> ./lib/libutil/pw_util.c
> ./release
On Mon, 25 Sep 2006, Dag-Erling Sm?rgrav wrote:
DS> Dmitry Morozovsky <[EMAIL PROTECTED]> writes:
DS> > chpass is a bit broken since chpass.c:1.23 - it uses pw_equal which
DS> > does not check crypted password field; so, you can not change _only_
DS> > crypt pass with chpass. Attached patch fixes
Dmitry Morozovsky <[EMAIL PROTECTED]> writes:
> chpass is a bit broken since chpass.c:1.23 - it uses pw_equal which
> does not check crypted password field; so, you can not change _only_
> crypt pass with chpass. Attached patch fixes this.
Looks fine to me, feel free to commit.
DES
--
Dag-Erling
Dear colleagues,
chpass is a bit broken since chpass.c:1.23 - it uses pw_equal which does not
check crypted password field; so, you can not change _only_ crypt pass with
chpass. Attached patch fixes this.
Any thoughts/objections?
Sincerely,
D.Marck [DM5020,