lynxis lazus has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/43599?usp=email )

(

1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: gprs_gmm: Routing Area Update: ensure PDP_CTX_STATUS has correct lenght ......................................................................

gprs_gmm: Routing Area Update: ensure PDP_CTX_STATUS has correct lenght

Change-Id: Ic9560108ee424634e15a5af1424ea44df491a474 --- M src/sgsn/gprs_gmm.c 1 file changed, 1 insertion(+), 1 deletion(-)

Approvals: pespin: Looks good to me, approved Jenkins Builder: Verified fixeria: Looks good to me, but someone else must approve

diff --git a/src/sgsn/gprs_gmm.c b/src/sgsn/gprs_gmm.c index 976e710..fcb243f 100644 --- a/src/sgsn/gprs_gmm.c +++ b/src/sgsn/gprs_gmm.c @@ -1782,7 +1782,7 @@

/* Look at PDP Context Status IE and see if MS's view of * activated/deactivated NSAPIs agrees with our view */ - if (TLVP_PRESENT(&req.tlv, GSM48_IE_GMM_PDP_CTX_STATUS)) { + if (TLVP_PRES_LEN(&req.tlv, GSM48_IE_GMM_PDP_CTX_STATUS, 2)) { uint16_t pdp_status = osmo_load16le(TLVP_VAL(&req.tlv, GSM48_IE_GMM_PDP_CTX_STATUS)); process_ms_ctx_status(mmctx, pdp_status); }