[PATCH] openbsc[master]: bsc_nat: forward_to_bsc: Fix memleak on send failure

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Pau Espin Pedrol gerrit-no-reply at lists.osmocom.org
Wed Apr 11 15:13:29 UTC 2018


Review at  https://gerrit.osmocom.org/7759

bsc_nat: forward_to_bsc: Fix memleak on send failure

Change-Id: If0dfae40f03db297eeb4e296daf5fe78ba53a11b
---
M openbsc/src/osmo-bsc_nat/bsc_nat_ctrl.c
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.osmocom.org:29418/openbsc refs/changes/59/7759/1

diff --git a/openbsc/src/osmo-bsc_nat/bsc_nat_ctrl.c b/openbsc/src/osmo-bsc_nat/bsc_nat_ctrl.c
index 797ddf8..d6bf1e5 100644
--- a/openbsc/src/osmo-bsc_nat/bsc_nat_ctrl.c
+++ b/openbsc/src/osmo-bsc_nat/bsc_nat_ctrl.c
@@ -219,7 +219,7 @@
 	int ret = CTRL_CMD_HANDLED;
 	struct ctrl_cmd *bsc_cmd = NULL;
 	struct bsc_connection *bsc;
-	struct bsc_cmd_list *pending;
+	struct bsc_cmd_list *pending = NULL;
 	unsigned int nr;
 	char *bsc_variable;
 
@@ -292,6 +292,7 @@
 	cmd->reply = "no BSC with this nr";
 err:
 	ret = CTRL_CMD_ERROR;
+	talloc_free(pending);
 done:
 	talloc_free(bsc_cmd);
 	return ret;

-- 
To view, visit https://gerrit.osmocom.org/7759
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If0dfae40f03db297eeb4e296daf5fe78ba53a11b
Gerrit-PatchSet: 1
Gerrit-Project: openbsc
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol <pespin at sysmocom.de>



More information about the gerrit-log mailing list