|  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #68786 Missing check
Submitted: 2015-01-10 10:19 UTC Modified: 2021-10-15 17:21 UTC
From: bugreports at internot dot info Assigned:
Status: Open Package: Scripting Engine problem
PHP Version: master-Git-2015-01-10 (Git) OS: Linux Ubuntu 14.04
Private report: No CVE-ID: None
View Add Comment Developer Edit
Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know!
Just going to say 'Me too!'? Don't clutter the database with that please — but make sure to vote on the bug!
Your email address:
Solve the problem:
38 - 23 = ?
Subscribe to this entry?

 [2015-01-10 10:19 UTC] bugreports at internot dot info

In Zend/zend_signal.c:

Zend/zend_signal.c:306:                 sigaction(zend_sigs[x], NULL, &sa);
Zend/zend_signal.c-307-                 if (sa.sa_sigaction != zend_signal_handler_defer) {
Zend/zend_signal.c-308-                         zend_error(E_CORE_WARNING, "zend_signal: handler was replaced for signal (%d) after startup", zend_sigs[x]);
Zend/zend_signal.c-309-                 }

The return value of "sigaction(zend_sigs[x], NULL, &sa)" is not checked, as it is everywhere else it is used.

It should be checked here, too.



Add a Patch

Pull Requests

Add a Pull Request


AllCommentsChangesGit/SVN commitsRelated reports
 [2021-10-15 17:21 UTC]
-Package: *General Issues +Package: Scripting Engine problem
 [2021-10-15 17:21 UTC]
While It might be okay not to check the return value on getting[1]
(it is certainly important on setting), it might still be prudent
to do so.

[1] <>
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Sat Apr 13 09:01:28 2024 UTC