Re: [PATCH] dnsmasq: procd-ujail: workaround startup failure, when leasefile location is in /tmp

2021-10-19 Thread Bastian Bittorf
On Mon, Oct 18, 2021 at 10:20:56AM +0100, Daniel Golle wrote: > On Mon, Oct 18, 2021 at 08:12:00AM +, Bastian Bittorf wrote: [...] > > This is v2 of the patch with a more correct description what is does. > > > > Ref: https://bugs.openwrt.org/index.php?do=details_id=4085 > > Suggested-by:

Re: [PATCH] dnsmasq: procd-ujail: workaround startup failure, when leasefile location is in /tmp

2021-10-18 Thread Daniel Golle
On Mon, Oct 18, 2021 at 08:12:00AM +, Bastian Bittorf wrote: > introduced with 44f694ba1bca1417d24e851c637c284f9f78c06d > ("build: select procd-ujail if !SMALL_FLASH") dnsmasq fails > to startup when the leasefile is configured to be in /tmp, > which is just not suited for beeing a jail

[PATCH] dnsmasq: procd-ujail: workaround startup failure, when leasefile location is in /tmp

2021-10-18 Thread Bastian Bittorf
introduced with 44f694ba1bca1417d24e851c637c284f9f78c06d ("build: select procd-ujail if !SMALL_FLASH") dnsmasq fails to startup when the leasefile is configured to be in /tmp, which is just not suited for beeing a jail location. With this patch we explicit call jail_mount_rw() for the (now

Re: [PATCH] dnsmasq: procd-ujail: workaround startup failure, when leasefile location is in /tmp

2021-10-17 Thread Bastian Bittorf
On Sun, Oct 17, 2021 at 05:45:19PM +0100, Daniel Golle wrote: > > + "because jailing /tmp does not work: choose > > e.g. /tmp/dnsmasq/leasefile" > > + } > To do what you describe in the commit message it would be > } else { > > I'm fine with either, just the

Re: [PATCH] dnsmasq: procd-ujail: workaround startup failure, when leasefile location is in /tmp

2021-10-17 Thread Daniel Golle
On Sun, Oct 17, 2021 at 03:42:18PM +, Bastian Bittorf wrote: > introduced with 44f694ba1bca1417d24e851c637c284f9f78c06d > ("build: select procd-ujail if !SMALL_FLASH") dnsmasq fails > to startup when the leasefile is configured to be in /tmp, > which is just not suited for beeing a jail

[PATCH] dnsmasq: procd-ujail: workaround startup failure, when leasefile location is in /tmp

2021-10-17 Thread Bastian Bittorf
introduced with 44f694ba1bca1417d24e851c637c284f9f78c06d ("build: select procd-ujail if !SMALL_FLASH") dnsmasq fails to startup when the leasefile is configured to be in /tmp, which is just not suited for beeing a jail location. Workaround this (no jailing for this file for this special case) and