chpass(1) patch for changing only crypted passwords

Dmitry Morozovsky marck at rinet.ru
Mon Sep 25 10:50:19 PDT 2006


On Mon, 25 Sep 2006, Dag-Erling Sm?rgrav wrote:

DS> Dmitry Morozovsky <marck at rinet.ru> 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 this.
DS> 
DS> Looks fine to me, feel free to commit.

Done, thanks.

BTW, a question inspired by ru@: why pw_equal() skips password field? Its 
interface is unpublished, and the only consumers are 

marck at woozle:/lh/src.current> grep -Rl pw_equal .
./lib/libutil/libutil.h
./lib/libutil/pw_util.c
./release/picobsd/tinyware/passwd/pw_copy.c
./usr.bin/chpass/chpass.c

and tyniware is false alarm ;)



Sincerely,
D.Marck                                     [DM5020, MCK-RIPE, DM3-RIPN]
------------------------------------------------------------------------
*** Dmitry Morozovsky --- D.Marck --- Wild Woozle --- marck at rinet.ru ***
------------------------------------------------------------------------


More information about the freebsd-stable mailing list