0029633: Data Exchange - access violation in StepVisual_PresentationStyleAssignment...

Add several checks for null.

diff --git a/src/STEPCAFControl/STEPCAFControl_Reader.cxx b/src/STEPCAFControl/STEPCAFControl_Reader.cxx
index c282e261211aa2660615c389692035f781274dcf..75fd04b0df8a339c49b4efdfbb87d25b9c6f4984 100644 (file)
--- a/src/STEPCAFControl/STEPCAFControl_Reader.cxx
+++ b/src/STEPCAFControl/STEPCAFControl_Reader.cxx
@@ -3415,7 +3415,7 @@ static void setDimObjectToXCAF(const Handle(Standard_Transient)& theEnt,
     aDimObj->SetClassOfTolerance(aHolle, aFV, aG);
   }
 
-  if (!aCRI.IsNull() && aCRI->ItemElement()->Length() > 0)
+  if (!aCRI.IsNull() && !aCRI->ItemElement().IsNull() && aCRI->ItemElement()->Length() > 0)
   {
     //get modifiers
     XCAFDimTolObjects_DimensionModifiersSequence aModifiers;

diff --git a/src/StepVisual/StepVisual_PresentationStyleAssignment.cxx b/src/StepVisual/StepVisual_PresentationStyleAssignment.cxx
index f846b5daddba051370e7dd23effa96dfb872e72f..50a71fb7d7020e45303ae194a196dbcefe49a0bd 100644 (file)
--- a/src/StepVisual/StepVisual_PresentationStyleAssignment.cxx
+++ b/src/StepVisual/StepVisual_PresentationStyleAssignment.cxx
@@ -45,5 +45,5 @@ StepVisual_PresentationStyleSelect StepVisual_PresentationStyleAssignment::Style
 
 Standard_Integer StepVisual_PresentationStyleAssignment::NbStyles () const
 {
-       return styles->Length();
+  return (styles.IsNull()) ?  0 : styles->Length();
 }

diff --git a/src/StepVisual/StepVisual_SurfaceSideStyle.cxx b/src/StepVisual/StepVisual_SurfaceSideStyle.cxx
index f6510a2ac9e068e8594df86f2deaa7a102ab6801..a345ff7545ec6703849e674bfe7d4a22bd7338b9 100644 (file)
--- a/src/StepVisual/StepVisual_SurfaceSideStyle.cxx
+++ b/src/StepVisual/StepVisual_SurfaceSideStyle.cxx
@@ -58,5 +58,5 @@ StepVisual_SurfaceStyleElementSelect StepVisual_SurfaceSideStyle::StylesValue(co
 
 Standard_Integer StepVisual_SurfaceSideStyle::NbStyles () const
 {
-       return styles->Length();
+  return (styles.IsNull()) ? 0 : styles->Length();
 }

diff --git a/tests/gdt/export/C3 b/tests/gdt/export/C3
new file mode 100644 (file)
index 0000000..52cd6bd
--- /dev/null
+++ b/tests/gdt/export/C3
@@ -0,0 +1,19 @@
+# !!!! This file is generated automatically, do not edit manually! See end script
+set filename bug29633_nist_ctc_05_asme1_ap242-1.stp
+
+set ref_data {
+
+ NbOfDimensions          : 28
+  NbOfDimensionalSize    : 1
+  NbOfDimensionalLocation: 5
+  NbOfAngular            : 0
+  NbOfWithPath           : 0
+  NbOfCommonLabels       : 2
+ NbOfTolerances          : 10
+  NbOfGTWithModifiers    : 0
+  NbOfGTWithMaxTolerance : 0
+  NbOfGTWithDatums       : 8
+ NbOfDatumFeature        : 2
+ NbOfAttachedDatum       : 9
+ NbOfDatumTarget         : 0
+}