neels has submitted this change. (
https://gerrit.osmocom.org/c/libgtpnl/+/31158 )
Change subject: fix memleak on del_tunnel() failure
......................................................................
fix memleak on del_tunnel() failure
Related: CID#307542
Change-Id: Ie29840a7518435447acf48f12e81fc520308a265
---
M tools/gtp-tunnel.c
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Jenkins Builder: Verified
pespin: Looks good to me, approved
laforge: Looks good to me, approved
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
https://gerrit.osmocom.org/c/libgtpnl/+/31158
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libgtpnl
Gerrit-Branch: master
Gerrit-Change-Id: Ie29840a7518435447acf48f12e81fc520308a265
Gerrit-Change-Number: 31158
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged