Attention is currently required from: jolly. Jenkins Builder has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/33611 )
Change subject: ASCI: Add FSM for VGCS/VBS ...................................................................... Patch Set 2: (4 comments) File include/osmocom/bsc/vgcs_fsm.h: Robot Comment from checkpatch (run ID jenkins-gerrit-lint-9058): https://gerrit.osmocom.org/c/osmo-bsc/+/33611/comment/6ada7a28_7de88b8f PS2, Line 59: /* Channel link failed. (Talker faild.) */ 'faild' may be misspelled - perhaps 'failed'? File src/osmo-bsc/vgcs_fsm.c: Robot Comment from checkpatch (run ID jenkins-gerrit-lint-9058): https://gerrit.osmocom.org/c/osmo-bsc/+/33611/comment/5aaafc56_77d02d48 PS2, Line 373: LOG_CALL(conn, LOGL_DEBUG, "Calling subscriber assiged and now on uplink.\n"); 'assiged' may be misspelled - perhaps 'assigned'? Robot Comment from checkpatch (run ID jenkins-gerrit-lint-9058): https://gerrit.osmocom.org/c/osmo-bsc/+/33611/comment/f64aba1b_6cd34c9d PS2, Line 668: no_aoip: labels should not be indented Robot Comment from checkpatch (run ID jenkins-gerrit-lint-9058): https://gerrit.osmocom.org/c/osmo-bsc/+/33611/comment/cb4735c0_c12579b8 PS2, Line 1155: strcpy(conn->vgcs_chan.msc_rtp_addr, req.msc_rtp_addr); Instead of strncpy()/strcpy(), use osmo_strlcpy() or OSMO_STRLCPY_ARRAY() -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/33611 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: Id9e94fb4f27bb438b7093c031344a3400bfa34f1 Gerrit-Change-Number: 33611 Gerrit-PatchSet: 2 Gerrit-Owner: jolly <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Attention: jolly <[email protected]> Gerrit-Comment-Date: Thu, 06 Jul 2023 14:41:52 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
