neels has uploaded this change for review.

View Change

fix memleak on del_tunnel() failure

Related: CID#307542
Change-Id: Ie29840a7518435447acf48f12e81fc520308a265
---
M tools/gtp-tunnel.c
1 file changed, 2 insertions(+), 0 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/libgtpnl refs/changes/58/31158/1
diff --git a/tools/gtp-tunnel.c b/tools/gtp-tunnel.c
index 1d8486c..1868097 100644
--- a/tools/gtp-tunnel.c
+++ b/tools/gtp-tunnel.c
@@ -130,6 +130,7 @@
gtp_ifidx = if_nametoindex(argv[2]);
if (gtp_ifidx == 0) {
fprintf(stderr, "wrong GTP interface %s\n", argv[2]);
+ gtp_tunnel_free(t);
return EXIT_FAILURE;
}
gtp_tunnel_set_ifidx(t, gtp_ifidx);
@@ -143,6 +144,7 @@
} else {
fprintf(stderr, "wrong GTP version %s, use v0 or v1\n",
argv[3]);
+ gtp_tunnel_free(t);
return EXIT_FAILURE;
}


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

Gerrit-Project: libgtpnl
Gerrit-Branch: master
Gerrit-Change-Id: Ie29840a7518435447acf48f12e81fc520308a265
Gerrit-Change-Number: 31158
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr@sysmocom.de>
Gerrit-MessageType: newchange