php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #66401 yield in SoapClients __doRequest cause Segmentation fault
Submitted: 2014-01-03 13:47 UTC Modified: 2014-01-09 07:55 UTC
From: erik at datahack dot se Assigned: krakjoe (profile)
Status: Closed Package: Reproducible crash
PHP Version: 5.5.7 OS:
Private report: No CVE-ID: None
View Add Comment Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
You can add a comment by following this link or if you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: erik at datahack dot se
New email:
PHP Version: OS:

 

 [2014-01-03 13:47 UTC] erik at datahack dot se
Description:
------------
Using yield in SoapClients __doRequest cause Segmentation fault.

Test script:
---------------
class XSoapClient extends SoapClient {
	function __doRequest($request, $location, $action, $version) {
		yield 123;
	}
}

$client = new XSoapClient(null, array('uri' => '', 'location' => ''));

$x = $client->foo();
foreach($x as $r)
	var_dump($r);

Expected result:
----------------
I wasn't really sure what result to expect, however not a Segmentation fault.

Actual result:
--------------
Segmentation fault

Patches

soapclient-bug-5.5.patch (last revision 2014-01-04 08:55 UTC by krakjoe@php.net)

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2014-01-04 08:55 UTC] krakjoe@php.net
The following patch has been added/updated:

Patch Name: soapclient-bug-5.5.patch
Revision:   1388825719
URL:        https://bugs.php.net/patch-display.php?bug=66401&patch=soapclient-bug-5.5.patch&revision=1388825719
 [2014-01-04 08:56 UTC] krakjoe@php.net
-Status: Open +Status: Verified
 [2014-01-04 08:59 UTC] krakjoe@php.net
-Assigned To: +Assigned To: dmitry
 [2014-01-04 08:59 UTC] krakjoe@php.net
5.5 is missing the fix for this bug, patch attached but no idea of the workflow to apply to specific 5.5 branches so will let someone else do it.

Dmitry is the only person left in the header that is still around, so assigned. Should be a quick fix for you :)
 [2014-01-09 07:46 UTC] dmitry@php.net
-Status: Verified +Status: Feedback -Assigned To: dmitry +Assigned To: krakjoe
 [2014-01-09 07:46 UTC] dmitry@php.net
This patch was applied to PHP-5.5 on Dec 10.

See `git show a65166531`.

I'm not sure if it was include into 5.5.7 release, but it's definitely in PHP-5.5 branch. So, the bug must be already fixed.
 [2014-01-09 07:55 UTC] krakjoe@php.net
-Status: Feedback +Status: Closed
 [2014-01-09 07:55 UTC] krakjoe@php.net
The fix for this bug has been committed.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.

 For Windows:

http://windows.php.net/snapshots/
 
Thank you for the report, and for helping us make PHP better.


 [2014-01-09 07:55 UTC] krakjoe@php.net
Thanks Dmitry, I missed that ...
 
PHP Copyright © 2001-2019 The PHP Group
All rights reserved.
Last updated: Wed Dec 11 08:01:24 2019 UTC