laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/40424?usp=email )
Change subject: Corrected description of MSISDN IE and IMEI IE coding
......................................................................
Corrected description of MSISDN IE and IMEI IE coding
1 2 3 4 5 6 7 8
+-------------------------------+
| MSISDN IE type |Res| octet 1
+-------------------------------+
| Length of IE content | octet 2
+-------------------------------+
| Length of BCD content | octet 3
+-------------------------------+
| Digit 1 | Digit 2 | octet 4..n
| .... | |
+-------------------------------+
The length of the BCD content specifies how many subsequent octets
contain BCD data (it does *not* indicate the number of digits).
This length must be less than the total length of the IE content.
Any octets following the BCD content shall be ignored.
Related: OS#6797
Change-Id: I4db05e4a01199334b75fffd60270b88f541b98f4
---
M common/chapters/gsup.adoc
1 file changed, 6 insertions(+), 3 deletions(-)
Approvals:
laforge: Looks good to me, approved
pespin: Looks good to me, but someone else must approve
Jenkins Builder: Verified
diff --git a/common/chapters/gsup.adoc b/common/chapters/gsup.adoc
index f5cce14..824ac90 100644
--- a/common/chapters/gsup.adoc
+++ b/common/chapters/gsup.adoc
@@ -1510,6 +1510,10 @@
BCD Number in 3GPP TS 24.008. It will be stored by the SGSN or VLR and then passed as is
to the GGSN during the activation of the primary PDP Context.
+The length of the BCD content specifies how many subsequent octets contain BCD data
+(it does *not* indicate the number of digits). This length must be less than the total
+length of the IE content. Any octets following the BCD content shall be ignored.
+
[packetdiag]
----
{
@@ -1519,9 +1523,7 @@
0-6: PDP type IEI
7: Res
8-15: Length of IE content
- 16-19: NPI
- 20-22: TON
- 23: ext
+ 16-23: Length of BCD content
24-27: Digit 1
28-31: Digit 2
32-35: Digit ...
@@ -1628,6 +1630,7 @@
==== IMEI
The IMEI encoded as Called Party BCD Number in 3GPP TS 24.008.
+This IE has the same format as described in <<gsup-ie-msisdn>>.
[[gsup-ie-imei-result]]
==== IMEI Check Result
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/40424?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: osmo-gsm-manuals
Gerrit-Branch: master
Gerrit-Change-Id: I4db05e4a01199334b75fffd60270b88f541b98f4
Gerrit-Change-Number: 40424
Gerrit-PatchSet: 3
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Attention is currently required from: pespin.
laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40421?usp=email )
Change subject: S1AP_Emulation: Remove commented duplicated function
......................................................................
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40421?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I1d6871d2a735a4bf39c87f6c976bd5d5630e9c04
Gerrit-Change-Number: 40421
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 04 Jun 2025 13:16:21 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Attention is currently required from: pespin.
laforge has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40420?usp=email )
Change subject: mme: Move LTE_CryptoFunction* to library/
......................................................................
Patch Set 3: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40420?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I76f86beeb2c49110eea0fac6abffee8307a3c22c
Gerrit-Change-Number: 40420
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 04 Jun 2025 13:16:06 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Attention is currently required from: daniel, fixeria, jolly, laforge, pespin.
Hello Jenkins Builder, daniel, fixeria, jolly, laforge,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40413?usp=email
to look at the new patch set (#9).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
Change subject: 5gc: Implement AKA authentication
......................................................................
5gc: Implement AKA authentication
deps/nas.git already has some Auth related functionaltities to calculate
XRES*, but I couldn't get them to generate proper output. Furthermore,
using them directly would include a lot of churn we don't want:
* ttcn: Otherwise ttcn/Lib_NG_NAS/LIB_NG_NAS_Functions.ttcn brings in
tons of dependencies about Emulation stuff which is not interesting
for us.
* .cc: We want to implement our own low level functions using our
eclipse Titan API, as well as our own C lib dependencies (because some
code for those dependencies is not really available in the repo).
Hence, instead use our own milenage.c implementation imported from
libosmocore (actually from a slightly modified self-contained copy we
already ported to some development branch of asterisk VoLTE project).
With this patch it is already possible to detect SQN Out-of-sync and
answer with an Auth Failure, then get a new Auth Req from network and
properly answer with an Auth Resp which the network likes and hence
answers with a Security Mode Command.
Change-Id: I11527f47e4310863124f3f02148e3f71da7d911e
---
M 5gc/C5G_Tests.cfg
M 5gc/C5G_Tests.ttcn
M 5gc/gen_links.sh
M 5gc/regen_makefile.sh
M deps/Makefile
M library/General_Types.ttcn
A library/NG_CryptoFunctionDefs.cc
A library/NG_CryptoFunctions.ttcn
M library/NG_NAS_Osmo_Templates.ttcn
A library/milenage/Milenage_FunctionDefs.cc
A library/milenage/Milenage_Functions.ttcn
A library/milenage/milenage.c
A library/milenage/milenage.h
A library/ng_crypto/key_derivation.c
A library/ng_crypto/key_derivation.h
15 files changed, 837 insertions(+), 17 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/13/40413/9
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40413?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I11527f47e4310863124f3f02148e3f71da7d911e
Gerrit-Change-Number: 40413
Gerrit-PatchSet: 9
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: daniel <dwillmann(a)sysmocom.de>
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40420?usp=email
to look at the new patch set (#3).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
Change subject: mme: Move LTE_CryptoFunction* to library/
......................................................................
mme: Move LTE_CryptoFunction* to library/
S1AP_Emulation is already importing LTE_CryptoFunction.ttcn, so it
really belongs to library and not to mme/ directory.
While at it, split Snow3G algos to its own subdir so it can be easily
exported standalone, eg. when used by 5G NAS in the future.
Change-Id: I76f86beeb2c49110eea0fac6abffee8307a3c22c
---
A library/LTE_CryptoFunctionDefs.cc
R library/LTE_CryptoFunctions.ttcn
R library/lte_crypto/key_derivation.c
R library/lte_crypto/key_derivation.h
R library/snow_3g/Snow3G_FunctionDefs.cc
A library/snow_3g/Snow3G_Functions.ttcn
R library/snow_3g/snow-3g.c
R library/snow_3g/snow-3g.h
M mme/gen_links.sh
M mme/regen_makefile.sh
10 files changed, 125 insertions(+), 72 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/20/40420/3
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40420?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I76f86beeb2c49110eea0fac6abffee8307a3c22c
Gerrit-Change-Number: 40420
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Attention is currently required from: jolly.
pespin has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/40424?usp=email )
Change subject: Corrected description of MSISDN IE and IMEI IE coding
......................................................................
Patch Set 3: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/40424?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-gsm-manuals
Gerrit-Branch: master
Gerrit-Change-Id: I4db05e4a01199334b75fffd60270b88f541b98f4
Gerrit-Change-Number: 40424
Gerrit-PatchSet: 3
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Comment-Date: Wed, 04 Jun 2025 10:22:44 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Attention is currently required from: pespin.
jolly has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/40424?usp=email )
Change subject: Corrected description of MSISDN IE and IMEI IE coding
......................................................................
Patch Set 3:
(1 comment)
File common/chapters/gsup.adoc:
https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/40424/comment/4ea558ea_cd5c… :
PS1, Line 1513: The length of the BCD content specifies how many subsequent octets contain BCD data,
> Uuuh well I meant specially on the gsup. […]
Done
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/40424?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-gsm-manuals
Gerrit-Branch: master
Gerrit-Change-Id: I4db05e4a01199334b75fffd60270b88f541b98f4
Gerrit-Change-Number: 40424
Gerrit-PatchSet: 3
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 04 Jun 2025 10:05:00 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>