Change in openbsc[master]: bsc-nat: forward_sccp_to_msc: Fix memleak on receive from non authent...

[Pau Espin Pedrol]

Pau Espin Pedrol has uploaded this change for review. ( https://gerrit.osmocom.org/9509


Change subject: bsc-nat: forward_sccp_to_msc: Fix memleak on receive from non authenticated bsc
......................................................................

bsc-nat: forward_sccp_to_msc: Fix memleak on receive from non authenticated bsc

variable "parsed" was not being freed in this case. By calling exit2 we
make sure it is freed.

Change-Id: Ifd0c145ff733fdfb2f6fcb32065de99ee951d106
---
M openbsc/src/osmo-bsc_nat/bsc_nat.c
1 file changed, 1 insertion(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/openbsc refs/changes/09/9509/1

diff --git a/openbsc/src/osmo-bsc_nat/bsc_nat.c b/openbsc/src/osmo-bsc_nat/bsc_nat.c
index 928dd4c..373ba97 100644
--- a/openbsc/src/osmo-bsc_nat/bsc_nat.c
+++ b/openbsc/src/osmo-bsc_nat/bsc_nat.c
@@ -1124,8 +1124,7 @@
 	 */
 	if (!bsc->authenticated) {
 		LOGP(DNAT, LOGL_ERROR, "BSC is not authenticated.\n");
-		msgb_free(msg);
-		return -1;
+		goto exit2;
 	}
 
 

-- 
To view, visit https://gerrit.osmocom.org/9509
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: openbsc
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifd0c145ff733fdfb2f6fcb32065de99ee951d106
Gerrit-Change-Number: 9509
Gerrit-PatchSet: 1
Gerrit-Owner: Pau Espin Pedrol <pespin at sysmocom.de>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20180608/2b907904/attachment.htm>