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

0033603: Data Exchange, Step Import - Crash reading corrupted STEP file

Browse Source 
Crash fixed by checking IsNull().
This commit is contained in:
ichesnok 2024-03-11 10:13:34 +00:00 committed by vglukhik
parent d1eae5b0d0
commit 3888b58c27
2 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/StepToGeom/StepToGeom.cxx
View File

@ -1008,7 +1008,7 @@ Handle(Geom_BSplineSurface) StepToGeom::MakeBSplineSurface (const Handle(StepGeo
Handle(Geom_CartesianPoint) StepToGeom::MakeCartesianPoint (const Handle(StepGeom_CartesianPoint)& SP, Handle(Geom_CartesianPoint) StepToGeom::MakeCartesianPoint (const Handle(StepGeom_CartesianPoint)& SP,
const StepData_Factors& theLocalFactors) const StepData_Factors& theLocalFactors)
{ {
if (SP->NbCoordinates() == 3) if (!SP.IsNull() && SP->NbCoordinates() == 3)
{ {
const Standard_Real LF = theLocalFactors.LengthFactor(); const Standard_Real LF = theLocalFactors.LengthFactor();
const Standard_Real X = SP->CoordinatesValue(1) * LF; const Standard_Real X = SP->CoordinatesValue(1) * LF;

5
src/StepToTopoDS/StepToTopoDS_TranslateVertex.cxx
View File

@ -96,6 +96,11 @@ void StepToTopoDS_TranslateVertex::Init(const Handle(StepShape_Vertex)& aVertex,
//:S4136 Standard_Real preci = BRepAPI::Precision(); //:S4136 Standard_Real preci = BRepAPI::Precision();
const Handle(StepShape_VertexPoint) VP = Handle(StepShape_VertexPoint)::DownCast(aVertex); const Handle(StepShape_VertexPoint) VP = Handle(StepShape_VertexPoint)::DownCast(aVertex);
const Handle(StepGeom_Point) P = VP->VertexGeometry(); const Handle(StepGeom_Point) P = VP->VertexGeometry();
if (P.IsNull()) {
myError = StepToTopoDS_TranslateVertexOther;
done = Standard_False;
return;
}
const Handle(StepGeom_CartesianPoint) P1 = Handle(StepGeom_CartesianPoint)::DownCast(P); const Handle(StepGeom_CartesianPoint) P1 = Handle(StepGeom_CartesianPoint)::DownCast(P);
Handle(Geom_CartesianPoint) P2 = StepToGeom::MakeCartesianPoint (P1, theLocalFactors); Handle(Geom_CartesianPoint) P2 = StepToGeom::MakeCartesianPoint (P1, theLocalFactors);
BRep_Builder B; BRep_Builder B;