From 2e5e7aa5880ec7a33731249b48aa04c3d7526f3d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?M=C3=A1t=C3=A9=20Kocsis?= <kocsismate@woohoolabs.com>
Date: Sat, 6 Apr 2024 23:02:15 +0200
Subject: [PATCH] Convert odbc_bindcols() function to void

---
 ext/odbc/php_odbc.c          | 78 ++++++++----------------------------
 ext/odbc/php_odbc_includes.h |  2 +-
 2 files changed, 17 insertions(+), 63 deletions(-)

diff --git a/ext/odbc/php_odbc.c b/ext/odbc/php_odbc.c
index 153c346124121..5981b93213170 100644
--- a/ext/odbc/php_odbc.c
+++ b/ext/odbc/php_odbc.c
@@ -529,7 +529,7 @@ void php_odbc_fetch_attribs(INTERNAL_FUNCTION_PARAMETERS, int mode)
 /* }}} */
 
 /* {{{ odbc_bindcols */
-int odbc_bindcols(odbc_result *result)
+void odbc_bindcols(odbc_result *result)
 {
 	RETCODE rc;
 	int i;
@@ -645,7 +645,6 @@ int odbc_bindcols(odbc_result *result)
 				break;
 		}
 	}
-	return 1;
 }
 /* }}} */
 
@@ -855,10 +854,7 @@ PHP_FUNCTION(odbc_prepare)
 	SQLNumResultCols(result->stmt, &(result->numcols));
 
 	if (result->numcols > 0) {
-		if (!odbc_bindcols(result)) {
-			efree(result);
-			RETURN_FALSE;
-		}
+		odbc_bindcols(result);
 	} else {
 		result->values = NULL;
 	}
@@ -1057,10 +1053,7 @@ PHP_FUNCTION(odbc_execute)
 		SQLNumResultCols(result->stmt, &(result->numcols));
 
 		if (result->numcols > 0) {
-			if (!odbc_bindcols(result)) {
-				efree(result);
-				RETVAL_FALSE;
-			}
+			odbc_bindcols(result);
 		} else {
 			result->values = NULL;
 		}
@@ -1253,10 +1246,7 @@ PHP_FUNCTION(odbc_exec)
 
 	/* For insert, update etc. cols == 0 */
 	if (result->numcols > 0) {
-		if (!odbc_bindcols(result)) {
-			efree(result);
-			RETURN_FALSE;
-		}
+		odbc_bindcols(result);
 	} else {
 		result->values = NULL;
 	}
@@ -2383,10 +2373,7 @@ PHP_FUNCTION(odbc_next_result)
 		SQLNumResultCols(result->stmt, &(result->numcols));
 
 		if (result->numcols > 0) {
-			if (!odbc_bindcols(result)) {
-				efree(result);
-				RETVAL_FALSE;
-			}
+			odbc_bindcols(result);
 		} else {
 			result->values = NULL;
 		}
@@ -2752,10 +2739,7 @@ PHP_FUNCTION(odbc_tables)
 	SQLNumResultCols(result->stmt, &(result->numcols));
 
 	if (result->numcols > 0) {
-		if (!odbc_bindcols(result)) {
-			efree(result);
-			RETURN_FALSE;
-		}
+		odbc_bindcols(result);
 	} else {
 		result->values = NULL;
 	}
@@ -2822,10 +2806,7 @@ PHP_FUNCTION(odbc_columns)
 	SQLNumResultCols(result->stmt, &(result->numcols));
 
 	if (result->numcols > 0) {
-		if (!odbc_bindcols(result)) {
-			efree(result);
-			RETURN_FALSE;
-		}
+		odbc_bindcols(result);
 	} else {
 		result->values = NULL;
 	}
@@ -2886,10 +2867,7 @@ PHP_FUNCTION(odbc_columnprivileges)
 	SQLNumResultCols(result->stmt, &(result->numcols));
 
 	if (result->numcols > 0) {
-		if (!odbc_bindcols(result)) {
-			efree(result);
-			RETURN_FALSE;
-		}
+		odbc_bindcols(result);
 	} else {
 		result->values = NULL;
 	}
@@ -2965,10 +2943,7 @@ PHP_FUNCTION(odbc_foreignkeys)
 	SQLNumResultCols(result->stmt, &(result->numcols));
 
 	if (result->numcols > 0) {
-		if (!odbc_bindcols(result)) {
-			efree(result);
-			RETURN_FALSE;
-		}
+		odbc_bindcols(result);
 	} else {
 		result->values = NULL;
 	}
@@ -3026,10 +3001,7 @@ PHP_FUNCTION(odbc_gettypeinfo)
 	SQLNumResultCols(result->stmt, &(result->numcols));
 
 	if (result->numcols > 0) {
-		if (!odbc_bindcols(result)) {
-			efree(result);
-			RETURN_FALSE;
-		}
+		odbc_bindcols(result);
 	} else {
 		result->values = NULL;
 	}
@@ -3087,10 +3059,7 @@ PHP_FUNCTION(odbc_primarykeys)
 	SQLNumResultCols(result->stmt, &(result->numcols));
 
 	if (result->numcols > 0) {
-		if (!odbc_bindcols(result)) {
-			efree(result);
-			RETURN_FALSE;
-		}
+		odbc_bindcols(result);
 	} else {
 		result->values = NULL;
 	}
@@ -3151,10 +3120,7 @@ PHP_FUNCTION(odbc_procedurecolumns)
 	SQLNumResultCols(result->stmt, &(result->numcols));
 
 	if (result->numcols > 0) {
-		if (!odbc_bindcols(result)) {
-			efree(result);
-			RETURN_FALSE;
-		}
+		odbc_bindcols(result);
 	} else {
 		result->values = NULL;
 	}
@@ -3214,10 +3180,7 @@ PHP_FUNCTION(odbc_procedures)
 	SQLNumResultCols(result->stmt, &(result->numcols));
 
 	if (result->numcols > 0) {
-		if (!odbc_bindcols(result)) {
-			efree(result);
-			RETURN_FALSE;
-		}
+		odbc_bindcols(result);
 	} else {
 		result->values = NULL;
 	}
@@ -3285,10 +3248,7 @@ PHP_FUNCTION(odbc_specialcolumns)
 	SQLNumResultCols(result->stmt, &(result->numcols));
 
 	if (result->numcols > 0) {
-		if (!odbc_bindcols(result)) {
-			efree(result);
-			RETURN_FALSE;
-		}
+		odbc_bindcols(result);
 	} else {
 		result->values = NULL;
 	}
@@ -3354,10 +3314,7 @@ PHP_FUNCTION(odbc_statistics)
 	SQLNumResultCols(result->stmt, &(result->numcols));
 
 	if (result->numcols > 0) {
-		if (!odbc_bindcols(result)) {
-			efree(result);
-			RETURN_FALSE;
-		}
+		odbc_bindcols(result);
 	} else {
 		result->values = NULL;
 	}
@@ -3416,10 +3373,7 @@ PHP_FUNCTION(odbc_tableprivileges)
 	SQLNumResultCols(result->stmt, &(result->numcols));
 
 	if (result->numcols > 0) {
-		if (!odbc_bindcols(result)) {
-			efree(result);
-			RETURN_FALSE;
-		}
+		odbc_bindcols(result);
 	} else {
 		result->values = NULL;
 	}
diff --git a/ext/odbc/php_odbc_includes.h b/ext/odbc/php_odbc_includes.h
index 21c74d317c68c..07ee99e2f0075 100644
--- a/ext/odbc/php_odbc_includes.h
+++ b/ext/odbc/php_odbc_includes.h
@@ -250,7 +250,7 @@ void odbc_del_result(HashTable *list, int count);
 int odbc_add_conn(HashTable *list, HDBC conn);
 odbc_connection *odbc_get_conn(HashTable *list, int count);
 void odbc_del_conn(HashTable *list, int ind);
-int odbc_bindcols(odbc_result *result);
+void odbc_bindcols(odbc_result *result);
 
 #define ODBC_SQL_ERROR_PARAMS odbc_connection *conn_resource, ODBC_SQL_STMT_T stmt, char *func