fixeria submitted this change.

View Change

Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve osmith: Looks good to me, but someone else must approve fixeria: Looks good to me, approved
trxcon: l1sched: ensure n_bits_total is always initialized

Change-Id: I590cfe55365b7ad021a3d0925a0f1ea136e67125
Related: CID#205451
---
M src/host/trxcon/src/sched_lchan_tchf.c
M src/host/trxcon/src/sched_lchan_tchh.c
2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/host/trxcon/src/sched_lchan_tchf.c b/src/host/trxcon/src/sched_lchan_tchf.c
index 1a1b93d..96acf0e 100644
--- a/src/host/trxcon/src/sched_lchan_tchf.c
+++ b/src/host/trxcon/src/sched_lchan_tchf.c
@@ -39,7 +39,7 @@
uint32_t fn, uint8_t bid, const sbit_t *bits,
const struct l1sched_meas_set *meas)
{
- int n_errors = -1, n_bits_total, rc;
+ int n_errors = -1, n_bits_total = 0, rc;
sbit_t *buffer, *offset;
uint8_t l2[128], *mask;
size_t l2_len;
diff --git a/src/host/trxcon/src/sched_lchan_tchh.c b/src/host/trxcon/src/sched_lchan_tchh.c
index a8c641b..12ddc15 100644
--- a/src/host/trxcon/src/sched_lchan_tchh.c
+++ b/src/host/trxcon/src/sched_lchan_tchh.c
@@ -195,7 +195,7 @@
uint32_t fn, uint8_t bid, const sbit_t *bits,
const struct l1sched_meas_set *meas)
{
- int n_errors = -1, n_bits_total, rc;
+ int n_errors = -1, n_bits_total = 0, rc;
sbit_t *buffer, *offset;
uint8_t l2[128], *mask;
size_t l2_len;

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

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