Remove an unnecessary \0 pad to the end of the wkb char*. Also fix a

g_j_m · g_j_m · commit d454fdbebb9f · 2006-07-17T09:18:14.000Z
bug whereby the first byte after the wkb array was being set to \0 git-svn-id: http://svn.osgeo.org/qgis/trunk@5609 c8812cc2-4d05-0410-92ff-de0c093fc19c
diff --git a/src/providers/postgres/qgspostgresprovider.cpp b/src/providers/postgres/qgspostgresprovider.cpp
@@ -817,10 +817,9 @@ void QgsPostgresProvider::getFeatureGeometry(int key, QgsFeature *f)
 
   if(returnedLength > 0)
   {
-      unsigned char *wkbgeom = new unsigned char[returnedLength + 1];
-      memset(wkbgeom, '\0', returnedLength + 1);
+      unsigned char *wkbgeom = new unsigned char[returnedLength];
       memcpy(wkbgeom, PQgetvalue(geomResult, row, 0), returnedLength);
-      f->setGeometryAndOwnership(wkbgeom, returnedLength + 1);
+      f->setGeometryAndOwnership(wkbgeom, returnedLength);
   }
   else
   {

Original file line number	Diff line number	Diff line change
`@@ -817,10 +817,9 @@ void QgsPostgresProvider::getFeatureGeometry(int key, QgsFeature *f)`
`817`	`817`
`818`	`818`	`if(returnedLength > 0)`
`819`	`819`	`{`
`820`		`- unsigned char *wkbgeom = new unsigned char[returnedLength + 1];`
`821`		`- memset(wkbgeom, '\0', returnedLength + 1);`
	`820`	`+ unsigned char *wkbgeom = new unsigned char[returnedLength];`
`822`	`821`	`memcpy(wkbgeom, PQgetvalue(geomResult, row, 0), returnedLength);`
`823`		`- f->setGeometryAndOwnership(wkbgeom, returnedLength + 1);`
	`822`	`+ f->setGeometryAndOwnership(wkbgeom, returnedLength);`
`824`	`823`	`}`
`825`	`824`	`else`
`826`	`825`	`{`