pespin submitted this change.

View Change


Approvals: Jenkins Builder: Verified fixeria: Looks good to me, approved
rlcmac: Fix null ptr access in dl_tbf object

Fixes: Coverity CID#310022
Change-Id: I3817d9e77a440682fa2c538e0fbee295ffdbcfff
---
M src/rlcmac/rlcmac.c
1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/src/rlcmac/rlcmac.c b/src/rlcmac/rlcmac.c
index b273b68..260749c 100644
--- a/src/rlcmac/rlcmac.c
+++ b/src/rlcmac/rlcmac.c
@@ -404,7 +404,7 @@

dl_tbf = gprs_rlcmac_find_dl_tbf_by_tfi(rlc_dec.tfi);
if (!dl_tbf) {
- LOGPTBFDL(dl_tbf, LOGL_INFO, "Rx DL data for unknown dl_tfi=%u\n", data_hdr->tfi);
+ LOGRLCMAC(LOGL_INFO, "Rx DL data for unknown dl_tfi=%u\n", data_hdr->tfi);
return -ENOENT;
}
LOGPTBFDL(dl_tbf, LOGL_DEBUG, "Rx new DL data\n");

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

Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: I3817d9e77a440682fa2c538e0fbee295ffdbcfff
Gerrit-Change-Number: 31437
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged