OpenCascade/occt
1
0
Fork 0
mirror of https://git.dev.opencascade.org/repos/occt.git synced 2025-08-04 13:13:25 +03:00
Code Activity

0026308: Segmentation fault in BSplCLib::LocateParameter

Browse Source 
Detection of "jumping" knot value has been improved.
Test-case for issue #26308
Comment has been added in test case bugs/modalg_6/bug26308.
This commit is contained in:
nbv
2015-07-20 15:23:00 +03:00
committed by abv
parent 534beb0a8d
commit efe826bff1
2 changed files with 56 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
src/BSplCLib/BSplCLib.cxx
View File

@@ -229,6 +229,18 @@ void BSplCLib::LocateParameter
const Standard_Real UFirst,
const Standard_Real ULast)
{
/*
Let Knots are distributed as follows (the array is sorted in ascending order):
K1, K1,..., K1, K1, K2, K2,..., K2, K2,..., Kn, Kn,..., Kn
M1 times M2 times Mn times
NbKnots = sum(M1+M2+...+Mn)
If U <= K1 then KnotIndex should be equal to M1.
If U >= Kn then KnotIndex should be equal to NbKnots-Mn-1.
If Ki <= U < K(i+1) then KnotIndex should be equal to sum (M1+M2+...+Mi).
*/
Standard_Integer First,Last;
if (FromK1 < ToK2) {
First = FromK1;
@@ -252,10 +264,12 @@ void BSplCLib::LocateParameter
BSplCLib::Hunt (Knots, NewU, KnotIndex);
Standard_Real Eps = Epsilon(U);
Standard_Real val;
if (Eps < 0) Eps = - Eps;
Standard_Integer KLower = Knots.Lower();
const Standard_Integer KLower = Knots.Lower(),
KUpper = Knots.Upper();
const Standard_Real Eps = Epsilon(Min(Abs(Knots(KUpper)), Abs(U)));
const Standard_Real *knots = &Knots(KLower);
knots -= KLower;
if ( KnotIndex < Knots.Upper()) {