php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #38898 #38431 once again (xmlrpc segfault)
Submitted: 2006-09-20 13:12 UTC Modified: 2006-09-25 11:20 UTC
From: grzegorz dot nosek at netart dot pl Assigned:
Status: Closed Package: XMLRPC-EPI related
PHP Version: 5.1.6 OS: Linux
Private report: No CVE-ID: None
View Add Comment Developer Edit
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If this is not your bug, you can add a comment by following this link.
If this is your bug, but you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: grzegorz dot nosek at netart dot pl
New email:
PHP Version: OS:

 

 [2006-09-20 13:12 UTC] grzegorz dot nosek at netart dot pl 
Description:
------------
Guys, you broke it again in 5.1.5 (5.1.6 is still broken, btw.)

This patch seems to fix it. Why was it reverted?

http://cvs.php.net/viewvc.cgi/php-src/ext/xmlrpc/xmlrpc-epi-php.c?r1=1.39.2.5&r2=1.39.2.6

Is there any reason for the commit linked below? 

http://cvs.php.net/viewvc.cgi/php-src/ext/xmlrpc/xmlrpc-epi-php.c?view=log#rev1.39.2.7

Test case below, taken from 5.1.6 distribution (ext/xmlrpc/tests/bug38431.phpt), fails with a segfault.

Reproduce code:
---------------
--TEST--
Bug #38431 (xmlrpc_get_type() crashes PHP on objects)
--SKIPIF--
<?php if (!extension_loaded("xmlrpc")) print "skip"; ?>
--FILE--
<?php

var_dump(xmlrpc_get_type(new stdclass));
var_dump(xmlrpc_get_type(array()));
$var = array(1,2,3);
var_dump(xmlrpc_get_type($var));
$var = array("test"=>1,2,3);
var_dump(xmlrpc_get_type($var));
$var = array("test"=>1,"test2"=>2);
var_dump(xmlrpc_get_type($var));

echo "Done\n";
?>
--EXPECTF--
string(5) "array"
string(5) "array"
string(5) "array"
string(5) "mixed"
string(6) "struct"
Done

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2006-09-23 11:37 UTC] tony2001@php.net 
Please try using this CVS snapshot:

  http://snaps.php.net/php5.2-latest.tar.gz
 
For Windows:
 
  http://snaps.php.net/win32/php5.2-win32-latest.zip
 [2006-09-25 11:14 UTC] grzegorz dot nosek at netart dot pl 
I see that 5.2-cvs has the fix applied so I assume it works (if it doesn't, it's a different issue). Unfortunately, I am not in position to test it. However, I think that it is 5.1.7 material and should be incorporated in the next 5.1 release.
 [2006-09-25 11:20 UTC] tony2001@php.net 
5.2 is the next PHP5 release, there are no 5.1.x releases planned.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Thu Apr 25 00:01:41 2024 UTC