[M] Change in osmo-sgsn[master]: gprs_gmm_util: add parsing of GMM Attach Requests - gerrit-log

31 Jan 2025

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 &lt;lynxis(a)fe80.eu&gt;
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge &lt;laforge(a)osmocom.org&gt;
Gerrit-Reviewer: pespin &lt;pespin(a)sysmocom.de&gt;
Gerrit-CC: fixeria &lt;vyanitskiy(a)sysmocom.de&gt;
Gerrit-Attention: laforge &lt;laforge(a)osmocom.org&gt;
Gerrit-Attention: fixeria &lt;vyanitskiy(a)sysmocom.de&gt;
Gerrit-Comment-Date: Fri, 31 Jan 2025 14:57:41 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria &lt;vyanitskiy(a)sysmocom.de&gt;