0033369: Modeling Algorithms - BRepBuilderAPI_Transform makes invalid shape after transformation

Added check to avoid setting parametric boundaries to infinity.

src/BRepTools/BRepTools_TrsfModification.cxx

@@ -182,11 +182,11 @@ Standard_Boolean BRepTools_TrsfModification::NewCurve2d
     if(l - lc > Precision::PConfusion()) l = lc;
     if(Abs(l - f) < Precision::PConfusion())
     {
-      if(Abs(f - fc) < Precision::PConfusion())
+      if(Abs(f - fc) < Precision::PConfusion() && !Precision::IsInfinite(lc))
       {
         l = lc;
       }
-      else
+      else if (!Precision::IsInfinite(fc))
       {
         f = fc;
       }

src/ShapeProcess/ShapeProcess_Context.cxx

@@ -20,12 +20,15 @@
 #include <ShapeProcess_Context.hxx>
 #include <Standard_ErrorHandler.hxx>
 #include <Standard_Failure.hxx>
+#include <Standard_Mutex.hxx>
 #include <Standard_Type.hxx>
 #include <TCollection_AsciiString.hxx>
 #include <TCollection_HAsciiString.hxx>
 
 #include <sys/stat.h>
-IMPLEMENT_STANDARD_RTTIEXT(ShapeProcess_Context,Standard_Transient)
+IMPLEMENT_STANDARD_RTTIEXT(ShapeProcess_Context, Standard_Transient)
+
+static Standard_Mutex THE_SHAPE_PROCESS_MUTEX;
 
 //=======================================================================
 //function : ShapeProcess_Context
@@ -73,6 +76,7 @@ Standard_Boolean ShapeProcess_Context::Init (const Standard_CString file,
 
 Handle(Resource_Manager) ShapeProcess_Context::LoadResourceManager (const Standard_CString name)
 {
+  Standard_Mutex::Sentry aLock(&THE_SHAPE_PROCESS_MUTEX);
   // Optimisation of loading resource file: file is load only once
   // and reloaded only if file date has changed
   static Handle(Resource_Manager) sRC;
@@ -127,7 +131,7 @@ Handle(Resource_Manager) ShapeProcess_Context::LoadResourceManager (const Standa
       sUMtime = aUMtime;
     }
   }
-  return sRC;
+  return new Resource_Manager(*sRC);
 }
 
 //=======================================================================

src/XSAlgo/XSAlgo_AlgoContainer.cxx

@@ -64,7 +64,9 @@
 #include <XSAlgo_ToolContainer.hxx>
 #include <TopExp_Explorer.hxx>
 
-IMPLEMENT_STANDARD_RTTIEXT(XSAlgo_AlgoContainer,Standard_Transient)
+IMPLEMENT_STANDARD_RTTIEXT(XSAlgo_AlgoContainer, Standard_Transient)
+
+static Standard_Mutex THE_RESOURCE_MUTEX;
 
 //=======================================================================
 //function : XSAlgo_AlgoContainer
@@ -167,9 +169,11 @@ TopoDS_Shape XSAlgo_AlgoContainer::ProcessShape (const TopoDS_Shape& shape,
   }
   
   // Define runtime tolerances and do Shape Processing 
-  rsc->SetResource ( "Runtime.Tolerance", Prec );
-  rsc->SetResource ( "Runtime.MaxTolerance", maxTol );
-
+  {
+    Standard_Mutex::Sentry aLock(&THE_RESOURCE_MUTEX);
+    rsc->SetResource("Runtime.Tolerance", Prec);
+    rsc->SetResource("Runtime.MaxTolerance", maxTol);
+  }
   if ( !ShapeProcess::Perform(context, seq, theProgress) )
     return shape; // return original shape
 

tests/bugs/modalg_7/bug33369

@@ -0,0 +1,11 @@
+puts "======================================================="
+puts "0033369: OCCT:Modeling Algorithms - BRepBuilderAPI_Transform makes invalid shape after transformation"
+puts "======================================================="
+puts ""
+
+restore [locate_data_file bug33369.brep] f
+
+ttranslate f 1 1 1 -copy
+
+checkshape f
+