Fix #33963. mssql_bind fails on input parameters

diff --git a/NEWS b/NEWS
index 01c796782f0e1bc36807f60b3744f140bf5666f8..0f5c81ddb63088614c6d73833591a6109c67181e 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -28,3 +28,4 @@ PHP                                                                        NEWS
 - Fixed bug #34286 (__toString() behavior is inconsistent). (Marcus)
 - Fixed bug #33153 (crash in mssql_next result). (Frank)
 - Fixed bug #32009 (crash when mssql_bind() is called more than once). (Frank)
+- Fixed bug #33963 (mssql_bind() fails on input parameters). (Frank)

diff --git a/ext/mssql/php_mssql.c b/ext/mssql/php_mssql.c
index 25d49582f58fcb1af23e79a3953fd6b7f477ca06..5635e651307b8e5e5f38fbecf226854b4f175392 100644 (file)
--- a/ext/mssql/php_mssql.c
+++ b/ext/mssql/php_mssql.c
@@ -2041,7 +2041,9 @@ PHP_FUNCTION(mssql_bind)
                                type=Z_LVAL_PP(yytype);
                                is_output=Z_LVAL_PP(yyis_output);
                                is_null=Z_LVAL_PP(yyis_null);
-                               maxlen=Z_LVAL_PP(yymaxlen);                             
+                               if (is_output) {
+                                       maxlen=Z_LVAL_PP(yymaxlen);
+                               }
                        }
                        break;