pespin submitted this change.

View Change


Approvals: Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve pespin: Looks good to me, approved
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(-)

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 change 31537. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: I2f89810dd346cee904dcff0b598e229bd5f84a9f
Gerrit-Change-Number: 31537
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged