[gdal] 01/04: Add upstream patch to fix crash on URLs that are not DODS servers.

[Bas Couwenberg]

This is an automated email from the git hooks/post-receive script.

sebastic pushed a commit to branch master
in repository gdal.

commit bfcbc09ee001b74bdd8b88e062150c6e150c30c2
Author: Bas Couwenberg <sebastic at xs4all.nl>
Date:   Wed Nov 9 18:36:27 2016 +0100

    Add upstream patch to fix crash on URLs that are not DODS servers.
---
 debian/changelog                                   |  7 +++++
 debian/patches/series                              |  1 +
 ...ix-crash-on-URL-that-are-not-DODS-servers.patch | 30 ++++++++++++++++++++++
 3 files changed, 38 insertions(+)

diff --git a/debian/changelog b/debian/changelog
index 8ae77a8..2a31e57 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+gdal (2.1.2+dfsg-2) UNRELEASED; urgency=medium
+
+  * Add upstream patch to fix crash on URLs that are not DODS servers.
+    (LP: #1640360)
+
+ -- Bas Couwenberg <sebastic at debian.org>  Wed, 09 Nov 2016 18:35:14 +0100
+
 gdal (2.1.2+dfsg-1) unstable; urgency=medium
 
   * Move from experimental to unstable.
diff --git a/debian/patches/series b/debian/patches/series
index d0d5927..aeefe2e 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -10,3 +10,4 @@ perl-vendor
 privacy-breach-logo.patch
 sort-files-in-static-library.patch
 spelling-errors.patch
+svn-r36175-DODS-fix-crash-on-URL-that-are-not-DODS-servers.patch
diff --git a/debian/patches/svn-r36175-DODS-fix-crash-on-URL-that-are-not-DODS-servers.patch b/debian/patches/svn-r36175-DODS-fix-crash-on-URL-that-are-not-DODS-servers.patch
new file mode 100644
index 0000000..e3b1ce1
--- /dev/null
+++ b/debian/patches/svn-r36175-DODS-fix-crash-on-URL-that-are-not-DODS-servers.patch
@@ -0,0 +1,30 @@
+Description: DODS: fix crash on URL that are not DODS servers
+Author: Even Rouault <even dot rouault at spatialys dot com>
+Origin: https://trac.osgeo.org/gdal/changeset/36175
+Bug: https://trac.osgeo.org/gdal/ticket/6718
+Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/gdal/+bug/1640360
+
+--- a/frmts/dods/dodsdataset2.cpp
++++ b/frmts/dods/dodsdataset2.cpp
+@@ -496,6 +496,8 @@ char **DODSDataset::CollectBandsFromDDSV
+ /* -------------------------------------------------------------------- */
+     BaseType *poVar = get_variable( GetDDS(), oVarName );
+ 
++    if( poVar == NULL )
++        return papszResultList;
+     if( poVar->type() == dods_array_c )
+     {
+         poGrid = NULL;
+@@ -1024,7 +1026,11 @@ DODSDataset::Open(GDALOpenInfo *poOpenIn
+ /*      Did we get any target variables?                                */
+ /* -------------------------------------------------------------------- */
+         if( CSLCount(papszVarConstraintList) == 0 )
+-            throw Error( "No apparent raster grids or arrays found in DDS.");
++        {
++            CPLDebug( "DODS", "No apparent raster grids or arrays found in DDS.");
++            delete poDS;
++            return NULL;
++        }
+ 
+ /* -------------------------------------------------------------------- */
+ /*      For now we support only a single band.                          */

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-grass/gdal.git