From 51075421d3361c6ab69a72c7ec0ed9290147f999 Mon Sep 17 00:00:00 2001
From: Robert Osfield <robert@openscenegraph.com>
Date: Tue, 23 Jul 2013 09:54:25 +0000
Subject: [PATCH] Fixed crash when open(..) returns an error.

---
 src/osgPlugins/shp/XBaseParser.cpp | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/osgPlugins/shp/XBaseParser.cpp b/src/osgPlugins/shp/XBaseParser.cpp
index d9c563c6b..977c22508 100644
--- a/src/osgPlugins/shp/XBaseParser.cpp
+++ b/src/osgPlugins/shp/XBaseParser.cpp
@@ -87,13 +87,12 @@ XBaseParser::XBaseParser(const std::string fileName):
     if (fileName.empty() == false)
     {
 #ifdef WIN32
-        if( (fd = open( fileName.c_str(), O_RDONLY | O_BINARY )) <= 0 )
+        if( (fd = open( fileName.c_str(), O_RDONLY | O_BINARY )) < 0 )
 #else
-        if( (fd = ::open( fileName.c_str(), O_RDONLY )) <= 0 )
+        if( (fd = ::open( fileName.c_str(), O_RDONLY )) < 0 )
 #endif
         {
             perror( fileName.c_str() );
-            if (fd) close( fd );
             return;
         }
     }