laforge has submitted this change. (
https://gerrit.osmocom.org/c/erlang/osmo_ss7/+/36104?usp=email )
Change subject: ipa_proto: start_listen: use correct ok/error pattern
......................................................................
ipa_proto: start_listen: use correct ok/error pattern
The ok path only returned the port without an "ok".
Also path back the ListenSock.
Change-Id: I9b317c13089fae2406f401a1f420623de9a2dffa
---
M src/ipa_proto.erl
1 file changed, 13 insertions(+), 1 deletion(-)
Approvals:
laforge: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/src/ipa_proto.erl b/src/ipa_proto.erl
index fcd594b..c96d6bb 100644
--- a/src/ipa_proto.erl
+++ b/src/ipa_proto.erl
@@ -367,7 +367,7 @@
{ok, ListenSock} ->
start_servers(NumServers, ListenSock, self()),
{ok, Port} = inet:port(ListenSock),
- Port;
+ {ok, ListenSock, Port};
{error, Reason} ->
{error, Reason}
end.
--
To view, visit
https://gerrit.osmocom.org/c/erlang/osmo_ss7/+/36104?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: erlang/osmo_ss7
Gerrit-Branch: master
Gerrit-Change-Id: I9b317c13089fae2406f401a1f420623de9a2dffa
Gerrit-Change-Number: 36104
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-MessageType: merged