Fixed bug #42242 (sybase_connect() crashes)

diff --git a/NEWS b/NEWS
index 482e93f53c15de5aa719c180a3dbbd9a06b80682..059c2d50fa1278da7af717a39ea150ad97ca3570 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -6,6 +6,7 @@ PHP                                                                        NEWS
 - Fixed bug #42247 (ldap_parse_result() not defined under win32). (Jani)
 - Fixed bug #42243 (copy() does not ouput an error when the first arg is a
   dir). (Ilia)
+- Fixed bug #42242 (sybase_connect() crashes). (Ilia)
 - Fixed bug #42237 (stream_copy_to_stream returns invalid values for mmaped 
   streams). (andrew dot minerd at sellingsource dot com, Ilia)
 - Fixed bug #42233 (Problems with æøå in extract()). (Jani)

diff --git a/ext/sybase/php_sybase_db.c b/ext/sybase/php_sybase_db.c
index 1f6bdfd5b4bca4b74b89366366023819297cb4f0..467f256acc21f5345bd722adc1f2726f05397bb6 100644 (file)
--- a/ext/sybase/php_sybase_db.c
+++ b/ext/sybase/php_sybase_db.c
@@ -355,7 +355,7 @@ static void php_sybase_do_connect(INTERNAL_FUNCTION_PARAMETERS,int persistent)
                                host = Z_STRVAL_PP(yyhost);
                                user = Z_STRVAL_PP(yyuser);
                                passwd = Z_STRVAL_PP(yypasswd);
-                               spprintf(&hashed_details, 0, "sybase_%s_%s_%s__", Z_STRVAL_PP(yyhost), Z_STRVAL_PP(yyuser), Z_STRVAL_PP(yypasswd));
+                               hashed_details_length = spprintf(&hashed_details, 0, "sybase_%s_%s_%s__", Z_STRVAL_PP(yyhost), Z_STRVAL_PP(yyuser), Z_STRVAL_PP(yypasswd));
                        }
                        break;
                case 4: {