December 2025 - gerrit-log - lists.osmocom.org

[M] Change in libosmo-asn1-tcap[master]: Remove unneeded asn1c-fixup.patch
by pespin 04 Dec '25

04 Dec '25

pespin has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-asn1-tcap/+/41582?usp=email ) Change subject: Remove unneeded asn1c-fixup.patch ...................................................................... Remove unneeded asn1c-fixup.patch This is not used nowadays. Change-Id: I219e17e9439405dc3488b437a234623807f1d8ab --- D asn1c-fixup.patch 1 file changed, 0 insertions(+), 163 deletions(-) Approvals: daniel: Looks good to me, approved Jenkins Builder: Verified diff --git a/asn1c-fixup.patch b/asn1c-fixup.patch deleted file mode 100644 index 1b29076..0000000 --- a/asn1c-fixup.patch +++ /dev/null @@ -1,163 +0,0 @@ -This patch needs to be applied to the asn1c output in order to -make sure we don't re-define the 'user_information' for every different -Dialogue portion PDU type and to make the EXTERNAL magic work -diff -Nru /usr/local/include/osmocom/tcap//AARE-apdu.h ./AARE-apdu.h ---- /usr/local/include/osmocom/tcap//AARE-apdu.h 2010-07-16 12:50:01.000000000 +0200 -+++ ./AARE-apdu.h 2010-07-09 15:40:49.000000000 +0200 -@@ -15,7 +15,7 @@ - #include <OBJECT_IDENTIFIER.h> - #include "Associate-result.h" - #include "Associate-source-diagnostic.h" --#include "EXTERNAL.h" -+#include <EXTERNAL.h> - #include <asn_SEQUENCE_OF.h> - #include <constr_SEQUENCE_OF.h> - #include <constr_SEQUENCE.h> -@@ -35,17 +35,12 @@ - OBJECT_IDENTIFIER_t application_context_name; - Associate_result_t result; - Associate_source_diagnostic_t result_source_diagnostic; --#if 0 - struct AARE_user_information { - A_SEQUENCE_OF(EXTERNAL_t) list; - - /* Context for parsing across buffer boundaries */ - asn_struct_ctx_t _asn_ctx; - } *user_information; --#else -- /* use the definition from ABRT-apdu.h */ -- struct user_information *user_information; --#endif - - /* Context for parsing across buffer boundaries */ - asn_struct_ctx_t _asn_ctx; -diff -Nru /usr/local/include/osmocom/tcap//AARQ-apdu.h ./AARQ-apdu.h ---- /usr/local/include/osmocom/tcap//AARQ-apdu.h 2010-07-16 12:50:00.000000000 +0200 -+++ ./AARQ-apdu.h 2010-07-09 15:40:49.000000000 +0200 -@@ -13,7 +13,7 @@ - /* Including external dependencies */ - #include <BIT_STRING.h> - #include <OBJECT_IDENTIFIER.h> --#include "EXTERNAL.h" -+#include <EXTERNAL.h> - #include <asn_SEQUENCE_OF.h> - #include <constr_SEQUENCE_OF.h> - #include <constr_SEQUENCE.h> -@@ -26,17 +26,12 @@ - typedef struct AARQ_apdu { - BIT_STRING_t *protocol_version /* DEFAULT {version1} */; - OBJECT_IDENTIFIER_t application_context_name; --#if 0 - struct AARQ_user_information { - A_SEQUENCE_OF(EXTERNAL_t) list; - - /* Context for parsing across buffer boundaries */ - asn_struct_ctx_t _asn_ctx; - } *user_information; --#else -- /* use the definition from ABRT-apdu.h */ -- struct user_information *user_information; --#endif - - /* Context for parsing across buffer boundaries */ - asn_struct_ctx_t _asn_ctx; -diff -Nru /usr/local/include/osmocom/tcap//ABRT-apdu.h ./ABRT-apdu.h ---- /usr/local/include/osmocom/tcap//ABRT-apdu.h 2010-07-16 12:50:02.000000000 +0200 -+++ ./ABRT-apdu.h 2010-07-09 15:40:49.000000000 +0200 -@@ -12,7 +12,7 @@ - - /* Including external dependencies */ - #include "ABRT-source.h" --#include "EXTERNAL.h" -+#include <EXTERNAL.h> - #include <asn_SEQUENCE_OF.h> - #include <constr_SEQUENCE_OF.h> - #include <constr_SEQUENCE.h> -diff -Nru /usr/local/include/osmocom/tcap//AUDT-apdu.h ./AUDT-apdu.h ---- /usr/local/include/osmocom/tcap//AUDT-apdu.h 2010-07-17 00:09:36.000000000 +0200 -+++ ./AUDT-apdu.h 2010-07-09 15:40:49.000000000 +0200 -@@ -13,7 +13,7 @@ - /* Including external dependencies */ - #include <BIT_STRING.h> - #include <OBJECT_IDENTIFIER.h> --#include "EXTERNAL.h" -+#include <EXTERNAL.h> - #include <asn_SEQUENCE_OF.h> - #include <constr_SEQUENCE_OF.h> - #include <constr_SEQUENCE.h> -@@ -26,16 +26,12 @@ - typedef struct AUDT_apdu { - BIT_STRING_t *protocol_version /* DEFAULT {version1} */; - OBJECT_IDENTIFIER_t application_context_name; --#if 0 - struct user_information { - A_SEQUENCE_OF(EXTERNAL_t) list; - - /* Context for parsing across buffer boundaries */ - asn_struct_ctx_t _asn_ctx; - } *user_information; --#else -- struct user_information *user_information; --#endif - - /* Context for parsing across buffer boundaries */ - asn_struct_ctx_t _asn_ctx; - -diff --git a/src/AARE-apdu.c b/src/AARE-apdu.c -index c560c0e..5d1e35a 100644 ---- a/src/AARE-apdu.c -+++ b/src/AARE-apdu.c -@@ -7,6 +7,7 @@ - #include <asn_internal.h> - - #include "AARE-apdu.h" -+#include "ABRT-apdu.h" - - static asn_TYPE_member_t asn_MBR_user_information_7[] = { - { ATF_POINTER, 0, 0, -@@ -24,8 +25,8 @@ static ber_tlv_tag_t asn_DEF_user_information_tags_7[] = { - (ASN_TAG_CLASS_UNIVERSAL | (16 << 2)) - }; - static asn_SET_OF_specifics_t asn_SPC_user_information_specs_7 = { -- sizeof(struct AARE_user_information), -- offsetof(struct AARE_user_information, _asn_ctx), -+ sizeof(struct user_information), -+ offsetof(struct user_information, _asn_ctx), - 0, /* XER encoding is XMLDelimitedItemList */ - }; - static /* Use -fall-defs-global to expose */ -diff --git a/src/AARQ-apdu.c b/src/AARQ-apdu.c -index f8de6f4..2e5ce4f 100644 ---- a/src/AARQ-apdu.c -+++ b/src/AARQ-apdu.c -@@ -7,6 +7,7 @@ - #include <asn_internal.h> - - #include "AARQ-apdu.h" -+#include "ABRT-apdu.h" - - static asn_TYPE_member_t asn_MBR_user_information_5[] = { - { ATF_POINTER, 0, 0, -@@ -24,8 +25,8 @@ static ber_tlv_tag_t asn_DEF_user_information_tags_5[] = { - (ASN_TAG_CLASS_UNIVERSAL | (16 << 2)) - }; - static asn_SET_OF_specifics_t asn_SPC_user_information_specs_5 = { -- sizeof(struct AARQ_user_information), -- offsetof(struct AARQ_user_information, _asn_ctx), -+ sizeof(struct user_information), -+ offsetof(struct user_information, _asn_ctx), - 0, /* XER encoding is XMLDelimitedItemList */ - }; - static /* Use -fall-defs-global to expose */ -diff --git a/src/AUDT-apdu.c b/src/AUDT-apdu.c -index 1ec47fc..81f63c2 100644 ---- a/src/AUDT-apdu.c -+++ b/src/AUDT-apdu.c -@@ -7,6 +7,7 @@ - #include <asn_internal.h> - - #include "AUDT-apdu.h" -+#include "ABRT-apdu.h" - - static asn_TYPE_member_t asn_MBR_user_information_5[] = { - { ATF_POINTER, 0, 0, -- To view, visit https://gerrit.osmocom.org/c/libosmo-asn1-tcap/+/41582?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: libosmo-asn1-tcap Gerrit-Branch: master Gerrit-Change-Id: I219e17e9439405dc3488b437a234623807f1d8ab Gerrit-Change-Number: 41582 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0

[M] Change in libosmo-asn1-tcap[master]: Remove unneeded asn1c-fixup.patch
by daniel 04 Dec '25

04 Dec '25

Attention is currently required from: pespin. daniel has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/libosmo-asn1-tcap/+/41582?usp=email ) Change subject: Remove unneeded asn1c-fixup.patch ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/libosmo-asn1-tcap/+/41582?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: libosmo-asn1-tcap Gerrit-Branch: master Gerrit-Change-Id: I219e17e9439405dc3488b437a234623807f1d8ab Gerrit-Change-Number: 41582 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Thu, 04 Dec 2025 16:41:55 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[M] Change in libosmo-asn1-tcap[master]: Remove unneeded asn1c-fixup.patch
by pespin 04 Dec '25

04 Dec '25

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-asn1-tcap/+/41582?usp=email ) Change subject: Remove unneeded asn1c-fixup.patch ...................................................................... Remove unneeded asn1c-fixup.patch This is not used nowadays. Change-Id: I219e17e9439405dc3488b437a234623807f1d8ab --- D asn1c-fixup.patch 1 file changed, 0 insertions(+), 163 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-asn1-tcap refs/changes/82/41582/1 diff --git a/asn1c-fixup.patch b/asn1c-fixup.patch deleted file mode 100644 index 1b29076..0000000 --- a/asn1c-fixup.patch +++ /dev/null @@ -1,163 +0,0 @@ -This patch needs to be applied to the asn1c output in order to -make sure we don't re-define the 'user_information' for every different -Dialogue portion PDU type and to make the EXTERNAL magic work -diff -Nru /usr/local/include/osmocom/tcap//AARE-apdu.h ./AARE-apdu.h ---- /usr/local/include/osmocom/tcap//AARE-apdu.h 2010-07-16 12:50:01.000000000 +0200 -+++ ./AARE-apdu.h 2010-07-09 15:40:49.000000000 +0200 -@@ -15,7 +15,7 @@ - #include <OBJECT_IDENTIFIER.h> - #include "Associate-result.h" - #include "Associate-source-diagnostic.h" --#include "EXTERNAL.h" -+#include <EXTERNAL.h> - #include <asn_SEQUENCE_OF.h> - #include <constr_SEQUENCE_OF.h> - #include <constr_SEQUENCE.h> -@@ -35,17 +35,12 @@ - OBJECT_IDENTIFIER_t application_context_name; - Associate_result_t result; - Associate_source_diagnostic_t result_source_diagnostic; --#if 0 - struct AARE_user_information { - A_SEQUENCE_OF(EXTERNAL_t) list; - - /* Context for parsing across buffer boundaries */ - asn_struct_ctx_t _asn_ctx; - } *user_information; --#else -- /* use the definition from ABRT-apdu.h */ -- struct user_information *user_information; --#endif - - /* Context for parsing across buffer boundaries */ - asn_struct_ctx_t _asn_ctx; -diff -Nru /usr/local/include/osmocom/tcap//AARQ-apdu.h ./AARQ-apdu.h ---- /usr/local/include/osmocom/tcap//AARQ-apdu.h 2010-07-16 12:50:00.000000000 +0200 -+++ ./AARQ-apdu.h 2010-07-09 15:40:49.000000000 +0200 -@@ -13,7 +13,7 @@ - /* Including external dependencies */ - #include <BIT_STRING.h> - #include <OBJECT_IDENTIFIER.h> --#include "EXTERNAL.h" -+#include <EXTERNAL.h> - #include <asn_SEQUENCE_OF.h> - #include <constr_SEQUENCE_OF.h> - #include <constr_SEQUENCE.h> -@@ -26,17 +26,12 @@ - typedef struct AARQ_apdu { - BIT_STRING_t *protocol_version /* DEFAULT {version1} */; - OBJECT_IDENTIFIER_t application_context_name; --#if 0 - struct AARQ_user_information { - A_SEQUENCE_OF(EXTERNAL_t) list; - - /* Context for parsing across buffer boundaries */ - asn_struct_ctx_t _asn_ctx; - } *user_information; --#else -- /* use the definition from ABRT-apdu.h */ -- struct user_information *user_information; --#endif - - /* Context for parsing across buffer boundaries */ - asn_struct_ctx_t _asn_ctx; -diff -Nru /usr/local/include/osmocom/tcap//ABRT-apdu.h ./ABRT-apdu.h ---- /usr/local/include/osmocom/tcap//ABRT-apdu.h 2010-07-16 12:50:02.000000000 +0200 -+++ ./ABRT-apdu.h 2010-07-09 15:40:49.000000000 +0200 -@@ -12,7 +12,7 @@ - - /* Including external dependencies */ - #include "ABRT-source.h" --#include "EXTERNAL.h" -+#include <EXTERNAL.h> - #include <asn_SEQUENCE_OF.h> - #include <constr_SEQUENCE_OF.h> - #include <constr_SEQUENCE.h> -diff -Nru /usr/local/include/osmocom/tcap//AUDT-apdu.h ./AUDT-apdu.h ---- /usr/local/include/osmocom/tcap//AUDT-apdu.h 2010-07-17 00:09:36.000000000 +0200 -+++ ./AUDT-apdu.h 2010-07-09 15:40:49.000000000 +0200 -@@ -13,7 +13,7 @@ - /* Including external dependencies */ - #include <BIT_STRING.h> - #include <OBJECT_IDENTIFIER.h> --#include "EXTERNAL.h" -+#include <EXTERNAL.h> - #include <asn_SEQUENCE_OF.h> - #include <constr_SEQUENCE_OF.h> - #include <constr_SEQUENCE.h> -@@ -26,16 +26,12 @@ - typedef struct AUDT_apdu { - BIT_STRING_t *protocol_version /* DEFAULT {version1} */; - OBJECT_IDENTIFIER_t application_context_name; --#if 0 - struct user_information { - A_SEQUENCE_OF(EXTERNAL_t) list; - - /* Context for parsing across buffer boundaries */ - asn_struct_ctx_t _asn_ctx; - } *user_information; --#else -- struct user_information *user_information; --#endif - - /* Context for parsing across buffer boundaries */ - asn_struct_ctx_t _asn_ctx; - -diff --git a/src/AARE-apdu.c b/src/AARE-apdu.c -index c560c0e..5d1e35a 100644 ---- a/src/AARE-apdu.c -+++ b/src/AARE-apdu.c -@@ -7,6 +7,7 @@ - #include <asn_internal.h> - - #include "AARE-apdu.h" -+#include "ABRT-apdu.h" - - static asn_TYPE_member_t asn_MBR_user_information_7[] = { - { ATF_POINTER, 0, 0, -@@ -24,8 +25,8 @@ static ber_tlv_tag_t asn_DEF_user_information_tags_7[] = { - (ASN_TAG_CLASS_UNIVERSAL | (16 << 2)) - }; - static asn_SET_OF_specifics_t asn_SPC_user_information_specs_7 = { -- sizeof(struct AARE_user_information), -- offsetof(struct AARE_user_information, _asn_ctx), -+ sizeof(struct user_information), -+ offsetof(struct user_information, _asn_ctx), - 0, /* XER encoding is XMLDelimitedItemList */ - }; - static /* Use -fall-defs-global to expose */ -diff --git a/src/AARQ-apdu.c b/src/AARQ-apdu.c -index f8de6f4..2e5ce4f 100644 ---- a/src/AARQ-apdu.c -+++ b/src/AARQ-apdu.c -@@ -7,6 +7,7 @@ - #include <asn_internal.h> - - #include "AARQ-apdu.h" -+#include "ABRT-apdu.h" - - static asn_TYPE_member_t asn_MBR_user_information_5[] = { - { ATF_POINTER, 0, 0, -@@ -24,8 +25,8 @@ static ber_tlv_tag_t asn_DEF_user_information_tags_5[] = { - (ASN_TAG_CLASS_UNIVERSAL | (16 << 2)) - }; - static asn_SET_OF_specifics_t asn_SPC_user_information_specs_5 = { -- sizeof(struct AARQ_user_information), -- offsetof(struct AARQ_user_information, _asn_ctx), -+ sizeof(struct user_information), -+ offsetof(struct user_information, _asn_ctx), - 0, /* XER encoding is XMLDelimitedItemList */ - }; - static /* Use -fall-defs-global to expose */ -diff --git a/src/AUDT-apdu.c b/src/AUDT-apdu.c -index 1ec47fc..81f63c2 100644 ---- a/src/AUDT-apdu.c -+++ b/src/AUDT-apdu.c -@@ -7,6 +7,7 @@ - #include <asn_internal.h> - - #include "AUDT-apdu.h" -+#include "ABRT-apdu.h" - - static asn_TYPE_member_t asn_MBR_user_information_5[] = { - { ATF_POINTER, 0, 0, -- To view, visit https://gerrit.osmocom.org/c/libosmo-asn1-tcap/+/41582?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: libosmo-asn1-tcap Gerrit-Branch: master Gerrit-Change-Id: I219e17e9439405dc3488b437a234623807f1d8ab Gerrit-Change-Number: 41582 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de>

1 0

[XS] Change in pysim[master]: ts_102_221: EF.ARR: fix read_arr_record
by lynxis lazus 04 Dec '25

04 Dec '25

lynxis lazus has uploaded this change for review. ( https://gerrit.osmocom.org/c/pysim/+/41580?usp=email ) Change subject: ts_102_221: EF.ARR: fix read_arr_record ...................................................................... ts_102_221: EF.ARR: fix read_arr_record `read_arr_record 1` failed with an AttributeError exception because RECORD_NR must be all caps. Change-Id: If44f9f2271293d3063f6c527e5a68dcfaeb5942e --- M pySim/ts_102_221.py 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/80/41580/1 diff --git a/pySim/ts_102_221.py b/pySim/ts_102_221.py index d97b3cf..e38483d 100644 --- a/pySim/ts_102_221.py +++ b/pySim/ts_102_221.py @@ -750,7 +750,7 @@ @cmd2.with_argparser(LinFixedEF.ShellCommands.read_rec_dec_parser) def do_read_arr_record(self, opts): """Read one EF.ARR record in flattened, human-friendly form.""" - (data, _sw) = self._cmd.lchan.read_record_dec(opts.record_nr) + (data, _sw) = self._cmd.lchan.read_record_dec(opts.RECORD_NR) data = self._cmd.lchan.selected_file.flatten(data) self._cmd.poutput_json(data, opts.oneline) -- To view, visit https://gerrit.osmocom.org/c/pysim/+/41580?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: If44f9f2271293d3063f6c527e5a68dcfaeb5942e Gerrit-Change-Number: 41580 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>

1 0

[XS] Change in pysim[master]: ts_31_102: EF SUCI_Calc_Info: fix decoding empty files
by lynxis lazus 04 Dec '25

04 Dec '25

lynxis lazus has uploaded this change for review. ( https://gerrit.osmocom.org/c/pysim/+/41581?usp=email ) Change subject: ts_31_102: EF SUCI_Calc_Info: fix decoding empty files ...................................................................... ts_31_102: EF SUCI_Calc_Info: fix decoding empty files When trying to use `edit_binary_decoded` with an empty file, pysim runs into a len(None) exception, because hpkl.to_dict()['hnet_pubkey_list'] returns None. Can reproduced with a CCC Camp 2023 usim and editing the file. Change-Id: Ib8e322e65dd768bfd49e7a5620a2163f12a74ec7 --- M pySim/ts_31_102.py 1 file changed, 4 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/81/41581/1 diff --git a/pySim/ts_31_102.py b/pySim/ts_31_102.py index 4ee6621..1f46241 100644 --- a/pySim/ts_31_102.py +++ b/pySim/ts_31_102.py @@ -389,7 +389,10 @@ # remaining data holds Home Network Public Key Data Object hpkl = EF_SUCI_Calc_Info.HnetPubkeyList() hpkl.from_tlv(in_bytes[pos:]) - hnet_pubkey_list = self._compact_pubkey_list(hpkl.to_dict()['hnet_pubkey_list']) + + hnet_pubkey_list = [] + if hpkl.to_dict()['hnet_pubkey_list']: + hnet_pubkey_list = self._compact_pubkey_list(hpkl.to_dict()['hnet_pubkey_list']) return { 'prot_scheme_id_list': prot_scheme_id_list, -- To view, visit https://gerrit.osmocom.org/c/pysim/+/41581?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Ib8e322e65dd768bfd49e7a5620a2163f12a74ec7 Gerrit-Change-Number: 41581 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>

1 0

[XS] Change in pysim[master]: pySim.runtime: Fix file selection by upper case hex FID
by laforge 04 Dec '25

04 Dec '25

laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/pysim/+/41579?usp=email ) Change subject: pySim.runtime: Fix file selection by upper case hex FID ...................................................................... pySim.runtime: Fix file selection by upper case hex FID When trying to remove a file (e.g. DF.5G_ProSe, 5FF0), there seems to be a case sensitive check when checking for the dict: pySim/runtime.py: get_file_for_filename(): 478 def get_file_for_filename(self, name: str): 479 """Get the related CardFile object for a specified filename.""" 480 sels = self.selected_file.get_selectables() 481 return sels[name] The dict sels contains 5ff0, but not 5FF0. The type of argument name is str. So a case sensitive check will be used. Change-Id: Idd0db1f4bbd3ee9eec20f5fd0f4371c2882950cd Closes: OS#6898 --- M pySim/runtime.py 1 file changed, 4 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/79/41579/1 diff --git a/pySim/runtime.py b/pySim/runtime.py index 597323f..35452a0 100644 --- a/pySim/runtime.py +++ b/pySim/runtime.py @@ -477,11 +477,15 @@ def get_file_for_filename(self, name: str): """Get the related CardFile object for a specified filename.""" + if is_hex(name): + name = name.lower() sels = self.selected_file.get_selectables() return sels[name] def activate_file(self, name: str): """Request ACTIVATE FILE of specified file.""" + if is_hex(name): + name = name.lower() sels = self.selected_file.get_selectables() f = sels[name] data, sw = self.scc.activate_file(f.fid) -- To view, visit https://gerrit.osmocom.org/c/pysim/+/41579?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Idd0db1f4bbd3ee9eec20f5fd0f4371c2882950cd Gerrit-Change-Number: 41579 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org>

1 0

[XS] Change in pysim[master]: ts_102_222: correct encode FILE_ID
by fixeria 04 Dec '25

04 Dec '25

Attention is currently required from: lynxis lazus. fixeria has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/pysim/+/41578?usp=email ) Change subject: ts_102_222: correct encode FILE_ID ...................................................................... Patch Set 1: (1 comment) File pySim/ts_102_222.py: https://gerrit.osmocom.org/c/pysim/+/41578/comment/945fc85e_c54c652f?usp=em… : PS1, Line 148: to_bytes > There're other instances of `FileIdentifier` in this file: […] No longer relevant, I see. -- To view, visit https://gerrit.osmocom.org/c/pysim/+/41578?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I95c81f6793f1fdbaee5365355bab06a1c1107106 Gerrit-Change-Number: 41578 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu> Gerrit-CC: Jenkins Builder Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu> Gerrit-Comment-Date: Thu, 04 Dec 2025 13:49:33 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>

1 0

[XS] Change in osmo-sgsn[pespin/rel-1.13.0]: mmctx: LOGIUP: allow ue to be NULL
by pespin 04 Dec '25

04 Dec '25

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/41576?usp=email ) Change subject: mmctx: LOGIUP: allow ue to be NULL ...................................................................... mmctx: LOGIUP: allow ue to be NULL In certain cases, the ue can be NULL. E.g. when receiving a new Iu event without a context. Change-Id: Ie569fa100abe518cb63c1b918ad7748a7ea3bb99 (cherry picked from commit 234da0eed4b6e5d372c34dc496677463650f3c97) --- M include/osmocom/sgsn/mmctx.h 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: osmith: Looks good to me, but someone else must approve fixeria: Looks good to me, approved Jenkins Builder: Verified diff --git a/include/osmocom/sgsn/mmctx.h b/include/osmocom/sgsn/mmctx.h index 03bb845..8502bdf 100644 --- a/include/osmocom/sgsn/mmctx.h +++ b/include/osmocom/sgsn/mmctx.h @@ -231,7 +231,8 @@ #ifdef BUILD_IU #define LOGIUP(ue, level, fmt, args...) \ - LOGP(DMM, level, "UE(0x%x){%s} " fmt, ue->conn_id, osmo_rai_name(&(ue)->ra_id), ## args) + LOGP(DMM, level, "UE(0x%x){%s} " fmt, (ue) ? (ue)->conn_id : 0xffffffff, \ + (ue) ? osmo_rai_name(&(ue)->ra_id) : "", ## args) #else #define LOGIUP(ue, level, fmt, args...) \ LOGP(DMM, level, "UE(%p){NOTSUPPORTED} " fmt, ue, ## args) -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/41576?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-sgsn Gerrit-Branch: pespin/rel-1.13.0 Gerrit-Change-Id: Ie569fa100abe518cb63c1b918ad7748a7ea3bb99 Gerrit-Change-Number: 41576 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: lynxis lazus <lynxis(a)fe80.eu>

1 0

[XS] Change in osmo-sgsn[pespin/rel-1.13.0]: gprs_gmm: fix endianess of GSM48_IE_GMM_PDP_CTX_STATUS
by pespin 04 Dec '25

04 Dec '25

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/41575?usp=email ) Change subject: gprs_gmm: fix endianess of GSM48_IE_GMM_PDP_CTX_STATUS ...................................................................... gprs_gmm: fix endianess of GSM48_IE_GMM_PDP_CTX_STATUS The pdp_status is actually encoded as little endian, but there is no tlvp_val16le yet. Change-Id: Ib9bee2f8d0b1f89986d15cf3ce6404ad76378c8c (cherry picked from commit b79797d0568ffd757540f6307d538135bdfa01b8) --- M src/sgsn/gprs_gmm.c 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified fixeria: Looks good to me, approved osmith: Looks good to me, but someone else must approve diff --git a/src/sgsn/gprs_gmm.c b/src/sgsn/gprs_gmm.c index 7a4ea20..d70ceec 100644 --- a/src/sgsn/gprs_gmm.c +++ b/src/sgsn/gprs_gmm.c @@ -2009,7 +2009,8 @@ /* Look at PDP Context Status IE and see if MS's view of * activated/deactivated NSAPIs agrees with our view */ if (TLVP_PRESENT(&tp, GSM48_IE_GMM_PDP_CTX_STATUS)) { - uint16_t pdp_status = tlvp_val16be(&tp, GSM48_IE_GMM_PDP_CTX_STATUS); + /* TODO: Use tlvp_val16le when available */ + uint16_t pdp_status = osmo_load16le(TLVP_VAL(&tp, GSM48_IE_GMM_PDP_CTX_STATUS)); process_ms_ctx_status(ctx, pdp_status); -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/41575?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-sgsn Gerrit-Branch: pespin/rel-1.13.0 Gerrit-Change-Id: Ib9bee2f8d0b1f89986d15cf3ce6404ad76378c8c Gerrit-Change-Number: 41575 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: lynxis lazus <lynxis(a)fe80.eu>

1 0

[XS] Change in osmo-sgsn[pespin/rel-1.13.0]: Bump version: 1.13.0.2-dea2 → 1.13.1
by pespin 04 Dec '25

04 Dec '25

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/41577?usp=email ) Change subject: Bump version: 1.13.0.2-dea2 → 1.13.1 ...................................................................... Bump version: 1.13.0.2-dea2 → 1.13.1 Change-Id: I016f0d2d169cf0a8a9638a19531289a91a2bc8f3 --- M debian/changelog 1 file changed, 8 insertions(+), 0 deletions(-) Approvals: fixeria: Looks good to me, but someone else must approve osmith: Looks good to me, approved Jenkins Builder: Verified diff --git a/debian/changelog b/debian/changelog index 7bc1351..c769bf4 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,11 @@ +osmo-sgsn (1.13.1) unstable; urgency=medium + + [ Alexander Couzens ] + * gprs_gmm: fix endianess of GSM48_IE_GMM_PDP_CTX_STATUS + * mmctx: LOGIUP: allow ue to be NULL + + -- Pau Espin Pedrol <pespin(a)sysmocom.de> Thu, 04 Dec 2025 14:32:14 +0100 + osmo-sgsn (1.13.0) unstable; urgency=medium [ Pau Espin Pedrol ] -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/41577?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-sgsn Gerrit-Branch: pespin/rel-1.13.0 Gerrit-Change-Id: I016f0d2d169cf0a8a9638a19531289a91a2bc8f3 Gerrit-Change-Number: 41577 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

1 0