daniel has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-netif/+/33254 ) ( 1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: stream: Use cli->state to check if cli is already closed ...................................................................... stream: Use cli->state to check if cli is already closed Change-Id: I92231528da08f8891e20c1226b61989a65e00ccd --- M src/stream.c 1 file changed, 10 insertions(+), 1 deletion(-) Approvals: pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/stream.c b/src/stream.c index 7c8e21e..6fb41f1 100644 --- a/src/stream.c +++ b/src/stream.c @@ -306,7 +306,7 @@ * abstraction and close the socket */ void osmo_stream_cli_close(struct osmo_stream_cli *cli) { - if (cli->ofd.fd == -1) + if (cli->state == STREAM_CLI_STATE_CLOSED) return; osmo_fd_unregister(&cli->ofd); close(cli->ofd.fd); -- To view, visit https://gerrit.osmocom.org/c/libosmo-netif/+/33254 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-netif Gerrit-Branch: master Gerrit-Change-Id: I92231528da08f8891e20c1226b61989a65e00ccd Gerrit-Change-Number: 33254 Gerrit-PatchSet: 3 Gerrit-Owner: daniel <dwillmann@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> Gerrit-Reviewer: laforge <laforge@osmocom.org> Gerrit-Reviewer: pespin <pespin@sysmocom.de> Gerrit-MessageType: merged