ms: Hold a reference during ms_alloc
Make the caller hold a reference to the MS object just allocated, so
that it hs to explicitly unref it and, in turn, if no new references
were added during its use, trigger release of the MS object.
This is useful to avoid leaking MS object if it was allocated and then
no TBF is attached to it because allocation of TBF failed.
Related: OS#6002
Change-Id: I2088a7ddd76fe9157b6626ef96ae4315e88779ea
diff --git a/tests/tbf/TbfTest.cpp b/tests/tbf/TbfTest.cpp
index e67bd17..4adbaf8 100644
--- a/tests/tbf/TbfTest.cpp
+++ b/tests/tbf/TbfTest.cpp
@@ -122,7 +122,7 @@
/*
* Make a uplink and downlink allocation
*/
- ms = ms_alloc(bts);
+ ms = ms_alloc(bts, NULL);
gprs_rlcmac_tbf *dl_tbf = dl_tbf_alloc(bts,
ms, 0, false);
OSMO_ASSERT(dl_tbf != NULL);
@@ -205,7 +205,7 @@
GprsMs *ms;
gprs_rlcmac_dl_tbf *dl_tbf;
- ms = ms_alloc(bts);
+ ms = ms_alloc(bts, NULL);
ms_set_ms_class(ms, ms_class);
ms_set_egprs_ms_class(ms, egprs_ms_class);
@@ -2346,7 +2346,7 @@
gprs_bssgp_init(bts, 4234, 4234, 1, 1, false, 0, 0, 0);
/* Does no support EGPRS */
- ms = ms_alloc(bts);
+ ms = ms_alloc(bts, NULL);
ms_set_ms_class(ms, ms_class);
dl_tbf = dl_tbf_alloc(bts, ms, 0, false);
@@ -2355,7 +2355,7 @@
/* EGPRS-only */
/* Does support EGPRS */
- ms = ms_alloc(bts);
+ ms = ms_alloc(bts, NULL);
ms_set_ms_class(ms, ms_class);
ms_set_egprs_ms_class(ms, ms_class);
dl_tbf = dl_tbf_alloc(bts, ms, 0, false);
@@ -2397,7 +2397,7 @@
/* EGPRS-only */
/* Does support EGPRS */
- ms = ms_alloc(bts);
+ ms = ms_alloc(bts, NULL);
ms_set_ms_class(ms, ms_class);
ms_set_egprs_ms_class(ms, ms_class);
dl_tbf = dl_tbf_alloc(bts, ms, 0, true);
@@ -2476,7 +2476,7 @@
trx0->pdch[2].enable();
trx0->pdch[3].enable();
- second_ms = ms_alloc(bts);
+ second_ms = ms_alloc(bts, NULL);
ms_set_tlli(second_ms, new_tlli);
ul_tbf = ul_tbf_alloc(bts, second_ms, 0, true);
OSMO_ASSERT(ul_tbf != NULL);
@@ -3335,7 +3335,7 @@
int rc = 0;
- ms = ms_alloc(bts);
+ ms = ms_alloc(bts, NULL);
ms_set_tlli(ms, tlli);
ul_tbf = ms_new_ul_tbf_rejected_pacch(ms, pdch);