Merge topic 'fix-exodus-crash'

8f73971 Opening a non-existent filename should not crash.

Acked-by: Kitware Robot <kwrobot@kitware.com>
Acked-by: T.J. Corona <tj.corona@kitware.com>
Merge-request: !4866

IO/Exodus/vtkExodusIIReader.cxx

@@ -3847,6 +3847,12 @@ int vtkExodusIIReaderPrivate::OpenFile( const char* filename )
 
   this->Exoid = ex_open( filename, EX_READ,
     &this->AppWordSize, &this->DiskWordSize, &this->ExodusVersion );
+  if ( this->Exoid <= 0 )
+  {
+    vtkErrorMacro( "Unable to open \"" << filename << "\" for reading" );
+    return 0;
+  }
+
 #ifdef VTK_USE_64BIT_IDS
   // Set the exodus API to always return integer types as 64-bit
   // without this call, large exodus files are not supported (which
@@ -3860,12 +3866,6 @@ int vtkExodusIIReaderPrivate::OpenFile( const char* filename )
   // in the ex_set_max_name_length() function.
   ex_set_max_name_length(this->Exoid, this->Parent->GetMaxNameLength());
 
-  if ( this->Exoid <= 0 )
-  {
-    vtkErrorMacro( "Unable to open \"" << filename << "\" for reading" );
-    return 0;
-  }
-
   vtkIdType numNodesInFile;
   char dummyChar;
   float dummyFloat;