EnablePaging methods: check return of Open() method to avoid assertions

Author: rouault

Diff for: mapmssql2008.c

@@ -3027,7 +3027,12 @@ void msMSSQL2008EnablePaging(layerObj *layer, int value)
     msMSSQL2008LayerInfo *layerinfo = NULL;
 
     if (!msMSSQL2008LayerIsOpen(layer))
-        msMSSQL2008LayerOpen(layer);
+    {
+      if(msMSSQL2008LayerOpen(layer) != MS_SUCCESS)
+      {
+        return;
+      }
+    }
 
     assert(layer->layerinfo != NULL);
     layerinfo = (msMSSQL2008LayerInfo *)layer->layerinfo;

Diff for: mapogr.cpp

@@ -5384,7 +5384,12 @@ static void msOGREnablePaging(layerObj *layer, int value)
   }
 
   if(!msOGRLayerIsOpen(layer))
-    msOGRLayerOpenVT(layer);
+  {
+    if(msOGRLayerOpenVT(layer) != MS_SUCCESS)
+    {
+      return;
+    }
+  }
 
   assert( layer->layerinfo != NULL);
 
@@ -5401,7 +5406,12 @@ static int msOGRGetPaging(layerObj *layer)
   }
 
   if(!msOGRLayerIsOpen(layer))
-    msOGRLayerOpenVT(layer);
+  {
+    if(msOGRLayerOpenVT(layer) != MS_SUCCESS)
+    {
+      return FALSE;
+    }
+  }
 
   assert( layer->layerinfo != NULL);
 

Diff for: maporaclespatial.c

@@ -3477,7 +3477,12 @@ void msOracleSpatialEnablePaging(layerObj *layer, int value)
     msDebug("msOracleSpatialLayerEnablePaging was called.\n");
 
   if(!msOracleSpatialLayerIsOpen(layer))
-    msOracleSpatialLayerOpen(layer);
+  {
+    if(msOracleSpatialLayerOpen(layer) != MS_SUCCESS)
+    {
+      return;
+    }
+  }
 
   assert( layer->layerinfo != NULL);
   layerinfo = (msOracleSpatialLayerInfo *)layer->layerinfo;

Diff for: mappostgis.c

@@ -3985,7 +3985,12 @@ void msPostGISEnablePaging(layerObj *layer, int value)
   }
 
   if(!msPostGISLayerIsOpen(layer))
-    msPostGISLayerOpen(layer);
+  {
+    if(msPostGISLayerOpen(layer) != MS_SUCCESS)
+    {
+      return;
+    }
+  }
 
   assert( layer->layerinfo != NULL);