February 2022 - gerrit-log - lists.osmocom.org

Change in pysim[master]: ts_31_102: Use perror() instead of poutput() for errors

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/27179 ) Change subject: ts_31_102: Use perror() instead of poutput() for errors ...................................................................... ts_31_102: Use perror() instead of poutput() for errors This adds colorization and ensures they go to stderr and not stdout Change-Id: I34b8f974b4ff13002679c4700bdf604db7d7f3cd --- M pySim/ts_31_102.py 1 file changed, 5 insertions(+), 5 deletions(-) Approvals: Jenkins Builder: Verified fixeria: Looks good to me, approved diff --git a/pySim/ts_31_102.py b/pySim/ts_31_102.py index ce32b20..7fcb08e 100644 --- a/pySim/ts_31_102.py +++ b/pySim/ts_31_102.py @@ -620,9 +620,9 @@ if exists != should_exist: num_problems += 1 if exists: - cmd.poutput(" ERROR: File %s is selectable but should not!" % f) + cmd.perror(" ERROR: File %s is selectable but should not!" % f) else: - cmd.poutput(" ERROR: File %s is not selectable (%s) but should!" % (f, sw)) + cmd.perror(" ERROR: File %s is not selectable (%s) but should!" % (f, sw)) finally: # re-select the EF.UST cmd.card._scc.select_file(self.fid) @@ -655,11 +655,11 @@ active_services = selected_file.get_active_services(self._cmd) # Service n°46 can only be declared "available" if service n°45 is declared "available" if 46 in active_services and not 45 in active_services: - self._cmd.poutput("ERROR: Service 46 available, but it requires Service 45") + self._cmd.perror("ERROR: Service 46 available, but it requires Service 45") num_problems += 1 # Service n°125 shall only be taken into account if Service n°124 is declared "available" if 125 in active_services and not 124 in active_services: - self._cmd.poutput("ERROR: Service 125 is ignored as Service 124 not available") + self._cmd.perror("ERROR: Service 125 is ignored as Service 124 not available") num_problems += 1 # Service n°95, n°99 and n°115 shall not be declared "available" if an ISIM application is present on the UICC non_isim_services = [95, 99, 115] @@ -667,7 +667,7 @@ if 'ADF.ISIM' in app_names: for s in non_isim_services: if s in active_services: - self._cmd.poutput("ERROR: Service %u shall not be available as ISIM application is present" % s) + self._cmd.perror("ERROR: Service %u shall not be available as ISIM application is present" % s) num_problems += 1 self._cmd.poutput("===> %u service / file inconsistencies detected" % num_problems) 1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. -- To view, visit https://gerrit.osmocom.org/c/pysim/+/27179 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I34b8f974b4ff13002679c4700bdf604db7d7f3cd Gerrit-Change-Number: 27179 Gerrit-PatchSet: 3 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-MessageType: merged

3 years, 4 months

1
0
0 0

Change in pysim[master]: ts_31_102: Add "ust_service_check" command.

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/27175 ) Change subject: ts_31_102: Add "ust_service_check" command. ...................................................................... ts_31_102: Add "ust_service_check" command. This command performs a consistency check between the services activated in EF.UST/EF.IST and the files that should (or should not) be active/selectable for the given service. Produces output like: Checking service No 48 (inactive) ERROR: File EF(EF.MWIS) is selectable but should not! Checking service No 49 (active) ERROR: File EF(EF.CFIS) is not selectable (SW=6a82) but should! Change-Id: Iea7166959e2015eb8fa34d86036560c9e42ce4d3 --- M pySim/ts_31_102.py 1 file changed, 36 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved fixeria: Looks good to me, but someone else must approve diff --git a/pySim/ts_31_102.py b/pySim/ts_31_102.py index 17d865d..9105418 100644 --- a/pySim/ts_31_102.py +++ b/pySim/ts_31_102.py @@ -600,6 +600,42 @@ """Deactivate a service within EF.UST""" self._cmd.card.update_ust(int(arg), 0) + def do_ust_service_check(self, arg): + """Check consistency between services of this file and files present/activated""" + # obtain list of currently active services + (service_data, sw) = self._cmd.rs.read_binary_dec() + for s in service_data.keys(): + if service_data[s]['activated']: + active_services.append(s) + # iterate over all the service-constraints we know of + selected_file = self._cmd.rs.selected_file + files_by_service = selected_file.parent.files_by_service + try: + for s in sorted(files_by_service.keys()): + active_str = 'active' if s in active_services else 'inactive' + self._cmd.poutput("Checking service No %u (%s)" % (s, active_str)) + for f in files_by_service[s]: + should_exist = f.should_exist_for_services(active_services) + try: + (data, sw) = self._cmd.card._scc.select_file(f.fid) + exists = True + fcp = f.decode_select_response(data) + # if we just selected a directory, go back + if fcp['file_descriptor']['file_type'] == 'df': + self._cmd.card._scc.select_parent_df() + except SwMatchError as e: + sw = e.sw_actual + exists = False + if exists != should_exist: + if exists: + self._cmd.poutput(" ERROR: File %s is selectable but should not!" % f) + else: + self._cmd.poutput(" ERROR: File %s is not selectable (SW=%s) but should!" % (f, sw)) + finally: + # re-select the EF.UST + self._cmd.card._scc.select_file(selected_file.fid) + + # TS 31.103 Section 4.2.7 - *not* the same as DF.GSM/EF.ECC! class EF_ECC(LinFixedEF): cc_construct = Rpad(BcdAdapter(Rpad(Bytes(3))), pattern='f') -- To view, visit https://gerrit.osmocom.org/c/pysim/+/27175 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Iea7166959e2015eb8fa34d86036560c9e42ce4d3 Gerrit-Change-Number: 27175 Gerrit-PatchSet: 4 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-MessageType: merged

3 years, 4 months

1
0
0 0

Change in pysim[master]: Split EF.UST handling from EF.IST and EF.SST

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/27176 ) Change subject: Split EF.UST handling from EF.IST and EF.SST ...................................................................... Split EF.UST handling from EF.IST and EF.SST The existing code had the following serious problems: * when trying to update EF.SST or EF.IST, it would write to EF.UST ! * shell commands were called ust_* even for the EST/IST files Let's introduce the proper separation between what is shared and what is file-specific. Change-Id: Ie55669ca37a4762fac9f71b1db528ca67056e8dd --- M pySim/cards.py M pySim/ts_31_102.py M pySim/ts_31_103.py 3 files changed, 114 insertions(+), 39 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved fixeria: Looks good to me, but someone else must approve diff --git a/pySim/cards.py b/pySim/cards.py index c3766e9..1ed1634 100644 --- a/pySim/cards.py +++ b/pySim/cards.py @@ -425,6 +425,15 @@ EF_USIM_ADF_map['UST'], content) return sw + def update_est(self, service, bit=1): + (res, sw) = self._scc.read_binary(EF_USIM_ADF_map['EST']) + if sw == '9000': + content = enc_st(res, service, bit) + (res, sw) = self._scc.update_binary( + EF_USIM_ADF_map['EST'], content) + return sw + + class IsimCard(SimCard): @@ -566,6 +575,14 @@ sw) return uiari_recs + def update_ist(self, service, bit=1): + (res, sw) = self._scc.read_binary(EF_ISIM_ADF_map['IST']) + if sw == '9000': + content = enc_st(res, service, bit) + (res, sw) = self._scc.update_binary( + EF_ISIM_ADF_map['IST'], content) + return sw + class MagicSimBase(abc.ABC, SimCard): """ diff --git a/pySim/ts_31_102.py b/pySim/ts_31_102.py index 9105418..8e2d1c8 100644 --- a/pySim/ts_31_102.py +++ b/pySim/ts_31_102.py @@ -542,8 +542,6 @@ def __init__(self, fid, sfid, name, desc, size, table, **kwargs): super().__init__(fid=fid, sfid=sfid, name=name, desc=desc, size=size, **kwargs) self.table = table - # add those commands to the general commands of a TransparentEF - self.shell_commands += [self.AddlShellCommands()] @staticmethod def _bit_byte_offset_for_service(service: int) -> Tuple[int, int]: @@ -587,6 +585,47 @@ out[byte_offset] |= (bit) << bit_offset return out + def ust_service_check(self, cmd): + """Check consistency between services of this file and files present/activated""" + # obtain list of currently active services + (service_data, sw) = cmd.rs.read_binary_dec() + active_services = [] + for s in service_data.keys(): + if service_data[s]['activated']: + active_services.append(s) + # iterate over all the service-constraints we know of + files_by_service = self.parent.files_by_service + try: + for s in sorted(files_by_service.keys()): + active_str = 'active' if s in active_services else 'inactive' + cmd.poutput("Checking service No %u (%s)" % (s, active_str)) + for f in files_by_service[s]: + should_exist = f.should_exist_for_services(active_services) + try: + (data, sw) = cmd.card._scc.select_file(f.fid) + exists = True + fcp = f.decode_select_response(data) + # if we just selected a directory, go back + if fcp['file_descriptor']['file_type'] == 'df': + cmd.card._scc.select_parent_df() + except SwMatchError as e: + sw = str(e) + exists = False + if exists != should_exist: + if exists: + cmd.poutput(" ERROR: File %s is selectable but should not!" % f) + else: + cmd.poutput(" ERROR: File %s is not selectable (%s) but should!" % (f, sw)) + finally: + # re-select the EF.UST + cmd.card._scc.select_file(self.fid) + +class EF_UST(EF_UServiceTable): + def __init__(self, **kwargs): + super().__init__(fid='6f38', sfid=0x04, name='EF.UST', desc='USIM Service Table', size={1,17}, table=EF_UST_map, **kwargs) + # add those commands to the general commands of a TransparentEF + self.shell_commands += [self.AddlShellCommands()] + @with_default_category('File-Specific Commands') class AddlShellCommands(CommandSet): def __init__(self): @@ -602,38 +641,8 @@ def do_ust_service_check(self, arg): """Check consistency between services of this file and files present/activated""" - # obtain list of currently active services - (service_data, sw) = self._cmd.rs.read_binary_dec() - for s in service_data.keys(): - if service_data[s]['activated']: - active_services.append(s) - # iterate over all the service-constraints we know of selected_file = self._cmd.rs.selected_file - files_by_service = selected_file.parent.files_by_service - try: - for s in sorted(files_by_service.keys()): - active_str = 'active' if s in active_services else 'inactive' - self._cmd.poutput("Checking service No %u (%s)" % (s, active_str)) - for f in files_by_service[s]: - should_exist = f.should_exist_for_services(active_services) - try: - (data, sw) = self._cmd.card._scc.select_file(f.fid) - exists = True - fcp = f.decode_select_response(data) - # if we just selected a directory, go back - if fcp['file_descriptor']['file_type'] == 'df': - self._cmd.card._scc.select_parent_df() - except SwMatchError as e: - sw = e.sw_actual - exists = False - if exists != should_exist: - if exists: - self._cmd.poutput(" ERROR: File %s is selectable but should not!" % f) - else: - self._cmd.poutput(" ERROR: File %s is not selectable (SW=%s) but should!" % (f, sw)) - finally: - # re-select the EF.UST - self._cmd.card._scc.select_file(selected_file.fid) + selected_file.ust_service_check(self._cmd) # TS 31.103 Section 4.2.7 - *not* the same as DF.GSM/EF.ECC! @@ -750,6 +759,33 @@ def __init__(self, fid='6f4f', sfid=0x16, name='EF.CCP2', desc='Capability Configuration Parameters 2', **kwargs): super().__init__(fid=fid, sfid=sfid, name=name, desc=desc, rec_len={15, None}, **kwargs) +# TS 31.102 Section 4.2.47 +class EF_EST(EF_UServiceTable): + def __init__(self, **kwargs): + super().__init__(fid='6f56', sfid=0x05, name='EF.EST', desc='Enabled Services Table', size={1,None}, table=EF_EST_map, **kwargs) + # add those commands to the general commands of a TransparentEF + self.shell_commands += [self.AddlShellCommands()] + + @with_default_category('File-Specific Commands') + class AddlShellCommands(CommandSet): + def __init__(self): + super().__init__() + + def do_est_service_activate(self, arg): + """Activate a service within EF.UST""" + self._cmd.card.update_est(int(arg), 1) + + def do_est_service_deactivate(self, arg): + """Deactivate a service within EF.UST""" + self._cmd.card.update_est(int(arg), 0) + + def do_est_service_check(self, arg): + """Check consistency between services of this file and files present/activated""" + # obtain list of currently active services + (service_data, sw) = self._cmd.rs.read_binary_dec() + active_services = service_data.keys() + + # TS 31.102 Section 4.2.48 class EF_ACL(TransparentEF): def __init__(self, fid='6f57', sfid=None, name='EF.ACL', size={32, None}, @@ -1128,8 +1164,7 @@ 'User controlled PLMN Selector with Access Technology', service=20), EF_HPPLMN(), EF_ACMmax(service=13), - EF_UServiceTable('6f38', 0x04, 'EF.UST', 'USIM Service Table', size={ - 1, 17}, table=EF_UST_map), + EF_UST(), CyclicEF('6f39', None, 'EF.ACM', 'Accumulated call meter', rec_len={3, 3}, service=13), TransparentEF('6f3e', None, 'EF.GID1', 'Group Identifier Level 1', service=17), @@ -1168,8 +1203,7 @@ EF_ADN('6f4d', None, 'EF.BDN', 'Barred Dialling Numbers', service=6), EF_EXT('6f55', None, 'EF.EXT4', 'Extension4 (BDN/SSC)', service=7), EF_CMI(service=6), - EF_UServiceTable('6f56', 0x05, 'EF.EST', 'Enabled Services Table', size={ - 1, None}, table=EF_EST_map, service=[2, 6, 34, 35]), + EF_EST(service=[2, 6, 34, 35]), EF_ACL(service=35), EF_DCK(service=36), EF_CNL(service=37), diff --git a/pySim/ts_31_103.py b/pySim/ts_31_103.py index 7d32409..f04f480 100644 --- a/pySim/ts_31_103.py +++ b/pySim/ts_31_103.py @@ -104,6 +104,31 @@ super().__init__(fid=fid, sfid=sfid, name=name, desc=desc, **kwargs) self._tlv = EF_IMPU.impu +# TS 31.103 Section 4.2.7 +class EF_IST(EF_UServiceTable): + def __init__(self, **kwargs): + super().__init__('6f07', 0x07, 'EF.IST', 'ISIM Service Table', {1, None}, EF_IST_map) + # add those commands to the general commands of a TransparentEF + self.shell_commands += [self.AddlShellCommands()] + + @with_default_category('File-Specific Commands') + class AddlShellCommands(CommandSet): + def __init__(self): + super().__init__() + + def do_ist_service_activate(self, arg): + """Activate a service within EF.IST""" + self._cmd.card.update_ist(int(arg), 1) + + def do_ist_service_deactivate(self, arg): + """Deactivate a service within EF.IST""" + self._cmd.card.update_ist(int(arg), 0) + + def do_ist_service_check(self, arg): + """Check consistency between services of this file and files present/activated""" + selected_file = self._cmd.rs.selected_file + selected_file.ust_service_check(self._cmd) + # TS 31.103 Section 4.2.8 class EF_PCSCF(LinFixedEF): def __init__(self, fid='6f09', sfid=None, name='EF.P-CSCF', desc='P-CSCF Address', **kwargs): @@ -179,8 +204,7 @@ EF_IMPU(), EF_AD(), EF_ARR('6f06', 0x06), - EF_UServiceTable('6f07', 0x07, 'EF.IST', - 'ISIM Service Table', {1, None}, EF_IST_map), + EF_IST(), EF_PCSCF(service=5), EF_GBABP(service=2), EF_GBANL(service=2), -- To view, visit https://gerrit.osmocom.org/c/pysim/+/27176 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Ie55669ca37a4762fac9f71b1db528ca67056e8dd Gerrit-Change-Number: 27176 Gerrit-PatchSet: 3 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-MessageType: merged

3 years, 4 months

1
0
0 0

Change in pysim[master]: ts_51_011: Proper decode of EF.SMSP

by laforge

Attention is currently required from: dexter. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/27183 ) Change subject: ts_51_011: Proper decode of EF.SMSP ...................................................................... Patch Set 4: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/27183 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Iac5bb87ed3350978dc8b207f052510fdba2e4883 Gerrit-Change-Number: 27183 Gerrit-PatchSet: 4 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 17 Feb 2022 11:41:17 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

3 years, 4 months

1
0
0 0

Change in pysim[master]: ust_service_check: proper treatment of files in sub-directories

by laforge

Attention is currently required from: fixeria, dexter. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/27181 ) Change subject: ust_service_check: proper treatment of files in sub-directories ...................................................................... Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/27181 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I61797fefa9dafa36a8a62c11aa2cfaeecb015740 Gerrit-Change-Number: 27181 Gerrit-PatchSet: 3 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 17 Feb 2022 11:41:11 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

3 years, 4 months

1
0
0 0

Change in pysim[master]: ts_31_102: Add more EF.UST checks to 'ust_service_check' command

by laforge

Attention is currently required from: dexter. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/27178 ) Change subject: ts_31_102: Add more EF.UST checks to 'ust_service_check' command ...................................................................... Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/27178 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Id47f8f2c8de299bbf91243d0c8900d22a7d35b10 Gerrit-Change-Number: 27178 Gerrit-PatchSet: 3 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 17 Feb 2022 11:41:00 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

3 years, 4 months

1
0
0 0

Change in pysim[master]: Split EF.UST handling from EF.IST and EF.SST

by laforge

Attention is currently required from: dexter. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/27176 ) Change subject: Split EF.UST handling from EF.IST and EF.SST ...................................................................... Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/27176 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Ie55669ca37a4762fac9f71b1db528ca67056e8dd Gerrit-Change-Number: 27176 Gerrit-PatchSet: 3 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 17 Feb 2022 11:40:51 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

3 years, 4 months

1
0
0 0

Change in pysim[master]: ts_31_102: Add "ust_service_check" command.

by laforge

Attention is currently required from: dexter. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/27175 ) Change subject: ts_31_102: Add "ust_service_check" command. ...................................................................... Patch Set 4: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/27175 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Iea7166959e2015eb8fa34d86036560c9e42ce4d3 Gerrit-Change-Number: 27175 Gerrit-PatchSet: 4 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 17 Feb 2022 11:40:47 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

3 years, 4 months

1
0
0 0

Change in osmo-hnbgw[master]: ranap_rab_ass_test: cosmetic: correct test function names

by dexter

Attention is currently required from: dexter. Hello Jenkins Builder, laforge, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/27200 to look at the new patch set (#3). Change subject: ranap_rab_ass_test: cosmetic: correct test function names ...................................................................... ranap_rab_ass_test: cosmetic: correct test function names The test function naming follows an older scheme, lets use the current scheme. Change-Id: Ib9db9d86e01551c8d9d8f8c4933025ca20ce5624 Related: OS#5152 --- M tests/ranap_rab_ass/ranap_rab_ass_test.c 1 file changed, 8 insertions(+), 8 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-hnbgw refs/changes/00/27200/3 -- To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/27200 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-hnbgw Gerrit-Branch: master Gerrit-Change-Id: Ib9db9d86e01551c8d9d8f8c4933025ca20ce5624 Gerrit-Change-Number: 27200 Gerrit-PatchSet: 3 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-MessageType: newpatchset

3 years, 4 months

1
0
0 0

Change in osmo-ttcn3-hacks[master]: bsc: add TC_ho_into_this_bsc_a5_1_3_no_chosen_enc_alg

by fixeria

Attention is currently required from: neels. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27233 ) Change subject: bsc: add TC_ho_into_this_bsc_a5_1_3_no_chosen_enc_alg ...................................................................... Patch Set 3: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27233 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: Ia94cc29bf66339ed782cb5101f3640f69cf73131 Gerrit-Change-Number: 27233 Gerrit-PatchSet: 3 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Comment-Date: Thu, 17 Feb 2022 10:07:57 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

3 years, 4 months

1
0
0 0

2025

2024

2023

2022

gerrit-log February 2022