From 5df1dad72cd81ad23e539bc59d4da2fceff91e1f Mon Sep 17 00:00:00 2001
From: "fox.cpp" <fox.cpp@disroot.org>
Date: Tue, 28 May 2019 20:32:15 +0300
Subject: [PATCH] Do not modify Node argument passed to initInlineModule

It causes config.Map state to be corrupted which make some
directives "disappear".

Closes #84.
---
 config.go     | 7 ++++---
 config/map.go | 1 +
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/config.go b/config.go
index 34e7b20..5c4246d 100644
--- a/config.go
+++ b/config.go
@@ -48,11 +48,12 @@ func initInlineModule(modObj module.Module, globals map[string]interface{}, node
 	//
 	// Expected: modName modArgs { ... }
 	// Actual: something modName modArgs { ... }
-	node.Name = node.Args[0]
-	node.Args = node.Args[1:]
+	nodeCpy := *node
+	nodeCpy.Name = node.Args[0]
+	nodeCpy.Args = node.Args[1:]
 
 	log.Debugln("module init", modObj.Name(), modObj.InstanceName(), "(inline)")
-	return modObj.Init(config.NewMap(globals, node))
+	return modObj.Init(config.NewMap(globals, &nodeCpy))
 }
 
 func deliverDirective(m *config.Map, node *config.Node) (interface{}, error) {
diff --git a/config/map.go b/config/map.go
index b8ba113..4d09b68 100644
--- a/config/map.go
+++ b/config/map.go
@@ -303,6 +303,7 @@ func (m *Map) Float(name string, inheritGlobal, required bool, defaultVal float6
 // mapper is a function that should convert configuration directive arguments
 // into variable value.  Both functions may fail with errors, configuration
 // processing will stop immediately then.
+// Note: mapper function should not modify passed values.
 //
 // store is where the value returned by mapper should be stored. Can be nil
 // (value will be saved only in Map.Values).