[2019-07-18 17:56] Thorsten Glaser <t.gla...@tarent.de>
> Dmitry: please also move the chmod BELOW the chown, as chown
> is known to sometimes reset permissions (in some documented
> cases, not a bug):
1. Your patch changes permissions from 644 to 664. Is it deliberate?
2. Can you please provide pointers to documentation about chown
permission rest to make comment about it?
The only thing I found is (from Texinfo manual):
The ‘chown’ command sometimes clears the set-user-ID or
set-group-ID permission bits.
but it does not seems to be relevant: `/var/run/utmp` is meant to
have neither suid nor sgid bits.
> tglase@tglase:~ $ diff -u /etc/init.d/bootmisc.sh bootmisc.sh
> --- /etc/init.d/bootmisc.sh 2019-05-15 13:03:46.000000000 +0200
> +++ bootmisc.sh 2019-07-18 17:51:51.887830376 +0200
> @@ -33,8 +33,8 @@
>
> readonly utmp='/var/run/utmp'
> if > "${utmp}" ; then
> - chmod 644 "${utmp}" || log_warning_msg "failed to chmod
> ${utmp}"
> chgrp utmp "${utmp}" || log_warning_msg "failed to chgrp
> ${utmp}"
> + chmod 664 "${utmp}" || log_warning_msg "failed to chmod
> ${utmp}"
> return 0
> else
> log_failure_msg "failed to truncate ${utmp}"
--
Note, that I send and fetch email in batch, once in a few days.
Please, mention in body of your reply when you add or remove recepients.
