]> granicus.if.org Git - icinga2/commitdiff
Fix: Dict initializer incorrectly re-initialized field that is set to an empty string
authorGunnar Beutner <gunnar@beutner.name>
Wed, 5 Aug 2015 05:45:10 +0000 (07:45 +0200)
committerGunnar Beutner <gunnar@beutner.name>
Wed, 5 Aug 2015 05:45:10 +0000 (07:45 +0200)
fixes #9835

lib/config/expression.cpp

index 5b26aa2ba8a8344ed7f0beeb8e81c37f83cdee36..f251910f97d3d13291af486235827498202e2d5f 100644 (file)
@@ -632,8 +632,12 @@ bool IndexerExpression::GetReference(ScriptFrame& frame, bool init_dict, Value *
                init_dict = false;
 
        if (m_Operand1->GetReference(frame, init_dict, &vparent, &vindex, &psdhint)) {
-               if (init_dict && VMOps::GetField(vparent, vindex, m_Operand1->GetDebugInfo()).IsEmpty())
-                       VMOps::SetField(vparent, vindex, new Dictionary(), m_Operand1->GetDebugInfo());
+               if (init_dict) {
+                       Value old_value =  VMOps::GetField(vparent, vindex, m_Operand1->GetDebugInfo());
+
+                       if (old_value.IsEmpty() && !old_value.IsString())
+                               VMOps::SetField(vparent, vindex, new Dictionary(), m_Operand1->GetDebugInfo());
+               }
 
                *parent = VMOps::GetField(vparent, vindex, m_DebugInfo);
                free_psd = true;