pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/31887 ) Change subject: osmo_bts_sock.c: Call osmo_fd_unregister() before closing and changing bfd->fd ...................................................................... osmo_bts_sock.c: Call osmo_fd_unregister() before closing and changing bfd->fd Change-Id: Iec0faba781bcb2bc6de8710392655785790b9af4 --- M src/osmobts_sock.c 1 file changed, 10 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, but someone else must approve fixeria: Looks good to me, approved diff --git a/src/osmobts_sock.c b/src/osmobts_sock.c index 282e33f..6d46a84 100644 --- a/src/osmobts_sock.c +++ b/src/osmobts_sock.c @@ -96,9 +96,9 @@ LOGP(DL1IF, LOGL_NOTICE, "PCU socket has %s connection\n", (lost) ? "LOST" : "closed"); + osmo_fd_unregister(bfd); close(bfd->fd); bfd->fd = -1; - osmo_fd_unregister(bfd); /* flush the queue */ while (!llist_empty(&pcu_sock_state.upqueue)) { -- To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/31887 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-Change-Id: Iec0faba781bcb2bc6de8710392655785790b9af4 Gerrit-Change-Number: 31887 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> Gerrit-Reviewer: laforge <laforge@osmocom.org> Gerrit-Reviewer: osmith <osmith@sysmocom.de> Gerrit-Reviewer: pespin <pespin@sysmocom.de> Gerrit-MessageType: merged