]> code.ossystems Code Review - openembedded-core.git/commitdiff
useradd_base: keep group if it still has users
authorIoan-Adrian Ratiu <adrian.ratiu@ni.com>
Mon, 23 Jan 2017 18:09:23 +0000 (20:09 +0200)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Thu, 26 Jan 2017 10:41:10 +0000 (10:41 +0000)
perform_groupdel() tries to delete a group irrespective if other
users have it as their primary group, thus the call to groupdel will
fail with the following error error:

groupdel: cannot remove the primary group of user '<user>'

Add a check to perform_groupdel() to determine if there are other
users and keep the group, printing a warning. This is called right
after a user is deleted to delete it's group. If the last user is
deleted, only then the group is also deleted.

Signed-off-by: Ioan-Adrian Ratiu <adrian.ratiu@ni.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/classes/useradd_base.bbclass

index ba87edc57ac87049512c5b4412a499a875976f03..551c82c32285e0d5ad07be6dbe99c3595c6ccb86 100644 (file)
@@ -69,11 +69,21 @@ perform_groupdel () {
        bbnote "${PN}: Performing groupdel with [$opts]"
        local groupname=`echo "$opts" | awk '{ print $NF }'`
        local group_exists="`grep "^$groupname:" $rootdir/etc/group || true`"
+
        if test "x$group_exists" != "x"; then
-               eval flock -x $rootdir${sysconfdir} -c \"$PSEUDO groupdel \$opts\" || true
-               group_exists="`grep "^$groupname:" $rootdir/etc/group || true`"
-               if test "x$group_exists" != "x"; then
-                       bbfatal "${PN}: groupdel command did not succeed."
+               local awk_input='BEGIN {FS=":"}; $1=="'$groupname'" { print $3 }'
+               local groupid=`echo "$awk_input" | awk -f- $rootdir/etc/group`
+               local awk_check_users='BEGIN {FS=":"}; $4=="'$groupid'" {print $1}'
+               local other_users=`echo "$awk_check_users" | awk -f- $rootdir/etc/passwd`
+
+               if test "x$other_users" = "x"; then
+                       eval flock -x $rootdir${sysconfdir} -c \"$PSEUDO groupdel \$opts\" || true
+                       group_exists="`grep "^$groupname:" $rootdir/etc/group || true`"
+                       if test "x$group_exists" != "x"; then
+                               bbfatal "${PN}: groupdel command did not succeed."
+                       fi
+               else
+                       bbnote "${PN}: '$groupname' is primary group for users '$other_users', not removing it"
                fi
        else
                bbnote "${PN}: group $groupname doesn't exist, not removing it"