Other check for null pointer before calling decref

qgis · May 23, 2016 · 9b2be7a · 9b2be7a
1 parent a1e826e
commit 9b2be7a
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/src/python/qgspythonutilsimpl.cpp b/src/python/qgspythonutilsimpl.cpp
@@ -580,11 +580,13 @@ bool QgsPythonUtilsImpl::evalString( const QString& command, QString& result )
   bool success = nullptr != res;
 
   // TODO: error handling
+  if ( res )
+  {
+    if ( success )
+      result = PyObjectToQString( res );
 
-  if ( success )
-    result = PyObjectToQString( res );
-
-  Py_XDECREF( res );
+    Py_XDECREF( res );
+  }
 
   // we are done calling python API, release global interpreter lock
   PyGILState_Release( gstate );