From: Raúl Marín Rodríguez <rmrodriguez@carto.com>
Date: Tue, 2 Oct 2018 09:45:39 +0000 (+0000)
Subject: Fix undefined behaviour in SADFWrite
X-Git-Tag: 2.3.8~15
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=475cf39deddea27633a33d86b1613df188181dc3;p=postgis

Fix undefined behaviour in SADFWrite

References #4189


git-svn-id: http://svn.osgeo.org/postgis/branches/2.3@16865 b70326c6-7e19-0410-871a-916f4a2858ee
---

diff --git a/NEWS b/NEWS
index 66a19121b..66c0fc2ed 100644
--- a/NEWS
+++ b/NEWS
@@ -8,6 +8,7 @@ PostGIS 2.3.8
   - #3997, fix bug in lwgeom_median and avoid division by zero (Raúl Marín)
   - #4093, Inconsistent results from qsort callback (yugr)
   - #4160, Use qualified names in topology extension install (Raúl Marín)
+  - #4189, Fix undefined behaviour in SADFWrite (Raúl Marín)
 
 
 PostGIS 2.3.7
diff --git a/loader/safileio.c b/loader/safileio.c
index 439ebb141..ee5a98afb 100644
--- a/loader/safileio.c
+++ b/loader/safileio.c
@@ -115,8 +115,9 @@ SAOffset SADFRead( void *p, SAOffset size, SAOffset nmemb, SAFile file )
 SAOffset SADFWrite( void *p, SAOffset size, SAOffset nmemb, SAFile file )
 
 {
-    return (SAOffset) fwrite( p, (size_t) size, (size_t) nmemb,
-                              (FILE *) file );
+	if (!nmemb || !p) return 0;
+	return (SAOffset) fwrite( p, (size_t) size, (size_t) nmemb,
+				(FILE *) file );
 }
 
 /************************************************************************/