pespin has submitted this change. ( https://gerrit.osmocom.org/c/openbsc/+/31898 ) Change subject: mgcp: Call osmo_fd_unregister() before closing and changing bfd->fd ...................................................................... mgcp: Call osmo_fd_unregister() before closing and changing bfd->fd Change-Id: I2227e97b1cffab5ab0138ce95a2523a824ec35f0 --- M openbsc/src/libmgcp/mgcp_network.c 1 file changed, 11 insertions(+), 2 deletions(-) Approvals: Jenkins Builder: Verified fixeria: Looks good to me, approved diff --git a/openbsc/src/libmgcp/mgcp_network.c b/openbsc/src/libmgcp/mgcp_network.c index e268658..c52bd0b 100644 --- a/openbsc/src/libmgcp/mgcp_network.c +++ b/openbsc/src/libmgcp/mgcp_network.c @@ -1050,15 +1050,15 @@ int mgcp_free_rtp_port(struct mgcp_rtp_end *end) { if (end->rtp.fd != -1) { + osmo_fd_unregister(&end->rtp); close(end->rtp.fd); end->rtp.fd = -1; - osmo_fd_unregister(&end->rtp); } if (end->rtcp.fd != -1) { + osmo_fd_unregister(&end->rtcp); close(end->rtcp.fd); end->rtcp.fd = -1; - osmo_fd_unregister(&end->rtcp); } return 0; -- To view, visit https://gerrit.osmocom.org/c/openbsc/+/31898 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: openbsc Gerrit-Branch: master Gerrit-Change-Id: I2227e97b1cffab5ab0138ce95a2523a824ec35f0 Gerrit-Change-Number: 31898 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> Gerrit-Reviewer: pespin <pespin@sysmocom.de> Gerrit-MessageType: merged