On Fri, Mar 26, 2010 at 17:58, Andrey Hristov <and...@php.net> wrote:
> andrey                                   Fri, 26 Mar 2010 16:58:25 +0000
>
> Revision: http://svn.php.net/viewvc?view=revision&revision=296885
>
> Log:
> Fix tests that fail when the MySQL's socket file is not /tmp/mysql.sock
> as it is in when compiled from source and the default for mysqlnd.
> SuSE for example uses /var/run/mysql/mysql.sock . Also, sql.safe_mode
> (ext/mysql and ingres) needs the socket.
> Fix possible crashes in mysqlnd. When packets are shorter, functions should
> return error.
>

Modified: php/php-src/trunk/ext/mysqlnd/mysqlnd.c
===================================================================
--- php/php-src/trunk/ext/mysqlnd/mysqlnd.c     2010-03-26 16:55:03 UTC (rev 
296884)
+++ php/php-src/trunk/ext/mysqlnd/mysqlnd.c     2010-03-26 16:58:25 UTC (rev 
296885)
@@ -516,20 +516,24 @@
        if (!host || !host[0]) {
                host = "localhost";
        }
-       if (!user || !user[0]) {
-               user = php_get_current_user();
+       if (!user) {
+               DBG_INF_FMT("no user given, using empty string");
+               user = "";
        }

Isn't this a backwards compatibility break?
That however doesn't seem to be documented, so I guess not many relied
on it.. but it may be worth mentioning?

-Hannes

--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to