pespin has submitted this change. (
https://gerrit.osmocom.org/c/libosmo-gprs/+/31537 )
Change subject: tbf_dl: Fix state_fsm instance not freed
......................................................................
tbf_dl: Fix state_fsm instance not freed
Change-Id: I2f89810dd346cee904dcff0b598e229bd5f84a9f
---
M src/rlcmac/tbf_dl.c
M tests/rlcmac/rlcmac_prim_test.err
2 files changed, 11 insertions(+), 1 deletion(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, but someone else must approve
pespin: Looks good to me, approved
diff --git a/src/rlcmac/tbf_dl.c b/src/rlcmac/tbf_dl.c
index 3861cae..13b4aab 100644
--- a/src/rlcmac/tbf_dl.c
+++ b/src/rlcmac/tbf_dl.c
@@ -76,7 +76,7 @@
gprs_rlcmac_rlc_dl_window_free(dl_tbf->dlw);
dl_tbf->dlw = NULL;
- //gprs_rlcmac_tbf_dl_fsm_destructor(dl_tbf);
+ gprs_rlcmac_tbf_dl_fsm_destructor(dl_tbf);
gprs_rlcmac_tbf_destructor(dl_tbf_as_tbf(dl_tbf));
talloc_free(dl_tbf);
diff --git a/tests/rlcmac/rlcmac_prim_test.err
b/tests/rlcmac/rlcmac_prim_test.err
index 5946e0a..0d8dd3a 100644
--- a/tests/rlcmac/rlcmac_prim_test.err
+++ b/tests/rlcmac/rlcmac_prim_test.err
@@ -619,3 +619,4 @@
DLGLOBAL INFO Rx from lower layers: L1CTL-PDCH_RTS.indication
DLGLOBAL DEBUG (ts=7,fn=21,usf=0) Tx DL ACK/NACK FinalAck=1
DLGLOBAL DEBUG TBF(DL:NR-0:TLLI-00000001) - V(N):
"IIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIR" R=Received
I=Invalid
+DLGLOBAL INFO DL_TBF{FINISHED}: Deallocated
--
To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/31537
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: I2f89810dd346cee904dcff0b598e229bd5f84a9f
Gerrit-Change-Number: 31537
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pes...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-Reviewer: pespin <pes...@sysmocom.de>
Gerrit-MessageType: merged
