busybox: fix missing install with suid bit set if FEATURE_SUID=y
authorHenrique de Moraes Holschuh <henrique@nic.br>
Fri, 18 Oct 2019 11:31:30 +0000 (08:31 -0300)
committerPetr Štetiar <ynezz@true.cz>
Thu, 7 Nov 2019 21:50:16 +0000 (22:50 +0100)
With FEATURE_SUID=y one can install busybox binary belonging to root
with the suid bit set, enabling some applets to perform root-level
operations even when run by ordinary users. Busybox then drops
privileges for applets that don't need root access, before entering
their main() function.

Currently we don't install busybox binary with suid bit set, rendering
this feature unusable.

Note that we can't just "chmod u+s /bin/busybox" at runtime as a
"cheaper" solution: it would waste approximately 200KiB of FLASH (the
whole /bin/busybox binary gets copied into the overlay).

Ref: PR#2502
Signed-off-by: Henrique de Moraes Holschuh <henrique@nic.br>
[commit title/description facelift, use INSTALL_SUID variable]
Signed-off-by: Petr Štetiar <ynezz@true.cz>
package/utils/busybox/Makefile

index 1e1712fe71a5290a94f25cb8a059b48ec606d3a1..1046d68d3e643a3dee83dccabc2eec380d41fa2b 100644 (file)
@@ -113,6 +113,9 @@ endif
 ifneq ($(CONFIG_BUSYBOX_$(BUSYBOX_SYM)_NTPD),)
        $(INSTALL_BIN) ./files/sysntpd $(1)/etc/init.d/sysntpd
        $(INSTALL_BIN) ./files/ntpd-hotplug $(1)/usr/sbin/ntpd-hotplug
+endif
+ifneq ($(CONFIG_BUSYBOX_$(BUSYBOX_SYM)_FEATURE_SUID),)
+       $(INSTALL_SUID) $(PKG_INSTALL_DIR)/bin/busybox $(1)/bin/busybox
 endif
        -rm -rf $(1)/lib64
 endef