]> code.ossystems Code Review - openembedded-core.git/commit
useradd_base.bbclass: Do not mess with the gshadow file in the sysroot
authorPeter Kjellerstedt <peter.kjellerstedt@axis.com>
Thu, 15 Sep 2016 15:44:47 +0000 (17:44 +0200)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Fri, 16 Sep 2016 14:15:33 +0000 (15:15 +0100)
commit4fcaa484a2e8046cf3277b5d14933cdaa94a4c3f
treefe921e670c46b6a1d15052d788ca06eb62accf1e
parent3226d89ff743c223181fda90f605c7579337941a
useradd_base.bbclass: Do not mess with the gshadow file in the sysroot

Previously, if the gshadow file did not exist in the sysroot when
perform_groupmems() was run, it would be temporarily created and
removed again afterwards. This was supposedly due to groupmems failing
if it does not exist.

However, based on empirical testing and examination of the source code
for groupmems, it should not fail if the gshadow file does not exist
when groupmems is started. But it WILL fail if the file is removed
sometime after its existence has been check at the beginning of the
execution, but before it needs to be modified. And this is exactly
what the previous code in perform_groupmems() could cause if multiple
tasks simultaneously modified users or groups. It could cause any of
the useradd, groupadd and groupmems commands to fail as long as at
least one other recipe invoked perform_groupmems().

Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/classes/useradd_base.bbclass