laforge has submitted this change. (
https://gerrit.osmocom.org/c/libgtpnl/+/35971?usp=email )
Change subject: tools/gtp-tunnel: pass rc of gtp_add_tunnel
......................................................................
tools/gtp-tunnel: pass rc of gtp_add_tunnel
Don't exit with 0 if gtp_add_tunnel fails.
Change-Id: If11b1fb4487a96dcb7f24d95794fea4e36202203
---
M tools/gtp-tunnel.c
1 file changed, 14 insertions(+), 2 deletions(-)
Approvals:
laforge: Looks good to me, approved
pespin: Looks good to me, but someone else must approve
Jenkins Builder: Verified
diff --git a/tools/gtp-tunnel.c b/tools/gtp-tunnel.c
index b29e386..1fcdee7 100644
--- a/tools/gtp-tunnel.c
+++ b/tools/gtp-tunnel.c
@@ -77,6 +77,7 @@
uint32_t gtp_ifidx;
uint32_t gtp_version;
int optidx;
+ int ret;
if (argc < 7 || argc > 8) {
add_usage(argv[0]);
@@ -118,10 +119,10 @@
set_addr(argv[optidx++], true, t);
set_addr(argv[optidx++], false, t);
- gtp_add_tunnel(genl_id, nl, t);
+ ret = gtp_add_tunnel(genl_id, nl, t);
gtp_tunnel_free(t);
- return 0;
+ return ret;
}
static int
--
To view, visit
https://gerrit.osmocom.org/c/libgtpnl/+/35971?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libgtpnl
Gerrit-Branch: master
Gerrit-Change-Id: If11b1fb4487a96dcb7f24d95794fea4e36202203
Gerrit-Change-Number: 35971
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged