From a965c45b2d74b23c96414f55e3f16e02bcb22e0b Mon Sep 17 00:00:00 2001
From: gka <gka@opencascade.com>
Date: Thu, 27 Oct 2016 17:50:18 +0300
Subject: [PATCH] 0027404: Data Exchange - access violation when reading STEP
 file

Added check on the null handle
Test case for issue CR27404
---
 src/STEPControl/STEPControl_Reader.cxx | 19 +++++++++++--------
 tests/bugs/step/bug27404               | 14 ++++++++++++++
 2 files changed, 25 insertions(+), 8 deletions(-)
 create mode 100755 tests/bugs/step/bug27404

diff --git a/src/STEPControl/STEPControl_Reader.cxx b/src/STEPControl/STEPControl_Reader.cxx
index 3ef05c4785..4997b1127e 100644
--- a/src/STEPControl/STEPControl_Reader.cxx
+++ b/src/STEPControl/STEPControl_Reader.cxx
@@ -271,14 +271,17 @@ Standard_Integer STEPControl_Reader::NbRootsForTransfer()
                 Handle(StepShape_ShapeRepresentation)::DownCast(RR->Rep1());
               if(SR==SR2)
                 SR2 = Handle(StepShape_ShapeRepresentation)::DownCast(RR->Rep2());
-              Interface_EntityIterator subs2 = graph.Sharings(SR2);
-              for(subs2.Start(); subs2.More(); subs2.Next()) {
-                Handle(StepShape_ShapeDefinitionRepresentation) SDR2 = 
-                  Handle(StepShape_ShapeDefinitionRepresentation)::DownCast(subs2.Value());
-                if(!SDR2.IsNull()) IsRoot = Standard_False;
-                //else {
-                //  if(SR==SRR->Rep2()) IsRoot = Standard_False;
-                //}
+              if(!SR2.IsNull())
+              {
+                Interface_EntityIterator subs2 = graph.Sharings(SR2);
+                for(subs2.Start(); subs2.More(); subs2.Next()) {
+                  Handle(StepShape_ShapeDefinitionRepresentation) SDR2 = 
+                    Handle(StepShape_ShapeDefinitionRepresentation)::DownCast(subs2.Value());
+                  if(!SDR2.IsNull()) IsRoot = Standard_False;
+                  //else {
+                  //  if(SR==SRR->Rep2()) IsRoot = Standard_False;
+                  //}
+                }
               }
             }
           }
diff --git a/tests/bugs/step/bug27404 b/tests/bugs/step/bug27404
new file mode 100755
index 0000000000..096bda0396
--- /dev/null
+++ b/tests/bugs/step/bug27404
@@ -0,0 +1,14 @@
+puts "============"
+puts "OCC27404"
+puts "============"
+puts ""
+#######################################################################
+# Data Exchange - access violation when reading STEP file
+#######################################################################
+
+param read.step.product.mode 0
+newmodel
+
+XNewDoc D
+
+ReadStep D [locate_data_file bug27404_pmi.stp]