Attention is currently required from: fixeria, laforge.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/35366?usp=email )
Change subject: LTE_CryptoFunctions.ttcn: Use received SeqNum as nus.rx_count
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS2:
Thanks @vyanitskiy@sysmocom.de for the support on finding out what may be the proper values to feed to the integrity check function!
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/35366?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I1ee25fd8fde6fbce95444ee4b73918ce1f61b427
Gerrit-Change-Number: 35366
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 15 Dec 2023 15:12:46 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: fixeria, laforge.
Hello Jenkins Builder, fixeria,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/35366?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
Change subject: LTE_CryptoFunctions.ttcn: Use received SeqNum as nus.rx_count
......................................................................
LTE_CryptoFunctions.ttcn: Use received SeqNum as nus.rx_count
Before this patch, the rx_count field was never set to anything other
than zero.
This makes f_nas_check_ip() success when receiving messages from
open5gs-mmed.
Change-Id: I1ee25fd8fde6fbce95444ee4b73918ce1f61b427
---
M mme/LTE_CryptoFunctions.ttcn
1 file changed, 16 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/66/35366/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/35366?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I1ee25fd8fde6fbce95444ee4b73918ce1f61b427
Gerrit-Change-Number: 35366
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: daniel, dexter, fixeria, laforge.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/35367?usp=email )
Change subject: mme: Implement missing UE attach steps
......................................................................
Patch Set 2:
(1 comment)
File library/s1ap/S1AP_Templates.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/35367/comment/60cad4ac_5cb2…
PS2, Line 603: /*OPTIONAL: template (value) E_RABList rab_items */
> Couldn't this be template(omit) here?
No, I tried and it throws Dynamic Test Error when trying to assing E_RABList := omit.
The main problem here is that the ASN1 generator doesn't provide us with a ttcn3 test case AFAIK to be able to easily pass the full IE {id:=...} as a template...
Anyway, this template was not used yet, I'm the first user, and this is not needed in the usual success scenario. If somebody needs a fail scenario, they can add a new template with that one enabled.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/35367?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ib4a69aef40bf9dd17d640f650e459668f8415aaa
Gerrit-Change-Number: 35367
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: daniel <dwillmann(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 15 Dec 2023 14:53:41 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: daniel <dwillmann(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: daniel, pespin.
osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-iuh/+/35374?usp=email )
Change subject: Bump version: 1.5.0.3-a99b → 1.5.1
......................................................................
Patch Set 1:
(1 comment)
File src/Makefile.am:
https://gerrit.osmocom.org/c/osmo-iuh/+/35374/comment/e1ef76d5_893c172b
PS1, Line 64: HNBAP_LIBVERSION=1:1:1
> I think iu_client. […]
right! same with iu_helpers.c, which was also modified.
--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/35374?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I01d1cebfe4fdd0bce53260d69e3f037068194ea8
Gerrit-Change-Number: 35374
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: daniel <dwillmann(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 15 Dec 2023 14:19:33 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: daniel, osmith, pespin.
Hello Jenkins Builder, daniel, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-iuh/+/35374?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Code-Review-1 by pespin, Verified+1 by Jenkins Builder
Change subject: Bump version: 1.5.0.3-a99b → 1.5.1
......................................................................
Bump version: 1.5.0.3-a99b → 1.5.1
Change-Id: I01d1cebfe4fdd0bce53260d69e3f037068194ea8
---
M debian/changelog
M src/Makefile.am
2 files changed, 21 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-iuh refs/changes/74/35374/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/35374?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I01d1cebfe4fdd0bce53260d69e3f037068194ea8
Gerrit-Change-Number: 35374
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: daniel <dwillmann(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: daniel, osmith.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-iuh/+/35374?usp=email )
Change subject: Bump version: 1.5.0.3-a99b → 1.5.1
......................................................................
Patch Set 1: Code-Review-1
(1 comment)
File src/Makefile.am:
https://gerrit.osmocom.org/c/osmo-iuh/+/35374/comment/b3680a8b_8fe76888
PS1, Line 64: HNBAP_LIBVERSION=1:1:1
I think iu_client.c only shows up libosmo_ranap_la_SOURCES, so only RANAP_LIBVERSION should be increased, not the others. Do check yourself too just in case.
--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/35374?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I01d1cebfe4fdd0bce53260d69e3f037068194ea8
Gerrit-Change-Number: 35374
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: daniel <dwillmann(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 15 Dec 2023 14:11:19 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment