From: nmanchen <nmanchen@opencascade.com>
Date: Sun, 4 Dec 2022 18:19:02 +0000 (+0300)
Subject: CR28720: Modeling Algorithms - HLRBrep_PolyAlgo crash with specific shape
X-Git-Url: http://git.dev.opencascade.org/gitweb/?a=commitdiff_plain;h=refs%2Fheads%2FCR28720_1;p=occt.git

CR28720: Modeling Algorithms - HLRBrep_PolyAlgo crash with specific shape

1 link update for each new segment
---

diff --git a/src/HLRAlgo/HLRAlgo_PolyInternalData.cxx b/src/HLRAlgo/HLRAlgo_PolyInternalData.cxx
index 3eca3e98d6..ca272107b8 100644
--- a/src/HLRAlgo/HLRAlgo_PolyInternalData.cxx
+++ b/src/HLRAlgo/HLRAlgo_PolyInternalData.cxx
@@ -78,11 +78,13 @@ void HLRAlgo_PolyInternalData::UpdateLinks (HLRAlgo_Array1OfTData& theTData,
     HLRAlgo_PolyInternalNode::NodeIndices& A2 = thePINod.ChangeValue(TD->Node2)->Indices();
     HLRAlgo_PolyInternalNode::NodeIndices& A3 = thePINod.ChangeValue(TD->Node3)->Indices();
 
+    bool notFound = true;
     {
     n1 = TD->Node1;
     n2 = TD->Node2;
     newSeg = Standard_False;
     if (A1.NdSg == 0 && A2.NdSg == 0) {
+      notFound = false;
       newSeg = Standard_True;
       myNbPISeg++;
       A1.NdSg = myNbPISeg;
@@ -107,6 +109,7 @@ void HLRAlgo_PolyInternalData::UpdateLinks (HLRAlgo_Array1OfTData& theTData,
 	  }
 	}
 	if (find == 0) {
+	  notFound = false;
 	  newSeg = Standard_True;
 	  myNbPISeg++;
 	  if (icsv == 1) aSegIndices->NxtSg1 = myNbPISeg;
@@ -115,6 +118,7 @@ void HLRAlgo_PolyInternalData::UpdateLinks (HLRAlgo_Array1OfTData& theTData,
 	else aSegIndices->Conex2 = i;
       }
       else {
+	notFound = false;
 	newSeg = Standard_True;
 	myNbPISeg++;
 	A1.NdSg = myNbPISeg;
@@ -145,11 +149,13 @@ void HLRAlgo_PolyInternalData::UpdateLinks (HLRAlgo_Array1OfTData& theTData,
     }
     }
 
+    if (notFound)
     {
     n1 = TD->Node2;
     n2 = TD->Node3;
     newSeg = Standard_False;
     if (A2.NdSg == 0 && A3.NdSg == 0) {
+      notFound = false;
       newSeg = Standard_True;
       myNbPISeg++;
       A2.NdSg = myNbPISeg;
@@ -174,6 +180,7 @@ void HLRAlgo_PolyInternalData::UpdateLinks (HLRAlgo_Array1OfTData& theTData,
 	  }
 	}
 	if (find == 0) {
+	  notFound = false;
 	  newSeg = Standard_True;
 	  myNbPISeg++;
 	  if (icsv == 1) aSegIndices->NxtSg1 = myNbPISeg;
@@ -182,6 +189,7 @@ void HLRAlgo_PolyInternalData::UpdateLinks (HLRAlgo_Array1OfTData& theTData,
 	else aSegIndices->Conex2 = i;
       }
       else {
+	notFound = false;
 	newSeg = Standard_True;
 	myNbPISeg++;
 	A2.NdSg = myNbPISeg;
@@ -212,11 +220,13 @@ void HLRAlgo_PolyInternalData::UpdateLinks (HLRAlgo_Array1OfTData& theTData,
     }
     }
 
+    if (notFound)
     {
     n1 = TD->Node3;
     n2 = TD->Node1;
     newSeg = Standard_False;
     if (A3.NdSg == 0 && A1.NdSg == 0) {
+      notFound = false;
       newSeg = Standard_True;
       myNbPISeg++;
       A3.NdSg = myNbPISeg;
@@ -241,6 +251,7 @@ void HLRAlgo_PolyInternalData::UpdateLinks (HLRAlgo_Array1OfTData& theTData,
 	  }
 	}
 	if (find == 0) {
+	  notFound = false;
 	  newSeg = Standard_True;
 	  myNbPISeg++;
 	  if (icsv == 1) aSegIndices->NxtSg1 = myNbPISeg;
@@ -249,6 +260,7 @@ void HLRAlgo_PolyInternalData::UpdateLinks (HLRAlgo_Array1OfTData& theTData,
 	else aSegIndices->Conex2 = i;
       }
       else {
+	notFound = false;
 	newSeg = Standard_True;
 	myNbPISeg++;
 	A3.NdSg = myNbPISeg;