git.ipfire.org Git - thirdparty/shadow.git/commit

]> git.ipfire.org Git - thirdparty/shadow.git/commit

projects / thirdparty / shadow.git / commit

author	Alejandro Colomar <alx@kernel.org>
	Fri, 18 Jul 2025 10:23:42 +0000 (12:23 +0200)
committer	Serge Hallyn <serge@hallyn.com>
	Sat, 9 Aug 2025 22:03:22 +0000 (17:03 -0500)
commit	2ff16902d125fa81b52579d488babcc0e6ffe5d0
tree	bb83cf4ca825ba6ed491484878e87f337992ec95	tree \| snapshot
parent	841776561f56bae7382c6bd47e428201a155d39c	commit \| diff

lib/, src/: Reorder while() conditions for safety

In conditions that perform simple assignment (=) before comparison,
it's safer to put the comparison first, as a mistake would result in a
compiler error, as opposed to assigning something incorrect.
It's also more readable, IMO.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

19 files changed:

lib/copydir.c		diff \| blob \| blame \| history
lib/find_new_gid.c		diff \| blob \| blame \| history
lib/find_new_uid.c		diff \| blob \| blame \| history
lib/gshadow.c		diff \| blob \| blame \| history
lib/port.c		diff \| blob \| blame \| history
lib/prefix_flag.c		diff \| blob \| blame \| history
lib/subordinateio.c		diff \| blob \| blame \| history
lib/user_busy.c		diff \| blob \| blame \| history
lib/utmp.c		diff \| blob \| blame \| history
src/groupmod.c		diff \| blob \| blame \| history
src/grpconv.c		diff \| blob \| blame \| history
src/grpunconv.c		diff \| blob \| blame \| history
src/logoutd.c		diff \| blob \| blame \| history
src/newgrp.c		diff \| blob \| blame \| history
src/pwconv.c		diff \| blob \| blame \| history
src/pwunconv.c		diff \| blob \| blame \| history
src/su.c		diff \| blob \| blame \| history
src/userdel.c		diff \| blob \| blame \| history
src/usermod.c		diff \| blob \| blame \| history

Mirror of https://github.com/shadow-maint/shadow.git

RSS Atom