mirror of https://gerrit.osmocom.org/libosmocore
gb: Use the NS-VCI to find an existing NS-VC
Currently when a NS-RESET is recevied over a link that has not yet been associated with a NS-VC, the NSEI is used to find an existing NS-VC. If one is found, the reset procedure is initiated. This behaviour is not conformant with 3GPP TS 08.16 (see chapter 4.2.3) which allows to use several NS-VC between two endpoints in parallel. The patch changes the implementation to use the NSVCI instead of the NSEI to search for an existing NS-VC object. Ticket: OW#874 Sponsored-by: On-Waves ehf
This commit is contained in:
parent
05395a63b7
commit
b6390f9bd8
|
@ -823,7 +823,7 @@ int gprs_ns_vc_create(struct gprs_ns_inst *nsi, struct msgb *msg,
|
|||
struct gprs_nsvc *existing_nsvc;
|
||||
|
||||
struct tlv_parsed tp;
|
||||
uint16_t nsei;
|
||||
uint16_t nsvci;
|
||||
|
||||
int rc;
|
||||
|
||||
|
@ -868,10 +868,10 @@ int gprs_ns_vc_create(struct gprs_ns_inst *nsi, struct msgb *msg,
|
|||
msg);
|
||||
return -EINVAL;
|
||||
}
|
||||
nsei = ntohs(*(uint16_t *) TLVP_VAL(&tp, NS_IE_NSEI));
|
||||
/* Check if we already know this NSEI, the remote end might
|
||||
nsvci = ntohs(*(uint16_t *) TLVP_VAL(&tp, NS_IE_VCI));
|
||||
/* Check if we already know this NSVCI, the remote end might
|
||||
* simply have changed addresses, or it is a SGSN */
|
||||
existing_nsvc = gprs_nsvc_by_nsei(nsi, nsei);
|
||||
existing_nsvc = gprs_nsvc_by_nsvci(nsi, nsvci);
|
||||
if (!existing_nsvc) {
|
||||
*new_nsvc = gprs_nsvc_create(nsi, 0xffff);
|
||||
log_set_context(GPRS_CTX_NSVC, *new_nsvc);
|
||||
|
|
|
@ -68,6 +68,7 @@ result (RESET) = 9
|
|||
|
||||
Current NS-VCIs:
|
||||
VCI 0x3344, NSEI 0x1122, peer 0x01020304:3333
|
||||
VCI 0x1122, NSEI 0x1122, peer 0x01020304:2222
|
||||
|
||||
--- Peer port changes, RESET, NSEI changes ---
|
||||
|
||||
|
@ -83,8 +84,8 @@ RESPONSE, msg length 1
|
|||
result (RESET) = 9
|
||||
|
||||
Current NS-VCIs:
|
||||
VCI 0x1122, NSEI 0x3344, peer 0x01020304:4444
|
||||
VCI 0x3344, NSEI 0x1122, peer 0x01020304:3333
|
||||
VCI 0x1122, NSEI 0x3344, peer 0x01020304:4444
|
||||
|
||||
--- Peer port 3333, RESET, VCI is changed back ---
|
||||
|
||||
|
@ -100,8 +101,8 @@ RESPONSE, msg length 1
|
|||
result (RESET) = 9
|
||||
|
||||
Current NS-VCIs:
|
||||
VCI 0x1122, NSEI 0x3344, peer 0x01020304:4444
|
||||
VCI 0x1122, NSEI 0x1122, peer 0x01020304:3333
|
||||
VCI 0x1122, NSEI 0x3344, peer 0x01020304:4444
|
||||
|
||||
--- Peer port 4444, RESET, NSEI is changed back ---
|
||||
|
||||
|
@ -117,8 +118,8 @@ RESPONSE, msg length 1
|
|||
result (RESET) = 9
|
||||
|
||||
Current NS-VCIs:
|
||||
VCI 0x1122, NSEI 0x1122, peer 0x01020304:4444
|
||||
VCI 0x1122, NSEI 0x1122, peer 0x01020304:3333
|
||||
VCI 0x1122, NSEI 0x1122, peer 0x01020304:4444
|
||||
|
||||
===== NS protocol test END
|
||||
|
||||
|
|
Loading…
Reference in New Issue