daniel submitted this change.

View Change



1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Approvals: pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved Jenkins Builder: Verified
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(-)

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 change 33254. To unsubscribe, or for help writing mail filters, visit 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