commit 72ec18ce5a7f4e873b6137460706f5ecc598e4dd
author Vadim Yanitskiy <axilirator@gmail.com> Sun Apr 05 03:19:43 2020 +0700
committer Vadim Yanitskiy <axilirator@gmail.com> Mon Apr 20 11:25:05 2020 +0700
tree 6229dfd39aaa3dc4e70313fe75975cbd52deb440
parent a4e8c10a388763715849eb4bdf234395bd81d79f

sba: fix possible memleak in SBAController::alloc()

Change-Id: I417eda155cd5b1e46dd0b05db3f507abd79121d1

src/sba.cpp

diff --git a/src/sba.cpp b/src/sba.cpp
index 567babe..32735e8 100644
--- a/src/sba.cpp
+++ b/src/sba.cpp
@@ -55,13 +55,13 @@
 	int8_t trx, ts;
 	uint32_t fn;
 
+	if (!gsm48_ta_is_valid(ta))
+		return -EINVAL;
+
 	sba = talloc_zero(tall_pcu_ctx, struct gprs_rlcmac_sba);
 	if (!sba)
 		return -ENOMEM;
 
-	if (!gsm48_ta_is_valid(ta))
-		return -EINVAL;
-
 	for (trx = 0; trx < 8; trx++) {
 		for (ts = 7; ts >= 0; ts--) {
 			pdch = &m_bts.bts_data()->trx[trx].pdch[ts];