diff options
author | Sören Tempel | 2020-06-09 17:51:26 +0200 |
---|---|---|
committer | Denys Vlasenko | 2020-06-09 18:04:31 +0200 |
commit | 0356607264b8e1476d98a81667488ba1d6295d23 (patch) | |
tree | 65879ea7e3ebdb9171baaed1b5c8c9be12446135 /selinux/sestatus.c | |
parent | d30d1ebc117db182a6156df182057291d6fbaae1 (diff) | |
download | busybox-0356607264b8e1476d98a81667488ba1d6295d23.zip busybox-0356607264b8e1476d98a81667488ba1d6295d23.tar.gz |
deluser: check if specified home is a directory before removing it
On Alpine, some users use /dev/null as a home directory. When removing
such a user with `deluser --remove-home` this causes the /dev/null
device file to be removed which is undesirable. To prevent this pitfall,
check if the home directory specified for the user is an actual
directory (or a symlink to a directory).
Implementations of similar tools for other operating systems also
implement such checks. For instance, the OpenBSD rmuser(1)
implementation [0].
[0]: https://github.com/openbsd/src/blob/b69faa6c70c5bfcfdddc6138cd8e0ee18cc15b03/usr.sbin/adduser/rmuser.perl#L143-L151
function old new delta
deluser_main 337 380 +43
Signed-off-by: Sören Tempel <soeren+git@soeren-tempel.net>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'selinux/sestatus.c')
0 files changed, 0 insertions, 0 deletions