Hi,
At first time, it was a bug in the useful xdebug, and I follow it to
the PHP code http://bugs.xdebug.org/bug_view_page.php?bug_id=0000340.
When exception::$message, exception::$line are not of the initial
type, it segfaults (for strings) or display non-predictible data. The
next snipset for test :
<?php
class my_file
{
public function __toString()
{
return "somebuildfilename" ;
}
}
class my_exception extends exception
{
public function __construct()
{
$this->message = new stdclass ;
$this->file = new my_file ;
$this->line = "12" ;
}
}
throw new my_exception ;
?>
I attach a patch that correct the segfault. Diff was produced with
this line : diff php5.3-200808141030{,-lupus}/Zend/zend_exceptions.c
Regards, Mickaël.
Mickaël Wolff aka Lupus Michaelis
http://lupusmic.org
Hello Lupus,
Thursday, August 14, 2008, 8:42:23 PM, you wrote:
Hi,
At first time, it was a bug in the useful xdebug, and I follow it to
the PHP code http://bugs.xdebug.org/bug_view_page.php?bug_id=0000340.
When exception::$message, exception::$line are not of the initial
type, it segfaults (for strings) or display non-predictible data. The
next snipset for test :
<?php
class my_file
{
public function __toString()
{
return "somebuildfilename" ;
}
}
class my_exception extends exception
{
public function __construct()
{
$this->message = new stdclass ;
$this->file = new my_file ;
$this->line = "12" ;
}
}
throw new my_exception ;
?>>
I attach a patch that correct the segfault. Diff was produced with
this line : diff php5.3-200808141030{,-lupus}/Zend/zend_exceptions.c
Thanks for letting us know. Can you provide the diff using 'cvs di -uwp'.
Best regards,
Marcus
Hello Marcus,
Thursday, August 14, 2008, 9:17:34 PM, you wrote:
Hello Lupus,
Thursday, August 14, 2008, 8:42:23 PM, you wrote:
Hi,
At first time, it was a bug in the useful xdebug, and I follow it to
the PHP code http://bugs.xdebug.org/bug_view_page.php?bug_id=0000340.
When exception::$message, exception::$line are not of the initial
type, it segfaults (for strings) or display non-predictible data. The
next snipset for test :
<?php
class my_file
{
public function __toString()
{
return "somebuildfilename" ;
}
}
class my_exception extends exception
{
public function __construct()
{
$this->message = new stdclass ;
$this->file = new my_file ;
$this->line = "12" ;
}
}
throw new my_exception ;
?>>>
I attach a patch that correct the segfault. Diff was produced with
this line : diff php5.3-200808141030{,-lupus}/Zend/zend_exceptions.c
Thanks for letting us know. Can you provide the diff using 'cvs di -uwp'.
Well, Felipe was much faster than I. He applied your patch already. Thanks
again a lot.
Best regards,
Marcus
Em Qui, 2008-08-14 às 21:18 +0200, Marcus Boerger escreveu:
Hello Marcus,
Thursday, August 14, 2008, 9:17:34 PM, you wrote:
Hello Lupus,
Thursday, August 14, 2008, 8:42:23 PM, you wrote:
Hi,
At first time, it was a bug in the useful xdebug, and I follow it to
the PHP code http://bugs.xdebug.org/bug_view_page.php?bug_id=0000340.When exception::$message, exception::$line are not of the initial
type, it segfaults (for strings) or display non-predictible data. The
next snipset for test :<?php
class my_file
{
public function __toString()
{
return "somebuildfilename" ;
}
}class my_exception extends exception
{
public function __construct()
{
$this->message = new stdclass ;
$this->file = new my_file ;
$this->line = "12" ;
}
}throw new my_exception ;
?>>>
I attach a patch that correct the segfault. Diff was produced with
this line : diff php5.3-200808141030{,-lupus}/Zend/zend_exceptions.cThanks for letting us know. Can you provide the diff using 'cvs di -uwp'.
Well, Felipe was much faster than I. He applied your patch already. Thanks
again a lot.
Hello,
I don't applied your patch, because I can reproduce it only in HEAD,
then I added the convert_* that are missing in HEAD in same place where
they are found in 5_3.
--
Regards,
Felipe Pena.
Marcus Boerger a écrit :
Thanks for letting us know. Can you provide the diff using 'cvs di -uwp'.
I'll do it in the future. Sorry for the inconvenient, I didn't found
what you're waiting for. So, I did a simple diff :)
Thanks to you and Felipe.
--
Mickaël Wolff aka Lupus Michaelis
http://lupusmic.org