FS#23309 - [initscripts] add utmp group support

Attached to Project: Arch Linux
Opened by Greg (dolby) - Thursday, 17 March 2011, 09:10 GMT
Last edited by Tom Gundersen (tomegun) - Wednesday, 13 April 2011, 12:53 GMT
Task Type Feature Request
Category Arch Projects
Status Closed
Assigned To Tom Gundersen (tomegun)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

A utmp group should be created in order for members of the group to be able to write in /var/run/utmp.
In  FS#23240  it is requested such a group to be added in the default /etc/group file.

/etc/rc.sysinit also needs to set permissions to /var/run/tmp to have root:utmp
The attached patch does that, according to whats in the first comment  FS#19392 
Closed by  Tom Gundersen (tomegun)
Wednesday, 13 April 2011, 12:53 GMT
Reason for closing:  Fixed
Comment by Greg (dolby) - Thursday, 17 March 2011, 09:11 GMT
And here's the patch

Edit: BTW: http://projects.archlinux.org/initscripts.git/commit/?id=0e4ff39e5a5c72b7d4e8d251d029d4342f68b7b6 based on  FS#12557 
   rc.sysinit.patch (0.4 KiB)
Comment by Greg (dolby) - Saturday, 26 March 2011, 11:16 GMT
No its not fixed with that commit from 2008. See the patch.
Comment by Tom Gundersen (tomegun) - Saturday, 26 March 2011, 11:30 GMT
Sorry, my bad. I've applied the patch locally and will push it out as soon as (if) utmp is added to /etc/group.
Comment by Tom Gundersen (tomegun) - Saturday, 26 March 2011, 11:35 GMT
Is there a bug in your patch though? Why did you change the permissions from 664 to 644?
Comment by Greg (dolby) - Saturday, 26 March 2011, 11:37 GMT
I was just about to comment on that:)
The permissions can/should stay 664.
I had made the patch based on https://bugs.archlinux.org/task/19392#comment61383
Thanks!
Comment by Greg (dolby) - Saturday, 26 March 2011, 11:44 GMT
This is the relevant part from Slackware's rc.S btw: http://slackware.osuosl.org/slackware64-current/source/a/sysvinit-scripts/scripts/rc.S

# Create a fresh utmp file:
touch /var/run/utmp
chown root:utmp /var/run/utmp
chmod 664 /var/run/utmp
Comment by Tom Gundersen (tomegun) - Saturday, 26 March 2011, 11:50 GMT
> The permissions can/should stay 664.

Good (that's what I did).
Comment by Greg (dolby) - Wednesday, 13 April 2011, 11:09 GMT
A filesystem package with utmp group is now in [core]
Comment by Tom Gundersen (tomegun) - Wednesday, 13 April 2011, 12:52 GMT
And the fix is in git: <http://projects.archlinux.org/initscripts.git/commit/?id=f144946d98a93b667dcf1def10ab066bda528aea>. I'll make a release soon-ish (there is nothing else pending).

Thanks for following up on this!

Loading...