From 302420ad4a736053ae3a076ae48bfaac7b7d4208 Mon Sep 17 00:00:00 2001 From: Stefan Schantl Date: Sun, 11 Apr 2021 09:02:33 +0200 Subject: [PATCH] convert-ids-multiple-providers: Fix setting ownership for the main oinkmaster provider includes file. Signed-off-by: Stefan Schantl --- config/suricata/convert-ids-multiple-providers | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) 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. -- 2.39.2