]> granicus.if.org Git - php/commit
projects / php / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 3744533) | patch
Fix SOAP bailout handling
authorNikita Popov <nikita.ppv@gmail.com>
Fri, 12 Apr 2019 10:47:39 +0000 (12:47 +0200)
committerNikita Popov <nikita.ppv@gmail.com>
Fri, 12 Apr 2019 10:51:17 +0000 (12:51 +0200)
commit4f28bbda51e1a74a69da61ac875e17656f710f4f
tree5c5f0692915b8a2757f118e2bbd4a213181c34d6tree | snapshot
parent3744533468767441f13ebbdda52f354ada0764fccommit | diff
Fix SOAP bailout handling

This code was reusing the _bailout variable from
SOAP_CLIENT_BEGIN/END_CODE(). As _bailout is not volatile, modifying
it after the setjmp call and then reading it back on return is
illegal. Use a separate local bailout variable instead.

This fixes the miscompile introduced by marking zend_bailout() as
noreturn.
ext/soap/soap.c diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.