laforge submitted this change.

View Change


Approvals: pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified laforge: Looks good to me, approved
tools/gtp-tunnel: pass rc of gtp_del_tunnel

Don't exit with 0 if gtp_del_tunnel fails.

Change-Id: I8046a57a5831e6bdee2ac37415380ce391c79c22
---
M tools/gtp-tunnel.c
1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/tools/gtp-tunnel.c b/tools/gtp-tunnel.c
index e8db171..2a5540a 100644
--- a/tools/gtp-tunnel.c
+++ b/tools/gtp-tunnel.c
@@ -131,6 +131,7 @@
{
struct gtp_tunnel *t;
uint32_t gtp_ifidx;
+ int ret;

if (argc != 6) {
printf("%s del <gtp device> <version> <tid|i_tei> <family>\n",
@@ -171,10 +172,10 @@
return EXIT_FAILURE;
}

- gtp_del_tunnel(genl_id, nl, t);
+ ret = gtp_del_tunnel(genl_id, nl, t);

gtp_tunnel_free(t);
- return 0;
+ return ret;
}

struct gtp_pdp {

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

Gerrit-Project: libgtpnl
Gerrit-Branch: master
Gerrit-Change-Id: I8046a57a5831e6bdee2ac37415380ce391c79c22
Gerrit-Change-Number: 36033
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged