]> granicus.if.org Git - icinga2/commitdiff
Namespace: place ObjectLock in all methods
authorAlexander A. Klimov <alexander.klimov@icinga.com>
Tue, 16 Apr 2019 15:38:58 +0000 (17:38 +0200)
committerMichael Friedrich <michael.friedrich@icinga.com>
Thu, 16 May 2019 14:19:29 +0000 (16:19 +0200)
(cherry picked from commit 5afda779435c848cf5a8411f713adc9891aa5092)

lib/base/namespace.cpp

index 8ca78531861bae39dd08702aeefea2e0e3549179..91373fb26107f4aeb254f33751542fbfe12a6e14 100644 (file)
@@ -37,6 +37,8 @@ Namespace::Namespace(NamespaceBehavior *behavior)
 
 Value Namespace::Get(const String& field) const
 {
+       ObjectLock olock(this);
+
        Value value;
        if (!GetOwnField(field, &value))
                BOOST_THROW_EXCEPTION(ScriptError("Namespace does not contain field '" + field + "'"));
@@ -45,6 +47,8 @@ Value Namespace::Get(const String& field) const
 
 bool Namespace::Get(const String& field, Value *value) const
 {
+       ObjectLock olock(this);
+
        auto nsVal = GetAttribute(field);
 
        if (!nsVal)
@@ -56,16 +60,22 @@ bool Namespace::Get(const String& field, Value *value) const
 
 void Namespace::Set(const String& field, const Value& value, bool overrideFrozen)
 {
+       ObjectLock olock(this);
+
        return SetFieldByName(field, value, overrideFrozen, DebugInfo());
 }
 
 bool Namespace::Contains(const String& field) const
 {
+       ObjectLock olock(this);
+
        return HasOwnField(field);
 }
 
 void Namespace::Remove(const String& field, bool overrideFrozen)
 {
+       ObjectLock olock(this);
+
        m_Behavior->Remove(this, field, overrideFrozen);
 }
 
@@ -103,6 +113,8 @@ void Namespace::SetAttribute(const String& key, const std::shared_ptr<NamespaceV
 
 Value Namespace::GetFieldByName(const String& field, bool, const DebugInfo& debugInfo) const
 {
+       ObjectLock olock(this);
+
        auto nsVal = GetAttribute(field);
 
        if (nsVal)
@@ -113,6 +125,8 @@ Value Namespace::GetFieldByName(const String& field, bool, const DebugInfo& debu
 
 void Namespace::SetFieldByName(const String& field, const Value& value, bool overrideFrozen, const DebugInfo& debugInfo)
 {
+       ObjectLock olock(this);
+
        auto nsVal = GetAttribute(field);
 
        if (!nsVal)
@@ -123,11 +137,15 @@ void Namespace::SetFieldByName(const String& field, const Value& value, bool ove
 
 bool Namespace::HasOwnField(const String& field) const
 {
+       ObjectLock olock(this);
+
        return GetAttribute(field) != nullptr;
 }
 
 bool Namespace::GetOwnField(const String& field, Value *result) const
 {
+       ObjectLock olock(this);
+
        auto nsVal = GetAttribute(field);
 
        if (!nsVal)