Patch Set 2: Code-Review+2 (1 comment)
https://gerrit.osmocom.org/#/c/2313/2/src/common/rsl.c File src/common/rsl.c: Line 439: len = sizeof(sysinfo_buf_t) - 2; does truncation here make sense? should we not at the very least print a LOGL_ERROR message that we are effectively generating broken messages? I know you just moved the code, but maybe as a follow-up commit? -- To view, visit https://gerrit.osmocom.org/2313 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie97be6ead6ce6d2d425fbfac8429bb90afb95acc Gerrit-PatchSet: 2 Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Owner: Max <[email protected]> Gerrit-Reviewer: Harald Welte <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-HasComments: Yes
