Attention is currently required from: msuraev.
neels has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-bsc/+/32421
)
Change subject: fix length check in abis_rsl_rx_rll()
......................................................................
Patch Set 1:
(1 comment)
Commit Message:
https://gerrit.osmocom.org/c/osmo-bsc/+/32421/comment/4fc547f2_df10360e
PS1, Line 21: - We are going to be accessing rrlh->data[3], so we must check for at
I think it's better to wrap this into static
function and add this comment there.
i agree but i'd rather fix the code dup as
separate patch; for now i just want to get the buffer access fix in...
--
To view, visit
https://gerrit.osmocom.org/c/osmo-bsc/+/32421
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ie4aee615c8c904ae8308ec0074d8bc5208137061
Gerrit-Change-Number: 32421
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: msuraev <msuraev(a)sysmocom.de>
Gerrit-Attention: msuraev <msuraev(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 27 Apr 2023 16:46:48 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: msuraev <msuraev(a)sysmocom.de>
Gerrit-MessageType: comment