Browse Source

Fix suid bits on plugin for debian packaging (#8996)

Signed-off-by: Arthur Outhenin-Chalandre <arthur@cri.epita.fr>
Arthur Outhenin-Chalandre 4 years ago
parent
commit
572ed46860
2 changed files with 10 additions and 1 deletions
  1. 9 0
      contrib/debian/netdata.postinst.in
  2. 1 1
      contrib/debian/rules

+ 9 - 0
contrib/debian/netdata.postinst.in

@@ -55,6 +55,15 @@ case "$1" in
     chown -R root:netdata /var/lib/netdata/www
     chown -R root:netdata /var/lib/netdata/www
     setcap cap_dac_read_search,cap_sys_ptrace+ep /usr/libexec/netdata/plugins.d/apps.plugin
     setcap cap_dac_read_search,cap_sys_ptrace+ep /usr/libexec/netdata/plugins.d/apps.plugin
 
 
+    chmod 4750 /usr/libexec/netdata/plugins.d/perf.plugin
+    chmod 4750 /usr/libexec/netdata/plugins.d/slabinfo.plugin
+    chmod 4750 /usr/libexec/netdata/plugins.d/cgroup-network
+    chmod 4750 /usr/libexec/netdata/plugins.d/nfacct.plugin
+    chmod 4750 /usr/libexec/netdata/plugins.d/ebpf.plugin
+
+    # Workaround for other plugins not installed directly by this package
+    chmod -f 4750 /usr/libexec/netdata/plugins.d/freeipmi.plugin || true
+
     ;;
     ;;
 esac
 esac
 
 

+ 1 - 1
contrib/debian/rules

@@ -125,7 +125,7 @@ override_dh_fixperms:
 	chmod 0750 $(TOP)-plugin-cups/usr/libexec/netdata/plugins.d/cups.plugin
 	chmod 0750 $(TOP)-plugin-cups/usr/libexec/netdata/plugins.d/cups.plugin
 
 
 	# freeIPMI plugin package
 	# freeIPMI plugin package
-	chmod 0754 $(TOP)-plugin-freeipmi/usr/libexec/netdata/plugins.d/freeipmi.plugin
+	chmod 4750 $(TOP)-plugin-freeipmi/usr/libexec/netdata/plugins.d/freeipmi.plugin
 
 
 override_dh_installlogrotate:
 override_dh_installlogrotate:
 	cp system/netdata.logrotate debian/netdata.logrotate
 	cp system/netdata.logrotate debian/netdata.logrotate