fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/30962 )

Change subject: layer23: fix integer overflow in l1ctl_tx_data_req() ......................................................................

layer23: fix integer overflow in l1ctl_tx_data_req()

Found by clang:

l1ctl.c:397:52: warning: implicit conversion from 'int' to 'int8_t' (aka 'signed char') changes value from 255 to -1 [-Wconstant-conversion]

Change-Id: I23e9ea5ad59099c24db60057c8e7da1e6a0d2293 --- M src/host/layer23/src/common/l1ctl.c 1 file changed, 1 insertion(+), 1 deletion(-)

Approvals: dexter: Looks good to me, approved pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified

diff --git a/src/host/layer23/src/common/l1ctl.c b/src/host/layer23/src/common/l1ctl.c index 2695e53..5ba1a1f 100644 --- a/src/host/layer23/src/common/l1ctl.c +++ b/src/host/layer23/src/common/l1ctl.c @@ -395,7 +395,7 @@ if (rsl_dec_chan_nr(chan_nr, &chan_type, &chan_ss, &chan_ts) == 0) { uint8_t gsmtap_chan_type = chantype_rsl2gsmtap2(chan_type, link_id, false); gsmtap_send(gsmtap_inst, ms->rrlayer.cd_now.arfcn | GSMTAP_ARFCN_F_UPLINK, - chan_ts, gsmtap_chan_type, chan_ss, 0, 127, 255, + chan_ts, gsmtap_chan_type, chan_ss, 0, 127, 0, msg->l2h, msgb_l2len(msg)); } else { LOGP(DL1C, LOGL_ERROR,