From: abd <abd@opencascade.com>
Date: Fri, 21 Apr 2006 12:45:44 +0000 (+0000)
Subject: Fixed BUG with omniORB refCount. see file comment
X-Git-Tag: LAST_STABLE_VERSION_21_09_2006_ON_3_2_0~41
X-Git-Url: http://git.salome-platform.org/gitweb/?a=commitdiff_plain;h=4c5ba18e8dd5ca29e559137a9244c21f0c4604f1;p=modules%2Fgeom.git

Fixed BUG with omniORB refCount. see file comment
---

diff --git a/src/GEOM_I/GEOM_IOperations_i.cc b/src/GEOM_I/GEOM_IOperations_i.cc
index 8170be2f7..b0f652b2e 100644
--- a/src/GEOM_I/GEOM_IOperations_i.cc
+++ b/src/GEOM_I/GEOM_IOperations_i.cc
@@ -38,7 +38,16 @@
 GEOM_IOperations_i::GEOM_IOperations_i(PortableServer::POA_ptr thePOA, GEOM::GEOM_Gen_ptr theEngine, ::GEOM_IOperations* theImpl)
 :SALOME::GenericObj_i( thePOA ), _impl(theImpl), _engine(theEngine)
 {
-  thePOA->activate_object(this);
+  // Win32 porting: the next line is dangerous - GEOM_IOperations_i is an intermediate 
+  // base class, therefore <this> is not completely constructed here ->
+  // passing it to activate_object() leads to unpredictable behavior 
+  // resulted from memory corruption ( Rational Purify reports ABR errors ).
+  // Moreover, all GEOM_IxxxOperation_i servant classes are activated implicitly 
+  // by GEOM_Gen_i::GetxxxOperations() methods.
+  // Therefore, this line is commented.
+  // In case if <thePOA> does not have ImplicitActivation policy, then
+  // activate_object() calls will be necessary in all GEOM_IxxxOperation_i constructors!
+  //thePOA->activate_object(this);
 }
 
 //=============================================================================