fix: Reset datafiles before loading the hub on HUP (#3712)

Author: LaurenceJJones

cmd/crowdsec/serve.go

@@ -96,6 +96,9 @@ func reloadHandler(sig os.Signal) (*csconfig.Config, error) {
 			return nil, err
 		}
 
+		// Reset data files to avoid any potential conflicts with the new configuration
+		exprhelpers.ResetDataFiles()
+
 		csParsers, datasources, err := initCrowdsec(cConfig, hub, false)
 		if err != nil {
 			return nil, fmt.Errorf("unable to init crowdsec: %w", err)

pkg/exprhelpers/helpers.go

@@ -133,6 +133,15 @@ func Init(databaseClient *database.Client) error {
 	return nil
 }
 
+// ResetDataFiles clears all datafile-related global variables.
+// This should be called during HUP reload to ensure clean state.
+func ResetDataFiles() {
+	dataFile = make(map[string][]string)
+	dataFileRegex = make(map[string][]*regexp.Regexp)
+	dataFileRe2 = make(map[string][]*re2.Regexp)
+	dataFileRegexCache = make(map[string]gcache.Cache)
+}
+
 func RegexpCacheInit(filename string, cacheCfg types.DataSource) error {
 	// cache is explicitly disabled
 	if cacheCfg.Cache != nil && !*cacheCfg.Cache {