[Buildroot] [PATCH] package/usbguard: enforce the right permissions on the configuration file

[Buildroot] [PATCH] package/usbguard: enforce the right permissions on the configuration file

[Miquel Raynal]

Only 0600 rights are allowed for the rules.conf. This file is read when the
usbguard daemon starts and will prevent it to run otherwise.

As Git only tracks the executable bit, setting the right permissions in the
package makefile is the cleanest solution, in particular when providing this
file from a rootfs overlay.

Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
---
 package/usbguard/usbguard.mk | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/package/usbguard/usbguard.mk b/package/usbguard/usbguard.mk
index 0626a7cc88..ea1be0dc48 100644
--- a/package/usbguard/usbguard.mk
+++ b/package/usbguard/usbguard.mk
@@ -47,5 +47,8 @@ define USBGUARD_INSTALL_INIT_SYSV
 		$(TARGET_DIR)/etc/init.d/S20usbguard
 endef
 
+define USBGUARD_PERMISSIONS
+	/etc/usbguard/rules.conf f 0600 0 0 - - - - -
+endef
 
 $(eval $(autotools-package))
-- 
2.27.0

[Buildroot] [PATCH] package/usbguard: enforce the right permissions on the configuration file

[Thomas Petazzoni]

On Fri, 18 Jun 2021 11:47:10 +0200
Miquel Raynal <miquel.raynal@bootlin.com> wrote:

> Only 0600 rights are allowed for the rules.conf. This file is read when the
> usbguard daemon starts and will prevent it to run otherwise.
> 
> As Git only tracks the executable bit, setting the right permissions in the
> package makefile is the cleanest solution, in particular when providing this
> file from a rootfs overlay.
> 
> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
> ---
>  package/usbguard/usbguard.mk | 3 +++
>  1 file changed, 3 insertions(+)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com