pespin submitted this change.

View Change


Approvals: Jenkins Builder: Verified fixeria: Looks good to me, approved
mncc_sock: Call osmo_fd_unregister() before closing and changing bfd->fd

Change-Id: I90f840da4cc26c1d970c3f5d18d3185678a1dba9
---
M src/libmsc/mncc_sock.c
1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/libmsc/mncc_sock.c b/src/libmsc/mncc_sock.c
index 7c65546..9c75d2c 100644
--- a/src/libmsc/mncc_sock.c
+++ b/src/libmsc/mncc_sock.c
@@ -71,9 +71,9 @@

LOGP(DMNCC, LOGL_NOTICE, "MNCC Socket has LOST connection\n");

+ osmo_fd_unregister(bfd);
close(bfd->fd);
bfd->fd = -1;
- osmo_fd_unregister(bfd);

/* re-enable the generation of ACCEPT for new connections */
osmo_fd_read_enable(&state->listen_bfd);

To view, visit change 31896. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: I90f840da4cc26c1d970c3f5d18d3185678a1dba9
Gerrit-Change-Number: 31896
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