OpenCascade/occt
1
0
Fork 0
mirror of https://git.dev.opencascade.org/repos/occt.git synced 2025-04-07 18:30:55 +03:00
Code Packages Releases Activity

0026605: Possible array out of bounds read in Extrema_GExtPC.gxx

Browse Source 
Degenerated case fixed.
This commit is contained in:
aml 2015-08-27 20:24:12 +03:00 committed by bugmaster
parent ee2038c00a
commit 1581d6511b
1 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/Extrema/Extrema_GExtPC.gxx
View File

@ -120,7 +120,7 @@ void Extrema_GExtPC::Perform(const ThePoint& P)
Standard_Integer anIdx; Standard_Integer anIdx;
// Find first and last used knot // Find first and last used knot.
Standard_Integer aFirstUsedKnot = aFirstIdx, Standard_Integer aFirstUsedKnot = aFirstIdx,
aLastUsedKnot = aLastIdx; aLastUsedKnot = aLastIdx;
for(anIdx = aFirstIdx; anIdx <= aLastIdx; anIdx++) for(anIdx = aFirstIdx; anIdx <= aLastIdx; anIdx++)
@ -141,6 +141,16 @@ void Extrema_GExtPC::Perform(const ThePoint& P)
break; break;
} }
if (aFirstUsedKnot == aLastUsedKnot)
{
// Degenerated case:
// Some bounds lies out of curve param space.
// In this case build one interval with [myuinf, myusup].
// Parameters of these indexes will be redefined.
aFirstUsedKnot = aFirstIdx;
aLastUsedKnot = aFirstIdx + 1;
}
mysample = (TheCurveTool::BSpline(aCurve))->Degree() + 1; mysample = (TheCurveTool::BSpline(aCurve))->Degree() + 1;
// Fill sample points. // Fill sample points.