diff options
author | Neels Hofmeyr <neels@hofmeyr.de> | 2020-06-24 14:22:52 +0200 |
---|---|---|
committer | Neels Hofmeyr <neels@hofmeyr.de> | 2020-06-24 14:22:52 +0200 |
commit | 68f50dad76818e4173cdb5f1474cd9286e00f495 (patch) | |
tree | f564566b83874ea64f172adc1b0a4a75edcc1508 /src/libmsc/gsm_04_08.c | |
parent | a4a45252df0aab3795bf104fa423b2c46d001336 (diff) |
add comments to clarify some complete l3 details
Change-Id: I6e289900d38d97c346d615b71d36656254e6f2b5
Diffstat (limited to 'src/libmsc/gsm_04_08.c')
-rw-r--r-- | src/libmsc/gsm_04_08.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/libmsc/gsm_04_08.c b/src/libmsc/gsm_04_08.c index a70c4b918..15303b599 100644 --- a/src/libmsc/gsm_04_08.c +++ b/src/libmsc/gsm_04_08.c @@ -1396,7 +1396,9 @@ static void msc_vlr_subscr_update(struct vlr_subscr *subscr) msub_update_id(msub); } -/* VLR informs us that the subscriber has been associated with a conn */ +/* VLR informs us that the subscriber has been associated with a conn. + * The subscriber has *not* been authenticated yet, so the vsub should be protected from potentially invalid information + * from the msc_a. */ static int msc_vlr_subscr_assoc(void *msc_conn_ref, struct vlr_subscr *vsub) { @@ -1406,6 +1408,10 @@ static int msc_vlr_subscr_assoc(void *msc_conn_ref, if (msub_set_vsub(msub, vsub)) return -EINVAL; + + /* FIXME: would be better to modify vsub->* only after the subscriber is authenticated, in + * evaluate_acceptance_outcome(conn_accepted == true). */ + vsub->cs.attached_via_ran = msc_a->c.ran->type; /* In case we have already received Classmark Information before the VLR Subscriber was |