daniel submitted this change.

View Change

Approvals: Jenkins Builder: Verified osmith: Looks good to me, but someone else must approve laforge: Looks good to me, but someone else must approve fixeria: Looks good to me, approved
ipa: Guard against calling ipa_client_conn_open* multiple times

Change-Id: I574af543429d01c9522e267265e97e4efe55b60c
---
M src/input/ipa.c
1 file changed, 3 insertions(+), 0 deletions(-)

diff --git a/src/input/ipa.c b/src/input/ipa.c
index 62b21e5..94e7e5a 100644
--- a/src/input/ipa.c
+++ b/src/input/ipa.c
@@ -249,6 +249,9 @@
{
int ret;

+ if (link->ofd->fd != -1)
+ return -EINVAL;
+
link->state = IPA_CLIENT_LINK_STATE_CONNECTING;
ret = osmo_sock_init2(AF_INET, SOCK_STREAM, IPPROTO_TCP,
link->local_addr, link->local_port,

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

Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: I574af543429d01c9522e267265e97e4efe55b60c
Gerrit-Change-Number: 30618
Gerrit-PatchSet: 2
Gerrit-Owner: daniel <dwillmann@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-MessageType: merged