#49098 [Opn->Csd]: Using custom session handler causes segfault in session_save_state

From: Date: Thu, 12 Nov 2009 01:10:33 +0000
Subject: #49098 [Opn->Csd]: Using custom session handler causes segfault in session_save_state
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-144506@lists.php.net to get a copy of this message 
 ID:               49098
 Updated by:       rasmus@php.net
 Reported By:      bugs at timj dot co dot uk
-Status:           Open
+Status:           Closed
 Bug Type:         Session related
 Operating System: Linux
 PHP Version:      5.2.10
 New Comment:

Should be fixed now in svn.  Please verify.


Previous Comments:
------------------------------------------------------------------------

[2009-11-12 01:09:16] svn@php.net

Automatic comment from SVN on behalf of rasmus
Revision: http://svn.php.net/viewvc/?view=revision&revision=290570
Log: Fix bug #49098

------------------------------------------------------------------------

[2009-11-11 23:59:51] timj@php.net

I'm not sure if this is useful/correct, but at first pass this stops
the crash:

--- ext/mysqli/mysqli_nonapi.c.orig	2009-10-15 23:34:41.000000000
+0100
+++ ext/mysqli/mysqli_nonapi.c	2009-11-11 23:56:40.271496635 +0000
@@ -46,7 +46,11 @@
 	if (MyG(error_msg)) {
 		efree(MyG(error_msg));
 	}
-	MyG(error_msg) = estrdup(mysql_err);
+	if(mysql_errno!=0) { 
+		MyG(error_msg) = estrdup(mysql_err);
+	} else {
+		MyG(error_msg) = NULL;
+	}
 }
 /* }}} */


------------------------------------------------------------------------

[2009-11-11 23:55:09] timj@php.net

Stepping through the code though, that patch wouldn't have made any
difference. On the final incantation of php_mysqli_set_error before
crash, estrdup() still gets called:

Breakpoint 1, php_mysqli_set_error (mysql_errno=0, mysql_err=0x10325a7
"") at /path/to/php5.3-200911111930/ext/mysqli/mysqli_nonapi.c:44
44	{
(gdb) step
45		MyG(error_no) = mysql_errno;
(gdb) step
44	{
(gdb) step
45		MyG(error_no) = mysql_errno;
(gdb) step
46		if (MyG(error_msg)) {
(gdb) step
47			efree(MyG(error_msg));
(gdb) next
49		if(mysql_err) { 
(gdb) step
50			MyG(error_msg) = estrdup(mysql_err);


------------------------------------------------------------------------

[2009-11-11 23:48:50] timj@php.net

Nope, still the same result. (back on the 5.3 snapshot now)

------------------------------------------------------------------------

[2009-11-11 23:38:53] rasmus@php.net

Could you try this patch and see if it changes anything?

Index: mysqli_nonapi.c
===================================================================
--- mysqli_nonapi.c	(revision 290565)
+++ mysqli_nonapi.c	(working copy)
@@ -46,7 +46,11 @@
 	if (MyG(error_msg)) {
 		efree(MyG(error_msg));
 	}
-	MyG(error_msg) = estrdup(mysql_err);
+	if(mysql_err) { 
+		MyG(error_msg) = estrdup(mysql_err);
+	} else {
+		MyG(error_msg) = NULL;
+	}
 }
 /* }}} */

------------------------------------------------------------------------

The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
    http://bugs.php.net/49098

-- 
Edit this bug report at http://bugs.php.net/?id=49098&edit=1

Thread (38 messages)

« previous php.bugs (#144506) next »