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);