laforge submitted this change.

View Change


Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved
virt_phy: fix wrong pointer being checked

Change-Id: I2f845b0335d2d9818aa15c30f8621acd120e82d4
Fixes: fa833e40 "l1gprs/l1ctl: Decouple RTS.ind from DL_BLOCK.ind"
Fixes: CID#323121
---
M src/host/virt_phy/src/virt_prim_pdch.c
1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/src/host/virt_phy/src/virt_prim_pdch.c b/src/host/virt_phy/src/virt_prim_pdch.c
index 6825ced..3fa7977 100644
--- a/src/host/virt_phy/src/virt_prim_pdch.c
+++ b/src/host/virt_phy/src/virt_prim_pdch.c
@@ -103,6 +103,6 @@
* resulting value is first fn of the next block. */
const uint32_t rts_fn = GSM_TDMA_FN_SUM(fn, (fn % 13 == 8) ? 5 : 4);
nmsg = l1gprs_handle_rts_ind(ms->gprs, rts_fn, tn, usf);
- if (msg != NULL)
+ if (nmsg != NULL)
l1ctl_sap_tx_to_l23_inst(ms, nmsg);
}

To view, visit change 34319. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I2f845b0335d2d9818aa15c30f8621acd120e82d4
Gerrit-Change-Number: 34319
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged