This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
pespin gerrit-no-reply at lists.osmocom.orgpespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-sccp/+/22777 ) Change subject: xua: Implement SNM availability/unavailability messaging ...................................................................... Patch Set 1: (10 comments) https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1//COMMIT_MSG Commit Message: https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1//COMMIT_MSG@10 PS1, Line 10: SG informs the ASP about certain desetinations (point codes) becoming destinations https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/m3ua.c File src/m3ua.c: https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/m3ua.c@777 PS1, Line 777: xua_msg_add_data(xua, M3UA_IEI_ROUTE_CTX, num_rctx * 4, (const uint8_t *)rctx); * sizeof(*rctx) would be clearer. https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/m3ua.c@779 PS1, Line 779: xua_msg_add_data(xua, M3UA_IEI_AFFECTED_PC, num_aff_pc * 4, (const uint8_t *) aff_pc); same for aff_pc https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/m3ua.c@817 PS1, Line 817: //struct xua_msg *m3ua_encode_dupu() { } This line can be dropped? https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/m3ua.c@820 PS1, Line 820: * \param[in] asp ASP through whihc to transmit message. Must be ACTIVE. typo: which https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/m3ua.c@825 PS1, Line 825: * \param[in] info_string optional information strng (can be NULL). typo: string https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/m3ua.c@826 PS1, Line 826: * \param[in] available are aff_pc now available (true) or unavailable (false */ false) https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/xua_snm.c File src/xua_snm.c: https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/xua_snm.c@57 PS1, Line 57: out = talloc_asprintf_append(out, "%s%s/%u, ", i == 0 ? "" : ", ", I assume this frees+malloc new string? https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/xua_snm.c@70 PS1, Line 70: llist_for_each_entry(as, &asp->inst->as_list, list) { You are missing buffer overflow check against rctx_size here. https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/xua_snm.c@118 PS1, Line 118: /* this can happen if the given ASP is only in the as that reports the change, AS -- To view, visit https://gerrit.osmocom.org/c/libosmo-sccp/+/22777 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-sccp Gerrit-Branch: master Gerrit-Change-Id: Id92be4691b0fd77598a6edb642c028bbd8c5b623 Gerrit-Change-Number: 22777 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge at osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin at sysmocom.de> Gerrit-Comment-Date: Mon, 08 Feb 2021 10:20:38 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210208/92112b0f/attachment.htm>