From: Stefan Schantl <stefan.schantl@ipfire.org>
Date: Sun, 11 Apr 2021 07:02:33 +0000 (+0200)
Subject: convert-ids-multiple-providers: Fix setting ownership for the main
X-Git-Url: http://git.ipfire.org/?p=people%2Fstevee%2Fipfire-2.x.git;a=commitdiff_plain;h=302420ad4a736053ae3a076ae48bfaac7b7d4208

convert-ids-multiple-providers: Fix setting ownership for the main
oinkmaster provider includes file.

Signed-off-by: Stefan Schantl <stefan.schantl@ipfire.org>
---

diff --git a/config/suricata/convert-ids-multiple-providers b/config/suricata/convert-ids-multiple-providers
index 971695dd73..82beb58036 100644
--- a/config/suricata/convert-ids-multiple-providers
+++ b/config/suricata/convert-ids-multiple-providers
@@ -167,14 +167,16 @@ if (%enabled_disabled_sids) {
 	# Add the provider modifications file to the oinkmaster provider includes file.
 	&IDS::alter_oinkmaster_provider_includes_file("add", "$ruleset_provider");
 
-	# Set correct ownerships.
-	&IDS::set_ownership("$IDS::oinkmaster_provider_includes_file");
+	# Set correct ownership for the new generated file.
 	&IDS::set_ownership("$oinkmaster_provider_modified_sids_file");
 }
 
-	# Remove old files.
-	unlink($old_enabled_sids_file);
-	unlink($old_disabled_sids_file);
+# Set correct ownership for the main file.
+&IDS::set_ownership("$IDS::oinkmaster_provider_includes_file");
+
+# Remove old files.
+unlink($old_enabled_sids_file);
+unlink($old_disabled_sids_file);
 
 #
 ## Step 6: Call oinkmaster and regenerate the ruleset structures.