capicxx-core-tools 3.1.12.3

Author: juergengehring

CHANGES

@@ -1,5 +1,10 @@
 Changes
 =======
+
+v3.1.12.3
+- Prevent crash due to concurrent attribute access from stub itself
+  and a remote client.
+
 v3.1.12.2
 - Reworked attribute locking in StubAdapter to prevent deadlock
 

org.genivi.commonapi.core/src/org/genivi/commonapi/core/generator/FInterfaceStubGenerator.xtend

@@ -550,13 +550,27 @@ class FInterfaceStubGenerator {
             }
 
             void «fInterface.stubDefaultClassName»::«attribute.stubDefaultClassSetMethodName»(«typeName» _value) {
-                «IF attribute.isObservable»const bool valueChanged = «ENDIF»«attribute.stubDefaultClassTrySetMethodName»(std::move(_value));
                 «IF attribute.isObservable»
-                if (valueChanged) {
-                    «attribute.stubAdapterClassFireChangedMethodName»(«attribute.stubDefaultClassVariableName»);
+                std::shared_ptr<«fInterface.stubAdapterClassName»> stubAdapter = CommonAPI::Stub<«fInterface.stubAdapterClassName»,
+                «fInterface.stubRemoteEventClassName»>::stubAdapter_.lock();
+                if(stubAdapter) {
+                    stubAdapter->«attribute.stubClassLockMethodName»(true);
+                    const bool valueChanged = («attribute.stubDefaultClassVariableName» != _value);
+                    if (valueChanged) {
+                        «attribute.stubDefaultClassVariableName» = std::move(_value);
+                        «attribute.stubAdapterClassFireChangedMethodName»(«attribute.stubDefaultClassVariableName»);
+                    }
+                    stubAdapter->«attribute.stubClassLockMethodName»(false);
+                } else {
+                    const bool valueChanged = «attribute.stubDefaultClassTrySetMethodName»(std::move(_value));
+                    if (valueChanged) {
+                        «attribute.stubAdapterClassFireChangedMethodName»(«attribute.stubDefaultClassVariableName»);
+                    }
                 }
+                «ELSE»
+                «attribute.stubDefaultClassTrySetMethodName»(std::move(_value));
                 «ENDIF»
-            }
+           }
 
             bool «fInterface.stubDefaultClassName»::«attribute.stubDefaultClassTrySetMethodName»(«typeName» _value) {
                 if (!«attribute.stubDefaultClassValidateMethodName»(_value))