php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #78656 Parse errors classified as highest log-level
Submitted: 2019-10-09 11:53 UTC Modified: 2019-10-09 15:26 UTC
From: erik at coretech dot se Assigned: cmb (profile)
Status: Closed Package: Scripting Engine problem
PHP Version: 7.3.10 OS: CentOS 7
Private report: No CVE-ID: None
 [2019-10-09 11:53 UTC] erik at coretech dot se
Description:
------------
Parse errors are classified as the highest log level (LOG_EMERG). This is problem since Centos/Fedora writes LOG_EMERG messages to all users:

https://src.fedoraproject.org/rpms/rsyslog/blob/master/f/rsyslog.conf#_59

LOG_EMERG is described as: "system is unusable". A php-script with a parse error would not in most cases render the system unstable. At most LOG_ERR or LOG_CRIT would be more suitable. Fatal errors are classified as LOG_ERR. Why not make parse errors the same level?

The source:
https://github.com/php/php-src/blob/PHP-7.3/main/main.c#L1300

The commit that changed this:

https://github.com/php/php-src/commit/3edf7d960cb14f2ee45944e7c567af92af25bda1


Patches

Add a Patch

Pull Requests

Pull requests:

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2019-10-09 12:52 UTC] sjon@php.net
I agree - this setup (by default) will cause any parse-error to be broadcasted (using wall) to all users on a shared development machine
 [2019-10-09 14:30 UTC] erik at coretech dot se
The following pull request has been associated:

Patch Name: Request #78656	Parse errors classified as highest log-level
On GitHub:  https://github.com/php/php-src/pull/4805
Patch:      https://github.com/php/php-src/pull/4805.patch
 [2019-10-09 15:23 UTC] cmb@php.net
-Assigned To: +Assigned To: cmb
 [2019-10-09 15:23 UTC] cmb@php.net
Reclassifying as bug, see <https://github.com/php/php-src/pull/4805#issuecomment-540004297>ff.
 [2019-10-09 15:26 UTC] cmb@php.net
-Type: Feature/Change Request +Type: Bug
 [2019-10-09 15:26 UTC] cmb@php.net
Grr, hit that bug again.
 [2019-10-09 15:31 UTC] cmb@php.net
Automatic comment on behalf of erik@coretech.se
Revision: http://git.php.net/?p=php-src.git;a=commit;h=3164186d53533ac5239790e692b33dd6e56c18df
Log: Fix #78656: Parse errors classified as highest log-level
 [2019-10-09 15:31 UTC] cmb@php.net
-Status: Assigned +Status: Closed
 
PHP Copyright © 2001-2019 The PHP Group
All rights reserved.
Last updated: Mon Dec 09 04:01:25 2019 UTC