<p>laforge <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-msc/+/18995">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Jenkins Builder: Verified
  pespin: Looks good to me, but someone else must approve
  laforge: Looks good to me, approved

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">add comments to clarify some complete l3 details<br><br>Change-Id: I6e289900d38d97c346d615b71d36656254e6f2b5<br>---<br>M src/libmsc/gsm_04_08.c<br>M src/libmsc/msc_a.c<br>2 files changed, 9 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/libmsc/gsm_04_08.c b/src/libmsc/gsm_04_08.c</span><br><span>index a70c4b9..15303b5 100644</span><br><span>--- a/src/libmsc/gsm_04_08.c</span><br><span>+++ b/src/libmsc/gsm_04_08.c</span><br><span>@@ -1396,7 +1396,9 @@</span><br><span>         msub_update_id(msub);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-/* VLR informs us that the subscriber has been associated with a conn */</span><br><span style="color: hsl(120, 100%, 40%);">+/* VLR informs us that the subscriber has been associated with a conn.</span><br><span style="color: hsl(120, 100%, 40%);">+ * The subscriber has *not* been authenticated yet, so the vsub should be protected from potentially invalid information</span><br><span style="color: hsl(120, 100%, 40%);">+ * from the msc_a. */</span><br><span> static int msc_vlr_subscr_assoc(void *msc_conn_ref,</span><br><span>                                struct vlr_subscr *vsub)</span><br><span> {</span><br><span>@@ -1406,6 +1408,10 @@</span><br><span> </span><br><span>  if (msub_set_vsub(msub, vsub))</span><br><span>               return -EINVAL;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+     /* FIXME: would be better to modify vsub->* only after the subscriber is authenticated, in</span><br><span style="color: hsl(120, 100%, 40%);">+  * evaluate_acceptance_outcome(conn_accepted == true). */</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>  vsub->cs.attached_via_ran = msc_a->c.ran->type;</span><br><span> </span><br><span>         /* In case we have already received Classmark Information before the VLR Subscriber was</span><br><span>diff --git a/src/libmsc/msc_a.c b/src/libmsc/msc_a.c</span><br><span>index 3bfc0f8..fd66ad3 100644</span><br><span>--- a/src/libmsc/msc_a.c</span><br><span>+++ b/src/libmsc/msc_a.c</span><br><span>@@ -1375,6 +1375,8 @@</span><br><span>         switch (msg->msg_type) {</span><br><span> </span><br><span>      case RAN_MSG_COMPL_L3:</span><br><span style="color: hsl(120, 100%, 40%);">+                /* In case the cell_id from Complete Layer 3 Information lacks a PLMN, write the configured PLMN code</span><br><span style="color: hsl(120, 100%, 40%);">+          * into msc_a->via_cell. Then overwrite with those bits obtained from Complete Layer 3 Information. */</span><br><span>            msc_a->via_cell = (struct osmo_cell_global_id){</span><br><span>                   .lai.plmn = msc_a_net(msc_a)->plmn,</span><br><span>               };</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-msc/+/18995">change 18995</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/osmo-msc/+/18995"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-msc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I6e289900d38d97c346d615b71d36656254e6f2b5 </div>
<div style="display:none"> Gerrit-Change-Number: 18995 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>