postgresql: any execute should close previous ResultSet

not only executeQuery.

Change-Id: I2ee72078c48f622484dba46436f53990a45da364

connectivity/source/drivers/postgresql/pq_preparedstatement.cxx

@@ -278,10 +278,6 @@ void PreparedStatement::raiseSQLException( const char * errorMsg )
 
 Reference< XResultSet > PreparedStatement::executeQuery( )
 {
-    Reference< XCloseable > lastResultSet = m_lastResultset;
-    if( lastResultSet.is() )
-        lastResultSet->close();
-
     if( ! execute( ) )
     {
         raiseSQLException(  "not a query" );
@@ -357,6 +353,10 @@ sal_Bool PreparedStatement::execute( )
 
     m_executedStatement = buf.makeStringAndClear();
 
+    Reference< XCloseable > lastResultSet = m_lastResultset;
+    if( lastResultSet.is() )
+        lastResultSet->close();
+
     m_lastResultset.clear();
     m_lastTableInserted.clear();
 

connectivity/source/drivers/postgresql/pq_statement.cxx

@@ -235,10 +235,6 @@ void Statement::raiseSQLException(
 
 Reference< XResultSet > Statement::executeQuery(const OUString& sql )
 {
-    Reference< XCloseable > lastResultSetHolder = m_lastResultset;
-    if( lastResultSetHolder.is() )
-        lastResultSetHolder->close();
-
     if( ! execute( sql ) )
     {
         raiseSQLException( sql, "not a query" );
@@ -805,6 +801,10 @@ sal_Bool Statement::execute( const OUString& sql )
     checkClosed();
     OString cmd = OUStringToOString( sql, m_pSettings );
 
+    Reference< XCloseable > lastResultSetHolder = m_lastResultset;
+    if( lastResultSetHolder.is() )
+        lastResultSetHolder->close();
+
     m_lastResultset.clear();
     m_lastTableInserted.clear();