31 Jan
2025
31 Jan
'25
3:57 p.m.
Attention is currently required from: fixeria, laforge.
lynxis lazus has posted comments on this change by lynxis lazus. (
https://gerrit.osmocom.org/c/osmo-sgsn/+/39246?usp=email )
Change subject: gprs_gmm_util: add parsing of GMM Attach Requests
......................................................................
Patch Set 3:
(6 comments)
File src/sgsn/gprs_gmm_util.c:
https://gerrit.osmocom.org/c/osmo-sgsn/+/39246/comment/ac78f87d_cbcbb795?us… :
PS3, Line 93: l3 pointers must point to gmm
what does this even mean...
l3 pointer of msgb
must point to the (start of) GMM
https://gerrit.osmocom.org/c/osmo-sgsn/+/39246/comment/2147b019_a681bd29?us… :
PS3, Line 94: rau_req
(copy-paste) `rau_req` -> `att_req`
Acknowledged
https://gerrit.osmocom.org/c/osmo-sgsn/+/39246/comment/2c77e7ed_0f9e7052?us… :
PS3, Line 97: struct msgb *msg
`const`? you're not modifying it, right?
Done
https://gerrit.osmocom.org/c/osmo-sgsn/+/39246/comment/c8c3a175_be17578d?us… :
PS3, Line 119: 21
where this magic number is coming from? do we have a
define for it?
The problem here are the dynamic, but mandatory, IEs in form of LV in
GMM.
But there are multiple dynamic LV IEs in the Attach Req.
So the GMM Attach Req can't be smaller then 25 bytes.
After MS Network Capability come at least 21 bytes.
https://gerrit.osmocom.org/c/osmo-sgsn/+/39246/comment/c5a33036_62bf4f34?us… :
PS3, Line 122: if (len == 0)
"must be at least 2 bytes long" suggests
`len < 2`?
Done
https://gerrit.osmocom.org/c/osmo-sgsn/+/39246/comment/8efafa3b_bd71876f?us… :
PS3, Line 143: 12
likewise
See the mention upwards. I wouldn't
use a define for a value here.
--
To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/39246?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I4a82e0c4070da982efd6c2f15fd145393423772b
Gerrit-Change-Number: 39246
Gerrit-PatchSet: 3
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 31 Jan 2025 14:57:41 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>