0032949: Modeling Algorithms - Infinite loop in ShapeUpgrade_UnifySameDomain

Problem occurs when InternalEdges is empty but the vertices from VV is not same.
Added condition to stop the loop when InternalEdges is empty.

diff --git a/src/ShapeUpgrade/ShapeUpgrade_UnifySameDomain.cxx b/src/ShapeUpgrade/ShapeUpgrade_UnifySameDomain.cxx
index e17fefa111c5635af6dc26258a427da61e56598d..d02ef6482477f90ad8f46fa995fbfd7980fdc626 100644 (file)
--- a/src/ShapeUpgrade/ShapeUpgrade_UnifySameDomain.cxx
+++ b/src/ShapeUpgrade/ShapeUpgrade_UnifySameDomain.cxx
@@ -3708,8 +3708,11 @@ void ShapeUpgrade_UnifySameDomain::IntUnifyFaces(const TopoDS_Shape& theInpShape
               TopoDS_Edge anEdge = TopoDS::Edge(itl.Value());
               if (anEdge.IsSame(EndEdges[ii]))
                 continue;
-              found = Standard_True;
-              InternalEdges.RemoveKey(anEdge);
+              found = InternalEdges.RemoveKey(anEdge);
+              if (!found)
+              {
+                break;
+              }
               BB.Add(anInternalWire, anEdge);
               TopoDS_Vertex V1, V2;
               TopExp::Vertices(anEdge, V1, V2);