php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #59360 detecting of callback is mistake in _php_bufferevent_errorcb
Submitted: 2010-08-10 22:28 UTC Modified: 2015-06-25 17:19 UTC
From: xiezhenye at gmail dot com Assigned: cmb (profile)
Status: Closed Package: libevent (PECL)
PHP Version: 5.2.8 OS: centos 5.2
Private report: No CVE-ID: None
View Add Comment Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
You can add a comment by following this link or if you reported this bug, you can edit this bug over here.
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: xiezhenye at gmail dot com
New email:
PHP Version: OS:

 

 [2010-08-10 22:28 UTC] xiezhenye at gmail dot com
Description:
------------
detecting of callback is mistake in _php_bufferevent_errorcb.
see patch below.

Reproduce code:
---------------
*** libevent.c.old      2010-08-11 10:16:32.000000000 +0800
--- libevent.c  2010-08-11 10:17:04.000000000 +0800
*************** static void _php_bufferevent_errorcb(str
*** 297,303 ****
        php_bufferevent_t *bevent = (php_bufferevent_t *)arg;
        TSRMLS_FETCH_FROM_CTX(bevent ? bevent->thread_ctx : NULL);

!       if (!bevent || !bevent->base || !bevent->writecb) {
                return;
        }

--- 297,303 ----
        php_bufferevent_t *bevent = (php_bufferevent_t *)arg;
        TSRMLS_FETCH_FROM_CTX(bevent ? bevent->thread_ctx : NULL);

!       if (!bevent || !bevent->base || !bevent->errorcb) {
                return;
        }




Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2015-06-25 17:19 UTC] cmb@php.net
-Status: Open +Status: Closed -Assigned To: +Assigned To: cmb
 [2015-06-25 17:19 UTC] cmb@php.net
This bug has been fixed as of libevent 0.0.5.

Thanks for reporting, and sorry for the late reply.
 
PHP Copyright © 2001-2017 The PHP Group
All rights reserved.
Last updated: Sun Nov 19 01:31:42 2017 UTC