Bug #76205 [Com]: PHP-FPM sporadic crash when running Infinitewp
| From: | spam2 at rhsoft dot net | Date: | Thu, 03 May 2018 19:08:12 +0000 |
| Subject: | Bug #76205 [Com]: PHP-FPM sporadic crash when running Infinitewp | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-215040@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=76205&edit=1
ID: 76205
Comment by: spam2 at rhsoft dot net
Reported by: post at minhost dot no
Summary: PHP-FPM sporadic crash when running Infinitewp
Status: Closed
Type: Bug
Package: opcache
Operating System: CentOS 7.4
PHP Version: 7.1.16
Assigned To: dmitry
Block user comment: N
Private report: N
New Comment:
the only one making noise here is you because you could have saved at least 4 comments by just deal
with patches like the rest of the world
smart people carry their patches in build scripts / rpm specs over years without touch the tarball
ever - mine for httpd to not send any server header is untouched since httpd 2.4 was released
why?
because they are agnostic to other changes in different areas of the same file and if they needs to
be rebased patch simply fails and you take notice
Previous Comments:
------------------------------------------------------------------------
[2018-05-03 18:40:29] post at minhost dot no
@spam2, you are wrong. _I_ am applying this patch manually. And I have not asked what other people
think about that. So please don't make noise in the comment field.
------------------------------------------------------------------------
[2018-05-03 18:22:56] spam2 at rhsoft dot net
do yourself a favor and look at "man patcz" - nobody is applying patches line by line and
by hand
------------------------------------------------------------------------
[2018-05-03 18:07:09] post at minhost dot no
when I download the current snapshot I see that zend_file_cache.c line number 100 changed from
((char*)(ptr) < (char*)script->size)
to
((char*)(ptr) <= (char*)script->size)
should I add that change to? Or is it safe to use your commit without above change?
------------------------------------------------------------------------
[2018-05-03 17:41:33] post at minhost dot no
Thanks. I just hope I am getting it correct. I am manually removing and copy/paste line by line from
this patch http://git.php.net/?p=php-src.git;a=blobdiff;f=ext/opcache/zend_file_cache.c;h=c66e5aeeca78e2c39cca19d9b4b59cf7e2709978;hp=23f2f9d0cb6d53dfbc12fdedafd33f6bccfee7f4;hb=5b3e1ded356dc877aa246d28e71e69286b8829b5;hpb=7ba70a5fedaba1f6085c8325b1fc4db54e8c8727
into the official PHP 7.1.17 release file: ext/opcache/zend_file_cache.c
------------------------------------------------------------------------
[2018-05-03 10:55:32] dmitry@php.net
yes, it can be merged.
------------------------------------------------------------------------
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=76205
--
Edit this bug report at https://bugs.php.net/bug.php?id=76205&edit=1