KumiSystems/openwrtv3
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

dnsmasq: fix remove pidfile on shutdown regression

Browse source 
Regression introduced by 3481d0d dnsmasq: run as dedicated UID/GID

dnsmasq is unable to remove its own pidfile as /var/run/dnsmasq is owned
by root and now dnsmasq runs as dnsmasq:dnsmasq.  Change directory
ownership to match.

dnsmasq initially starts as root, creates the pidfile, then drops to
requested non-root user.  Until this fix dnsmasq had insufficient
privilege to remove its own pidfile.

Signed-off-by: Kevin Darbyshire-Bryant <kevin@darbyshire-bryant.me.uk>
This commit is contained in:
Kevin Darbyshire-Bryant 2016-09-05 09:01:19 +01:00 committed by Felix Fietkau
parent c5913264e7
commit 9209f4304b
1 changed files with 1 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
package/network/services/dnsmasq/files/dnsmasq.init
View file

@ -643,6 +643,7 @@ start_service() {
mkdir -p $(dirname $CONFIGFILE) mkdir -p $(dirname $CONFIGFILE)
mkdir -p /var/lib/misc mkdir -p /var/lib/misc
touch /tmp/dhcp.leases touch /tmp/dhcp.leases
chown dnsmasq:dnsmasq /var/run/dnsmasq
[ -f "$TIMESTAMPFILE" ] && rm -f "$TIMESTAMPFILE" [ -f "$TIMESTAMPFILE" ] && rm -f "$TIMESTAMPFILE"