mirror of
https://github.com/samba-team/samba.git
synced 2025-01-11 05:18:09 +03:00
tevent: handle multiplexed fde's in epoll_add_event()
Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-by: Jeremy Allison <jra@samba.org>
This commit is contained in:
parent
e4b496cc09
commit
809593e7f5
@ -365,21 +365,41 @@ static void epoll_add_event(struct epoll_event_context *epoll_ev, struct tevent_
|
||||
{
|
||||
struct epoll_event event;
|
||||
int ret;
|
||||
struct tevent_fd *mpx_fde = NULL;
|
||||
|
||||
fde->additional_flags &= ~EPOLL_ADDITIONAL_FD_FLAG_HAS_EVENT;
|
||||
fde->additional_flags &= ~EPOLL_ADDITIONAL_FD_FLAG_REPORT_ERROR;
|
||||
|
||||
if (fde->additional_flags & EPOLL_ADDITIONAL_FD_FLAG_HAS_MPX) {
|
||||
/*
|
||||
* This is a multiplexed fde, we need to include both
|
||||
* flags in the modified event.
|
||||
*/
|
||||
mpx_fde = talloc_get_type_abort(fde->additional_data,
|
||||
struct tevent_fd);
|
||||
|
||||
mpx_fde->additional_flags &= ~EPOLL_ADDITIONAL_FD_FLAG_HAS_EVENT;
|
||||
mpx_fde->additional_flags &= ~EPOLL_ADDITIONAL_FD_FLAG_REPORT_ERROR;
|
||||
}
|
||||
|
||||
ZERO_STRUCT(event);
|
||||
event.events = epoll_map_flags(fde->flags);
|
||||
if (mpx_fde != NULL) {
|
||||
event.events |= epoll_map_flags(mpx_fde->flags);
|
||||
}
|
||||
event.data.ptr = fde;
|
||||
ret = epoll_ctl(epoll_ev->epoll_fd, EPOLL_CTL_ADD, fde->fd, &event);
|
||||
if (ret != 0 && errno == EBADF) {
|
||||
tevent_debug(epoll_ev->ev, TEVENT_DEBUG_ERROR,
|
||||
"EPOLL_CTL_ADD EBADF for "
|
||||
"fde[%p] fd[%d] - disabling\n",
|
||||
fde, fde->fd);
|
||||
"fde[%p] mpx_fde[%p] fd[%d] - disabling\n",
|
||||
fde, mpx_fde, fde->fd);
|
||||
DLIST_REMOVE(epoll_ev->ev->fd_events, fde);
|
||||
fde->event_ctx = NULL;
|
||||
if (mpx_fde != NULL) {
|
||||
DLIST_REMOVE(epoll_ev->ev->fd_events, mpx_fde);
|
||||
mpx_fde->event_ctx = NULL;
|
||||
}
|
||||
return;
|
||||
} else if (ret != 0) {
|
||||
epoll_panic(epoll_ev, "EPOLL_CTL_ADD failed", false);
|
||||
@ -391,6 +411,16 @@ static void epoll_add_event(struct epoll_event_context *epoll_ev, struct tevent_
|
||||
if (fde->flags & TEVENT_FD_READ) {
|
||||
fde->additional_flags |= EPOLL_ADDITIONAL_FD_FLAG_REPORT_ERROR;
|
||||
}
|
||||
|
||||
if (mpx_fde == NULL) {
|
||||
return;
|
||||
}
|
||||
|
||||
mpx_fde->additional_flags |= EPOLL_ADDITIONAL_FD_FLAG_HAS_EVENT;
|
||||
/* only if we want to read we want to tell the event handler about errors */
|
||||
if (mpx_fde->flags & TEVENT_FD_READ) {
|
||||
mpx_fde->additional_flags |= EPOLL_ADDITIONAL_FD_FLAG_REPORT_ERROR;
|
||||
}
|
||||
}
|
||||
|
||||
/*
|
||||
|
Loading…
Reference in New Issue
Block a user