diff mbox series

update-ids-ruleset: Set correct ownership for therulestarball.

Message ID 20190501144925.3766-1-stefan.schantl@ipfire.org
State Accepted
Commit 0034a92ad79d6c9853216602960c8ebe22d13b21
Headers
Series update-ids-ruleset: Set correct ownership for therulestarball. |

Commit Message

Stefan Schantl May 2, 2019, 12:49 a.m. UTC 
  The script usualy will be executed by cron which will start it with
root permissions, so the downloaded tarball is owned by this user.

This has to be changed to the user which runs the WUI (nobody:nobody) to
allow, changing the ruleset to an other one and to display the ruleset area.

Fixes #12066

Signed-off-by: Stefan Schantl <stefan.schantl@ipfire.org>
---
 src/scripts/update-ids-ruleset | 3 +++
 1 file changed, 3 insertions(+)
diff mbox series

Patch

diff --git a/src/scripts/update-ids-ruleset b/src/scripts/update-ids-ruleset
index 14ea25ec6..f28a8c156 100644
--- a/src/scripts/update-ids-ruleset
+++ b/src/scripts/update-ids-ruleset
@@ -58,6 +58,9 @@  if(&IDS::downloadruleset()) {
 	exit 0;
 }
 
+# Set correct ownership for the downloaded tarball.
+&IDS::set_ownership("$IDS::rulestarball");
+
 # Call oinkmaster to alter the ruleset.
 &IDS::oinkmaster();