ID: 10565
Updated by: cynic
Reported By: [EMAIL PROTECTED]
Old-Status: Open
Status: Analyzed
Bug Type: MySQL related
PHP Version: 4.0.4pl1
Assigned To:
Comments:
I had a conversation with Sinisa, this is the outcome. If it isn't true, please
contact the MySQL team directly. All in all, you said it's a bug in MySQL.
From: Sinisa Milivojevic <[EMAIL PROTECTED]>
To: [EMAIL PROTECTED]
Cc: [EMAIL PROTECTED]
Subject: Re: mysql_real_connect dumps core
Cynic writes:
> no, the patch was (probably) generated with diff -c. read:
>
> - mysql_init(mysql);
> ---
> + mysql = mysql_init(NULL);
>
MYSQL *mysql = (MYSQL *)NULL;
mysql = mysql_init(mysql);
mysql_real_connect(mysql,...
must work on any system with 3.23 client API.
Regards,
Sinisa
____ __ _____ _____ ___ == MySQL AB
/*/\*\/\*\ /*/ \*\ /*/ \*\ |*| Sinisa Milivojevic
/*/ /*/ /*/ \*\_ |*| |*||*| mailto:[EMAIL PROTECTED]
/*/ /*/ /*/\*\/*/ \*\|*| |*||*| Larnaca, Cyprus
/*/ /*/ /*/\*\_/*/ \*\_/*/ |*|____
^^^^^^^^^^^^/*/^^^^^^^^^^^\*\^^^^^^^^^^^
/*/ \*\ Developers Team
Previous Comments:
---------------------------------------------------------------------------
[2001-05-02 06:59:39] [EMAIL PROTECTED]
mailed MySQL
---------------------------------------------------------------------------
[2001-04-30 16:57:34] [EMAIL PROTECTED]
** This is a problem in MySql. This report provides a code
modification to compensate for the MySql problem. **
Under SCO OpenServer 5.0.6, Apache 1.3.19, PHP 4.0.4 PL 1,
and MySql 3.23.36 (precompiled MySQL for OpenServer 5.0.x),
calls to mysql_real_connect will silently dump core if
mysql_init was not allowed to *allocate* the memory for the
MySQL structure.
To function properly, mysql_init must be passed NULL, thus
allowing it to allocate and manage the memory. If you use
a previously malloc()'ed or static structure, MySQL will
dump core on connect.
We find this problem to be present in MySQL, and can
duplicate it using a C code stub. The problem, of course,
also exists in PHP, causing a core dump there as well,
since PHP pre-malloc()'s its own structure.
Here is a DIFF for ext/mysql/php_mysql.c which fixes the
problem for us. It's ugly, and hack-y, but it works. FYI.
198c198
< efree(link);
---
> /* efree(link); */
456c456
< mysql = (MYSQL *) malloc(sizeof(MYSQL));
---
> /* mysql = (MYSQL *) malloc(sizeof(MYSQL)); */
458c458
< mysql_init(mysql);
---
> mysql = mysql_init(NULL);
542c542
< mysql = (MYSQL *) emalloc(sizeof(MYSQL));
---
> /* mysql = (MYSQL *) emalloc(sizeof(MYSQL)); */
544c544
< mysql_init(mysql);
---
> mysql = mysql_init(NULL);
---------------------------------------------------------------------------
ATTENTION! Do NOT reply to this email!
To reply, use the web interface found at http://bugs.php.net/?id=10565&edit=2
--
PHP Development Mailing List <http://www.php.net/>
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
To contact the list administrators, e-mail: [EMAIL PROTECTED]