From: Ilia Alshanetsky <iliaa@php.net>
Date: Wed, 1 Jun 2011 13:23:25 +0000 (+0000)
Subject: Fixed bug #52104 (bindColumn creates Warning regardless of ATTR_ERRMODE settings).
X-Git-Tag: php-5.3.7RC1~56
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=83d01bd81b605bea1d3be624f3422d5352b39f6d;p=php

Fixed bug #52104 (bindColumn creates Warning regardless of ATTR_ERRMODE settings).
---

diff --git a/NEWS b/NEWS
index 29d6bd95bc..34b2c8b9bf 100644
--- a/NEWS
+++ b/NEWS
@@ -115,6 +115,10 @@ PHP                                                                        NEWS
     set mysqlnd.net_cmd_buffer_size to at least 9000. (Andrey)
   . Fixed bug #54674 mysqlnd valid_sjis_(head|tail) is using invalid operator
     and range). (nihen at megabbs dot com, Andrey)
+
+- PDO extension:
+  . Fixed bug #52104 (bindColumn creates Warning regardless of ATTR_ERRMODE 
+    settings). (Ilia)
     
 - PDO DBlib driver:
   . Fixed bug #54167 (PDO_DBLIB returns null on SQLUNIQUE field).
diff --git a/ext/pdo/pdo_stmt.c b/ext/pdo/pdo_stmt.c
index a8435d235a..b8a6f5f0f8 100755
--- a/ext/pdo/pdo_stmt.c
+++ b/ext/pdo/pdo_stmt.c
@@ -349,7 +349,10 @@ static int really_register_bound_param(struct pdo_bound_param_data *param, pdo_s
 		/* if you prepare and then execute passing an array of params keyed by names,
 		 * then this will trigger, and we don't want that */
 		if (param->paramno == -1) {
-			php_error_docref(NULL TSRMLS_CC, E_WARNING, "Did not found column name '%s' in the defined columns; it will not be bound", param->name);
+			char *tmp;
+			spprintf(&tmp, 0, "Did not find column name '%s' in the defined columns; it will not be bound", param->name);
+			pdo_raise_impl_error(stmt->dbh, stmt, "HY000", tmp TSRMLS_CC);
+			efree(tmp);
 		}
 	}