From ffdb1ce79f054237c2ab5c57436c4ca6963d36f6 Mon Sep 17 00:00:00 2001
From: urso <steffen.siering@elastic.co>
Date: Mon, 4 May 2020 19:07:45 +0200
Subject: [PATCH] review

---
 libbeat/common/mapstr.go | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/libbeat/common/mapstr.go b/libbeat/common/mapstr.go
index 7c44e2d5ac3..d70f05dd349 100644
--- a/libbeat/common/mapstr.go
+++ b/libbeat/common/mapstr.go
@@ -96,10 +96,6 @@ func (m MapStr) deepUpdateMap(d MapStr, overwrite bool) {
 }
 
 func deepUpdateValue(old interface{}, val MapStr, overwrite bool) interface{} {
-	if old == nil {
-		return val
-	}
-
 	switch sub := old.(type) {
 	case MapStr:
 		if sub == nil {
@@ -117,7 +113,9 @@ func deepUpdateValue(old interface{}, val MapStr, overwrite bool) interface{} {
 		tmp.deepUpdateMap(val, overwrite)
 		return tmp
 	default:
-		// This should never happen
+		// We reach the default branch if old is no map or if old == nil.
+		// In either case we return `val`, such that the old value is completely
+		// replaced when merging.
 		return val
 	}
 }