From 100be67aa408e03c4f344dbb63ed418291eb661e Mon Sep 17 00:00:00 2001 From: ika Date: Fri, 26 Oct 2018 18:04:10 +0300 Subject: [PATCH] 0030315: Data Exchange - Crash reading views during STEP import. Add NULL check to protect against crash. --- src/STEPCAFControl/STEPCAFControl_Reader.cxx | 4 ++++ tests/gdt/view/B7 | 9 +++++++++ 2 files changed, 13 insertions(+) create mode 100644 tests/gdt/view/B7 diff --git a/src/STEPCAFControl/STEPCAFControl_Reader.cxx b/src/STEPCAFControl/STEPCAFControl_Reader.cxx index cdd20cb327..5d19346fd5 100644 --- a/src/STEPCAFControl/STEPCAFControl_Reader.cxx +++ b/src/STEPCAFControl/STEPCAFControl_Reader.cxx @@ -4117,6 +4117,10 @@ Handle(TCollection_HAsciiString) buildClippingPlanes(const Handle(StepGeom_Geome if (theClippingCameraModel->IsKind(STANDARD_TYPE(StepVisual_CameraModelD3MultiClipping))) { Handle(StepVisual_CameraModelD3MultiClipping) aCameraModel = Handle(StepVisual_CameraModelD3MultiClipping)::DownCast(theClippingCameraModel); + + if (aCameraModel->ShapeClipping().IsNull()) + return anExpression; + // Root of clipping planes tree if (aCameraModel->ShapeClipping()->Length() == 1) { Handle(StepVisual_CameraModelD3MultiClippingUnion) aCameraModelUnion = diff --git a/tests/gdt/view/B7 b/tests/gdt/view/B7 new file mode 100644 index 0000000000..b585a6327f --- /dev/null +++ b/tests/gdt/view/B7 @@ -0,0 +1,9 @@ +set filename bug30315.stp + +set ref_data { +D_First 0:1:7:1 0 0 0 "ALL" parallel 0 0 1.9593266248700001 0 0 -1 1 -5.5511151231299994e-017 0 0 1000 1000 +D_First 0:1:7:2 0 0 0 "CPC Process" parallel 0 0 1.9593266248700001 0 0 -1 1 -5.5511151231299994e-017 0 0 1000 1000 +D_First 0:1:7:3 0 0 0 "CPC D\\X2\\00E9\\X0\\tente" parallel 0 0 1.9593266248700001 0 0 -1 1 -5.5511151231299994e-017 0 0 1000 1000 +D_First 0:1:7:4 0 0 0 "REF A/B/C" parallel 0 0 1.9593266248700001 0 0 -1 1 -5.5511151231299994e-017 0 0 1000 1000 +D_First 0:1:7:5 0 0 0 "CTF A/B/C" parallel 0 0 1.9593266248700001 0 0 -1 1 -5.5511151231299994e-017 0 0 1000 1000 +}