From 3f5bebe804e15e7d8c2ab7191f1d97dc4e82740c Mon Sep 17 00:00:00 2001 From: razmyslovich Date: Wed, 14 May 2014 12:28:15 +0200 Subject: [PATCH] 0024919: ShapeFix can crash time-to-time due to inproper interator handling inside of Approx_SameParameter::Build function Check inter variable to be inside of the desired range --- src/Approx/Approx_SameParameter.cxx | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/Approx/Approx_SameParameter.cxx b/src/Approx/Approx_SameParameter.cxx index 3fc41f3f17..3589ae0fc8 100644 --- a/src/Approx/Approx_SameParameter.cxx +++ b/src/Approx/Approx_SameParameter.cxx @@ -451,10 +451,10 @@ void Approx_SameParameter::Build(const Standard_Real Tolerance) ii =1; new_par.Append(fcons); - while(Param_de_decoupeC1(inter) <= fcons + deltamin) inter++; - while(Param_de_decoupeC1(NbInt) >= lcons - deltamin) NbInt--; + while(inter <= NbInt && Param_de_decoupeC1(inter) <= fcons + deltamin) inter++; + while(NbInt > 0 && Param_de_decoupeC1(NbInt) >= lcons - deltamin) NbInt--; - while(inter <= NbInt || ii < NCONTROL) { + while(inter <= NbInt || (ii < NCONTROL && inter <= Param_de_decoupeC1.Length()) ) { if(Param_de_decoupeC1(inter) < pcons[ii]) { new_par.Append(Param_de_decoupeC1(inter)); if((pcons[ii] - Param_de_decoupeC1(inter)) <= deltamin) {