Attention is currently required from: laforge, pespin.

osmith has posted comments on this change by osmith. ( 
https://gerrit.osmocom.org/c/osmo-ggsn/+/38383?usp=email )

Change subject: gtp: allow packets with E-bit set
......................................................................


Patch Set 1:

(1 comment)

Commit Message:

https://gerrit.osmocom.org/c/osmo-ggsn/+/38383/comment/8ac75fa7_10cc00a1?usp=email
 :
PS1, Line 17: Related: OS#6223
> I think this needs some spec analysis of what the expected behaviour is when 
> packets with unknown/un […]
I've looked into it some more: 3GPP TS 29.060 § 6 towards the end explains when 
comprehension is required or not, it depends on bits 7 and 8 of the Extension 
Header Type. If comprehension is required, but the recipient doesn't understand 
it, it is supposed to:
* send a response with cause "unknown mandatory extension header"
* send a Supported Extension Headers Notification
* log an error

So I guess the right thing to do would be:
* accept packets with the E-bit set
* iterate over all extension headers
  * look at the comprehension bits, and if no comprehension is required then 
ignore the extension header
  * otherwise reject the packet with the steps listed above

@[email protected]: do you want me to implement this, or put it in a new 
ticket? I'm not sure how important this is.



--
To view, visit https://gerrit.osmocom.org/c/osmo-ggsn/+/38383?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: osmo-ggsn
Gerrit-Branch: master
Gerrit-Change-Id: Ic62f95874a877a2f295765b807d7f23b011aeaa3
Gerrit-Change-Number: 38383
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-CC: pespin <[email protected]>
Gerrit-Attention: laforge <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Comment-Date: Mon, 14 Oct 2024 09:37:50 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <[email protected]>
Comment-In-Reply-To: pespin <[email protected]>

Reply via email to