carlwgeorge / rpms / php

Forked from rpms/php 5 years ago
Clone
Remi Collet 250e9e4
From 657494235eafe048e9fa6a19dcdb3c73a0cbe6ec Mon Sep 17 00:00:00 2001
Remi Collet 250e9e4
From: Remi Collet <fedora@famillecollet.com>
Remi Collet 250e9e4
Date: Thu, 27 Sep 2012 13:45:32 +0200
Remi Collet 250e9e4
Subject: [PATCH] Fixed bug #63171, script hangs if odbc call during timeout
Remi Collet 250e9e4
Remi Collet 250e9e4
---
Remi Collet 250e9e4
 ext/odbc/php_odbc.c |   21 ++++++++++++++-------
Remi Collet 250e9e4
 1 file changed, 14 insertions(+), 7 deletions(-)
Remi Collet 250e9e4
Remi Collet 250e9e4
diff --git a/ext/odbc/php_odbc.c b/ext/odbc/php_odbc.c
Remi Collet 250e9e4
index 52d46b2..2169e65 100644
Remi Collet 250e9e4
--- a/ext/odbc/php_odbc.c
Remi Collet 250e9e4
+++ b/ext/odbc/php_odbc.c
Remi Collet 250e9e4
@@ -431,7 +431,8 @@ static void _free_odbc_result(zend_rsrc_list_entry *rsrc TSRMLS_DC)
Remi Collet 250e9e4
 			efree(res->values);
Remi Collet 250e9e4
 			res->values = NULL;
Remi Collet 250e9e4
 		}
Remi Collet 250e9e4
-		if (res->stmt) {
Remi Collet 250e9e4
+		/* If aborted via timer expiration, don't try to call any unixODBC function */
Remi Collet 250e9e4
+		if (res->stmt && !(PG(connection_status) & PHP_CONNECTION_TIMEOUT)) {
Remi Collet 250e9e4
 #if defined(HAVE_SOLID) || defined(HAVE_SOLID_30) || defined(HAVE_SOLID_35)
Remi Collet 250e9e4
 			SQLTransact(res->conn_ptr->henv, res->conn_ptr->hdbc,
Remi Collet 250e9e4
 						(SQLUSMALLINT) SQL_COMMIT);
Remi Collet 250e9e4
@@ -484,9 +485,12 @@ static void _close_odbc_conn(zend_rsrc_list_entry *rsrc TSRMLS_DC)
Remi Collet 250e9e4
 		}
Remi Collet 250e9e4
 	}
Remi Collet 250e9e4
 
Remi Collet 250e9e4
-   	safe_odbc_disconnect(conn->hdbc);
Remi Collet 250e9e4
-	SQLFreeConnect(conn->hdbc);
Remi Collet 250e9e4
-	SQLFreeEnv(conn->henv);
Remi Collet 250e9e4
+	/* If aborted via timer expiration, don't try to call any unixODBC function */
Remi Collet 250e9e4
+	if (!(PG(connection_status) & PHP_CONNECTION_TIMEOUT)) {
Remi Collet 250e9e4
+		safe_odbc_disconnect(conn->hdbc);
Remi Collet 250e9e4
+		SQLFreeConnect(conn->hdbc);
Remi Collet 250e9e4
+		SQLFreeEnv(conn->henv);
Remi Collet 250e9e4
+	}
Remi Collet 250e9e4
 	efree(conn);
Remi Collet 250e9e4
 	ODBCG(num_links)--;
Remi Collet 250e9e4
 }
Remi Collet 250e9e4
@@ -512,9 +516,12 @@ static void _close_odbc_pconn(zend_rsrc_list_entry *rsrc TSRMLS_DC)
Remi Collet 250e9e4
 		}
Remi Collet 250e9e4
 	}
Remi Collet 250e9e4
 	
Remi Collet 250e9e4
-	safe_odbc_disconnect(conn->hdbc);
Remi Collet 250e9e4
-	SQLFreeConnect(conn->hdbc);
Remi Collet 250e9e4
-	SQLFreeEnv(conn->henv);
Remi Collet 250e9e4
+	/* If aborted via timer expiration, don't try to call any unixODBC function */
Remi Collet 250e9e4
+	if (!(PG(connection_status) & PHP_CONNECTION_TIMEOUT)) {
Remi Collet 250e9e4
+		safe_odbc_disconnect(conn->hdbc);
Remi Collet 250e9e4
+		SQLFreeConnect(conn->hdbc);
Remi Collet 250e9e4
+		SQLFreeEnv(conn->henv);
Remi Collet 250e9e4
+	}
Remi Collet 250e9e4
 	free(conn);
Remi Collet 250e9e4
 
Remi Collet 250e9e4
 	ODBCG(num_links)--;
Remi Collet 250e9e4
-- 
Remi Collet 250e9e4
1.7.10
Remi Collet 250e9e4