Bug #73801 [Com]: switch-Statement mit Class-Constants lead to uninitialized return

From: Date: Mon, 23 Jan 2017 07:57:23 +0000
Subject: Bug #73801 [Com]: switch-Statement mit Class-Constants lead to uninitialized return
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-206872@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=73801&edit=1

 ID:                 73801
 Comment by:         matthias dot kuehne at ellerhold dot de
 Reported by:        matthias dot kuehne at ellerhold dot de
 Summary:            switch-Statement mit Class-Constants lead to
                     uninitialized return
 Status:             Duplicate
 Type:               Bug
 Package:            opcache
 Operating System:   Debian Jessie
 PHP Version:        7.1.0
 Block user comment: N
 Private report:     N

 New Comment:

Ok, I can confirm that the error is fixed in PHP 7.1.1-1+0~20170120094658.14+jessie~1.gbp69d416.

Thanks!


Previous Comments:
------------------------------------------------------------------------
[2016-12-22 14:43:07] matthias dot kuehne at ellerhold dot de

Ondrej Sury had missed the middle commit and added it to 7.1.0-5. Its not ready yet and I'll be
able to test it after the holidays. Thanks for your effort!

------------------------------------------------------------------------
[2016-12-22 12:45:00] nikic@php.net

Just checked, your test case runs fine under current 7.1 branch. Note that there have been three
different fixes regarding handling of return types in opcache, namely:

https://github.com/php/php-src/commit/3b79f8f408ab090825bc15656e517746fdc43db9
https://github.com/php/php-src/commit/60249f439012598717f97ce250f06181116502ea
https://github.com/php/php-src/commit/6bd41a1d47417ad2c7938a57c8abca4e210d5cf6

I did not check it, but it may be that only pulling in the last one does not fix the issue.

------------------------------------------------------------------------
[2016-12-22 12:22:24] matthias dot kuehne at ellerhold dot de

More discussion at https://github.com/oerdnj/deb.sury.org/issues/507

Ondrej Sury built a new PHP-version with the fixes from #73789 and I still got the same error. So
the patch from #73789 might not be enough.

------------------------------------------------------------------------
[2016-12-22 07:12:42] matthias dot kuehne at ellerhold dot de

Yeah the descriptions sound similar, didnt find it in my research prior to opening the bug. :(

Thanks and have a nice christmas and new years eve!

------------------------------------------------------------------------
[2016-12-21 20:35:37] nikic@php.net

This is a duplicate of bug #73789, which will be fixed in PHP 7.1.1 :)

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


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

    https://bugs.php.net/bug.php?id=73801


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


Thread (7 messages)

« previous php.bugs (#206872) next »