Ticket #146 (closed defect: fixed)

Opened 19 months ago

Last modified 19 months ago

umask option doesn't actually work

Reported by: douglas@openplans.org Owned by: roy
Priority: major Milestone:
Component: rc Version: 0.4
Keywords: umask Cc:

Description

start-stop-daemon has a -k or --umask option to set the daemon's umask parameter. The code at source:trunk/src/rc/start-stop-daemon.c?rev=1480#L1044 updates the umask if it has been supplied on the command line.

Unfortunately source:trunk/src/rc/start-stop-daemon.c?rev=1480#L1164 overrides this value. I'm including a patch with a fix.

Attachments

openrc-0.4.1-umask.patch Download (602 bytes) - added by douglas@openplans.org 19 months ago.
A fix that unifies the umask handling code

Change History

Changed 19 months ago by douglas@openplans.org

A fix that unifies the umask handling code

comment:1 Changed 19 months ago by roy

  • Status changed from new to closed
  • Resolution set to fixed

As we always set a umask, there is no need for the boolean flag either.
Fixed in r1494.

Thanks!

Note: See TracTickets for help on using tickets.