<p><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/22777">View Change</a></p><p>10 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1//COMMIT_MSG">Commit Message:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1//COMMIT_MSG@10">Patch Set #1, Line 10:</a> <code style="font-family:monospace,monospace">SG informs the ASP about certain desetinations (point codes) becoming</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">destinations</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/m3ua.c">File src/m3ua.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/m3ua.c@777">Patch Set #1, Line 777:</a> <code style="font-family:monospace,monospace">          xua_msg_add_data(xua, M3UA_IEI_ROUTE_CTX, num_rctx * 4, (const uint8_t *)rctx);</code></p><ul><li>sizeof(*rctx) would be clearer.</li></ul></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/m3ua.c@779">Patch Set #1, Line 779:</a> <code style="font-family:monospace,monospace">      xua_msg_add_data(xua, M3UA_IEI_AFFECTED_PC, num_aff_pc * 4, (const uint8_t *) aff_pc);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">same for aff_pc</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/m3ua.c@817">Patch Set #1, Line 817:</a> <code style="font-family:monospace,monospace">//struct xua_msg *m3ua_encode_dupu() { }</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This line can be dropped?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/m3ua.c@820">Patch Set #1, Line 820:</a> <code style="font-family:monospace,monospace"> *  \param[in] asp ASP through whihc to transmit message. Must be ACTIVE.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">typo: which</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/m3ua.c@825">Patch Set #1, Line 825:</a> <code style="font-family:monospace,monospace"> *  \param[in] info_string optional information strng (can be NULL).</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">typo: string</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/m3ua.c@826">Patch Set #1, Line 826:</a> <code style="font-family:monospace,monospace"> *  \param[in] available are aff_pc now available (true) or unavailable (false */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">false)</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/xua_snm.c">File src/xua_snm.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/xua_snm.c@57">Patch Set #1, Line 57:</a> <code style="font-family:monospace,monospace">              out = talloc_asprintf_append(out, "%s%s/%u, ", i == 0 ? "" : ", ",</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I assume this frees+malloc new string?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/xua_snm.c@70">Patch Set #1, Line 70:</a> <code style="font-family:monospace,monospace">    llist_for_each_entry(as, &asp->inst->as_list, list) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">You are missing buffer overflow check against rctx_size here.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/22777/1/src/xua_snm.c@118">Patch Set #1, Line 118:</a> <code style="font-family:monospace,monospace">            /* this can happen if the given ASP is only in the as that reports the change,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">AS</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/22777">change 22777</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/libosmo-sccp/+/22777"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmo-sccp </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Id92be4691b0fd77598a6edb642c028bbd8c5b623 </div>
<div style="display:none"> Gerrit-Change-Number: 22777 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 08 Feb 2021 10:20:38 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>