Ruslan Osmanov avatar Ruslan Osmanov committed 0deace7

Fix: return false in bevent methods when the ptr is null

Comments (0)

Files changed (1)

classes/buffer_event.c

     if (!bev->bevent) {                             \
         php_error_docref(NULL TSRMLS_CC, E_WARNING, \
                 "Buffer Event is not initialized"); \
+        RETURN_FALSE; \
     }                                               \
 }
 
 	}
 
 	PHP_EVENT_FETCH_BEVENT(bev, zbevent);
+	_ret_if_invalid_bevent_ptr(bev);
 
 	if (bufferevent_enable(bev->bevent, events)) {
 		RETURN_FALSE;
 	}
 
 	PHP_EVENT_FETCH_BEVENT(bev, zbevent);
+	_ret_if_invalid_bevent_ptr(bev);
 
 	if (bufferevent_disable(bev->bevent, events)) {
 		RETURN_FALSE;
 	}
 
 	PHP_EVENT_FETCH_BEVENT(bev, zbevent);
+	_ret_if_invalid_bevent_ptr(bev);
 
 	PHP_EVENT_INIT_CLASS_OBJECT(return_value, php_event_buffer_ce);
 	PHP_EVENT_FETCH_BUFFER(b, return_value);
Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.