PHP 4.0 Bug #8951: php_mysql_do_connect -> parameter host modified when using port or socket
| From: | jiri dot kaderavek at webstep dot net | Date: | Sat, 27 Jan 2001 17:46:46 +0000 |
| Subject: | PHP 4.0 Bug #8951: php_mysql_do_connect -> parameter host modified when using port or socket | ||
| Groups: | php.dev | ||
| Request: | Send a blank email to php-dev+get-45138@lists.php.net to get a copy of this message | ||
From: jiri.kaderavek@webstep.net
Operating system: Linux
PHP version: 4.0.4pl1
PHP Bug Type: MySQL related
Bug description: php_mysql_do_connect -> parameter host modified when using port or socket
Parameter $host is modified in function php_mysql_do_connect(). Example:
$host = "localhost:/tmp/mysql.sock";
$conn = mysql_connect($host .....
echo $host;
Output is: localhost/tmp/mysql.sock
because of replacing of ":" charcter by null char in
this part of php_mysql_do_connect
===================
static void php_mysql_do_connect(INTERNAL_FUNCTION_PARAMETERS,int persistent)
{
char *user,*passwd,*host,*socket=NULL,*tmp;
char *hashed_details;
int hashed_details_length,port = MYSQL_PORT;
....
/* We cannot use mysql_port anymore in windows, need to use
* mysql_real_connect() to set the port.
*/
if (host && (tmp=strchr(host,':'))) {
*tmp=0;
tmp++;
if (tmp[0] != '/') {
port = atoi(tmp);
} else {
socket = tmp;
}
.....
}
===================
solution:
copying of *host content through *host2 pointer
===================
static void php_mysql_do_connect(INTERNAL_FUNCTION_PARAMETERS,int persistent)
{
char *user,*passwd,*host,*socket=NULL,*tmp,*host2;
char *hashed_details;
int hashed_details_length,port = MYSQL_PORT;
....
/* We cannot use mysql_port anymore in windows, need to use
* mysql_real_connect() to set the port.
*/
host2 = (char *) emalloc(strlen(SAFE_STRING(host) + 1));
strcpy(host2, host);
host = host2;
if (host && (tmp=strchr(host,':'))) {
*tmp=0;
tmp++;
if (tmp[0] != '/') {
port = atoi(tmp);
} else {
socket = tmp;
}
...
}
--
Edit Bug report at: http://bugs.php.net/?id=8951&edit=1