Fix semgrep issue with dgryski Go ruleset (#3719)

prebid · Jun 11, 2024 · a812221 · a812221
1 parent 1979036
commit a812221
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/util/jsonutil/merge.go b/util/jsonutil/merge.go
@@ -30,12 +30,12 @@ func init() {
 // Fields of type json.RawMessage are merged rather than replaced.
 func MergeClone(v any, data json.RawMessage) error {
 	err := jsonConfigMergeClone.Unmarshal(data, v)
-	if err != nil {
-		return &errortypes.FailedToUnmarshal{
-			Message: tryExtractErrorMessage(err),
-		}
+	if err == nil {
+		return nil
+	}
+	return &errortypes.FailedToUnmarshal{
+		Message: tryExtractErrorMessage(err),
 	}
-	return err
 }
 
 type mergeCloneExtension struct {