[Asterisk-code-review] core: Use eventfd for alert pipes on Linux when possible (asterisk[14])
Sean Bright
asteriskteam at digium.com
Wed Apr 19 09:20:00 CDT 2017
Sean Bright has posted comments on this change. ( https://gerrit.asterisk.org/5498 )
Change subject: core: Use eventfd for alert pipes on Linux when possible
......................................................................
Patch Set 2:
(1 comment)
https://gerrit.asterisk.org/#/c/5498/2/main/alertpipe.c
File main/alertpipe.c:
PS2, Line 76: if (alert_pipe[0] > -1) {
: close(alert_pipe[0]);
: alert_pipe[0] = -1;
: }
: if (alert_pipe[1] > -1) {
: close(alert_pipe[1]);
: alert_pipe[1] = -1;
: }
> This would close the same eventfd twice. Is that acceptable?
> This would close the same eventfd twice. Is that acceptable?
The second close() would just fail with EBADF. I'll fix.
--
To view, visit https://gerrit.asterisk.org/5498
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iff0fb1573e7f7a187d5211ddc60aa8f3da3edb1d
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: 14
Gerrit-Owner: Sean Bright <sean.bright at gmail.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Sean Bright <sean.bright at gmail.com>
Gerrit-HasComments: Yes
More information about the asterisk-code-review
mailing list