pespin has submitted this change. (
https://gerrit.osmocom.org/c/osmo-mgw/+/31806 )
Change subject: mgcp_network: Unregister osmo_fd before closing fd
......................................................................
mgcp_network: Unregister osmo_fd before closing fd
Change-Id: Ib937e56be62327f52d9cf03a07d7620080356ee8
---
M src/libosmo-mgcp/mgcp_network.c
1 file changed, 11 insertions(+), 2 deletions(-)
Approvals:
Jenkins Builder: Verified
daniel: Looks good to me, but someone else must approve
osmith: Looks good to me, approved
diff --git a/src/libosmo-mgcp/mgcp_network.c b/src/libosmo-mgcp/mgcp_network.c
index d6028a2..af9ae99 100644
--- a/src/libosmo-mgcp/mgcp_network.c
+++ b/src/libosmo-mgcp/mgcp_network.c
@@ -1676,14 +1676,14 @@
void 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);
}
}
--
To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/31806
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: Ib937e56be62327f52d9cf03a07d7620080356ee8
Gerrit-Change-Number: 31806
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <[email protected]>
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged
