September 2024 - gerrit-log - lists.osmocom.org

[XS] Change in pysim[master]: ts_102_221: se _test_de_encode instead of _test_decode in EF.DIR unit...
by laforge 30 Sep '24

30 Sep '24

laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/38201?usp=email ) Change subject: ts_102_221: se _test_de_encode instead of _test_decode in EF.DIR unittest ...................................................................... ts_102_221: se _test_de_encode instead of _test_decode in EF.DIR unittest The unittest for EF.DIR only runs with _test_decode, but it also runs with _test_de_encode without any problems Related: OS#5714 Change-Id: If459073c6ff927c1cc1790d506e3979243b1fb4c --- M pySim/ts_102_221.py 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/pySim/ts_102_221.py b/pySim/ts_102_221.py index fa1bd39..5508be7 100644 --- a/pySim/ts_102_221.py +++ b/pySim/ts_102_221.py @@ -637,8 +637,7 @@ # TS 102 221 Section 13.1 class EF_DIR(LinFixedEF): - # FIXME: re-encode failure when changing to _test_de_encode - _test_decode = [ + _test_de_encode = [ ( '61294f10a0000000871002ffffffff890709000050055553696d31730ea00c80011781025f608203454150', { "application_template": [ { "application_id": h2b("a0000000871002ffffffff8907090000") }, { "application_label": "USim1" }, -- To view, visit https://gerrit.osmocom.org/c/pysim/+/38201?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: If459073c6ff927c1cc1790d506e3979243b1fb4c Gerrit-Change-Number: 38201 Gerrit-PatchSet: 8 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org>

1 0

[M] Change in pysim[master]: utils: move enc_msisdn and dec_msisdn to legacy/utils.py
by laforge 30 Sep '24

30 Sep '24

laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/38118?usp=email ) Change subject: utils: move enc_msisdn and dec_msisdn to legacy/utils.py ...................................................................... utils: move enc_msisdn and dec_msisdn to legacy/utils.py We now have a construct based encoder/decoder for the record content of EF.MSISDN. This means that we do not need the functions enc_msisdn and dec_msisdn in the non-legacy code anymore. We can now move both to legacy/utils.py. Related: OS#5714 Change-Id: I19ec8ba14551ec282fc0cc12ae2f6d528bdfc527 --- M pySim-read.py M pySim/legacy/cards.py M pySim/legacy/utils.py M pySim/utils.py M tests/unittests/test_utils.py 5 files changed, 97 insertions(+), 96 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/pySim-read.py b/pySim-read.py index aae5ad4..4ede6dd 100755 --- a/pySim-read.py +++ b/pySim-read.py @@ -42,8 +42,8 @@ from pySim.transport import init_reader, argparse_add_reader_args from pySim.exceptions import SwMatchError from pySim.legacy.cards import card_detect, SimCard, UsimCard, IsimCard -from pySim.utils import dec_imsi, dec_iccid, dec_msisdn -from pySim.legacy.utils import format_xplmn_w_act, dec_st +from pySim.utils import dec_imsi, dec_iccid +from pySim.legacy.utils import format_xplmn_w_act, dec_st, dec_msisdn option_parser = argparse.ArgumentParser(description='Legacy tool for reading some parts of a SIM card', formatter_class=argparse.ArgumentDefaultsHelpFormatter) diff --git a/pySim/legacy/cards.py b/pySim/legacy/cards.py index 5e64e95..1f5db8a 100644 --- a/pySim/legacy/cards.py +++ b/pySim/legacy/cards.py @@ -7,11 +7,11 @@ from pytlv.TLV import * from pySim.cards import SimCardBase, UiccCardBase -from pySim.utils import dec_iccid, enc_iccid, dec_imsi, enc_imsi, dec_msisdn, enc_msisdn +from pySim.utils import dec_iccid, enc_iccid, dec_imsi, enc_imsi from pySim.utils import enc_plmn, get_addr_type from pySim.utils import is_hex, h2b, b2h, h2s, s2h, lpad, rpad from pySim.legacy.utils import enc_ePDGSelection, format_xplmn_w_act, format_xplmn, dec_st, enc_st -from pySim.legacy.utils import format_ePDGSelection, dec_addr_tlv, enc_addr_tlv +from pySim.legacy.utils import format_ePDGSelection, dec_addr_tlv, enc_addr_tlv, dec_msisdn, enc_msisdn from pySim.legacy.ts_51_011 import EF, DF from pySim.legacy.ts_31_102 import EF_USIM_ADF_map from pySim.legacy.ts_31_103 import EF_ISIM_ADF_map diff --git a/pySim/legacy/utils.py b/pySim/legacy/utils.py index cf2416e..4bfd061 100644 --- a/pySim/legacy/utils.py +++ b/pySim/legacy/utils.py @@ -20,8 +20,10 @@ # along with this program. If not, see <http://www.gnu.org/licenses/>. # +from typing import Optional, Tuple from pySim.utils import Hexstr, rpad, enc_plmn, h2i, i2s, s2h from pySim.utils import dec_xplmn_w_act, dec_xplmn, dec_mcc_from_plmn, dec_mnc_from_plmn +from osmocom.utils import swap_nibbles, h2b, b2h def hexstr_to_Nbytearr(s, nbytes): return [s[i:i+(nbytes*2)] for i in range(0, len(s), (nbytes*2))] @@ -330,3 +332,82 @@ s += '80' + ('%02x' % ((len(ipv4_str)//2)+2)) + '01' + 'ff' + ipv4_str return s + + +def dec_msisdn(ef_msisdn: Hexstr) -> Optional[Tuple[int, int, Optional[str]]]: + """ + Decode MSISDN from EF.MSISDN or EF.ADN (same structure). + See 3GPP TS 31.102, section 4.2.26 and 4.4.2.3. + """ + + # Convert from str to (kind of) 'bytes' + ef_msisdn = h2b(ef_msisdn) + + # Make sure mandatory fields are present + if len(ef_msisdn) < 14: + raise ValueError("EF.MSISDN is too short") + + # Skip optional Alpha Identifier + xlen = len(ef_msisdn) - 14 + msisdn_lhv = ef_msisdn[xlen:] + + # Parse the length (in bytes) of the BCD encoded number + bcd_len = msisdn_lhv[0] + # BCD length = length of dial num (max. 10 bytes) + 1 byte ToN and NPI + if bcd_len == 0xff: + return None + elif bcd_len > 11 or bcd_len < 1: + raise ValueError( + "Length of MSISDN (%d bytes) is out of range" % bcd_len) + + # Parse ToN / NPI + ton = (msisdn_lhv[1] >> 4) & 0x07 + npi = msisdn_lhv[1] & 0x0f + bcd_len -= 1 + + # No MSISDN? + if not bcd_len: + return (npi, ton, None) + + msisdn = swap_nibbles(b2h(msisdn_lhv[2:][:bcd_len])).rstrip('f') + # International number 10.5.118/3GPP TS 24.008 + if ton == 0x01: + msisdn = '+' + msisdn + + return (npi, ton, msisdn) + + +def enc_msisdn(msisdn: str, npi: int = 0x01, ton: int = 0x03) -> Hexstr: + """ + Encode MSISDN as LHV so it can be stored to EF.MSISDN. + See 3GPP TS 31.102, section 4.2.26 and 4.4.2.3. (The result + will not contain the optional Alpha Identifier at the beginning.) + + Default NPI / ToN values: + - NPI: ISDN / telephony numbering plan (E.164 / E.163), + - ToN: network specific or international number (if starts with '+'). + """ + + # If no MSISDN is supplied then encode the file contents as all "ff" + if msisdn in ["", "+"]: + return "ff" * 14 + + # Leading '+' indicates International Number + if msisdn[0] == '+': + msisdn = msisdn[1:] + ton = 0x01 + + # An MSISDN must not exceed 20 digits + if len(msisdn) > 20: + raise ValueError("msisdn must not be longer than 20 digits") + + # Append 'f' padding if number of digits is odd + if len(msisdn) % 2 > 0: + msisdn += 'f' + + # BCD length also includes NPI/ToN header + bcd_len = len(msisdn) // 2 + 1 + npi_ton = (npi & 0x0f) | ((ton & 0x07) << 4) | 0x80 + bcd = rpad(swap_nibbles(msisdn), 10 * 2) # pad to 10 octets + + return ('%02x' % bcd_len) + ('%02x' % npi_ton) + bcd + ("ff" * 2) diff --git a/pySim/utils.py b/pySim/utils.py index 05ad962..c706b34 100644 --- a/pySim/utils.py +++ b/pySim/utils.py @@ -341,86 +341,6 @@ return mnc - -def dec_msisdn(ef_msisdn: Hexstr) -> Optional[Tuple[int, int, Optional[str]]]: - """ - Decode MSISDN from EF.MSISDN or EF.ADN (same structure). - See 3GPP TS 31.102, section 4.2.26 and 4.4.2.3. - """ - - # Convert from str to (kind of) 'bytes' - ef_msisdn = h2b(ef_msisdn) - - # Make sure mandatory fields are present - if len(ef_msisdn) < 14: - raise ValueError("EF.MSISDN is too short") - - # Skip optional Alpha Identifier - xlen = len(ef_msisdn) - 14 - msisdn_lhv = ef_msisdn[xlen:] - - # Parse the length (in bytes) of the BCD encoded number - bcd_len = msisdn_lhv[0] - # BCD length = length of dial num (max. 10 bytes) + 1 byte ToN and NPI - if bcd_len == 0xff: - return None - elif bcd_len > 11 or bcd_len < 1: - raise ValueError( - "Length of MSISDN (%d bytes) is out of range" % bcd_len) - - # Parse ToN / NPI - ton = (msisdn_lhv[1] >> 4) & 0x07 - npi = msisdn_lhv[1] & 0x0f - bcd_len -= 1 - - # No MSISDN? - if not bcd_len: - return (npi, ton, None) - - msisdn = swap_nibbles(b2h(msisdn_lhv[2:][:bcd_len])).rstrip('f') - # International number 10.5.118/3GPP TS 24.008 - if ton == 0x01: - msisdn = '+' + msisdn - - return (npi, ton, msisdn) - - -def enc_msisdn(msisdn: str, npi: int = 0x01, ton: int = 0x03) -> Hexstr: - """ - Encode MSISDN as LHV so it can be stored to EF.MSISDN. - See 3GPP TS 31.102, section 4.2.26 and 4.4.2.3. (The result - will not contain the optional Alpha Identifier at the beginning.) - - Default NPI / ToN values: - - NPI: ISDN / telephony numbering plan (E.164 / E.163), - - ToN: network specific or international number (if starts with '+'). - """ - - # If no MSISDN is supplied then encode the file contents as all "ff" - if msisdn in ["", "+"]: - return "ff" * 14 - - # Leading '+' indicates International Number - if msisdn[0] == '+': - msisdn = msisdn[1:] - ton = 0x01 - - # An MSISDN must not exceed 20 digits - if len(msisdn) > 20: - raise ValueError("msisdn must not be longer than 20 digits") - - # Append 'f' padding if number of digits is odd - if len(msisdn) % 2 > 0: - msisdn += 'f' - - # BCD length also includes NPI/ToN header - bcd_len = len(msisdn) // 2 + 1 - npi_ton = (npi & 0x0f) | ((ton & 0x07) << 4) | 0x80 - bcd = rpad(swap_nibbles(msisdn), 10 * 2) # pad to 10 octets - - return ('%02x' % bcd_len) + ('%02x' % npi_ton) + bcd + ("ff" * 2) - - def sanitize_pin_adm(pin_adm, pin_adm_hex=None) -> Hexstr: """ The ADM pin can be supplied either in its hexadecimal form or as diff --git a/tests/unittests/test_utils.py b/tests/unittests/test_utils.py index 0e61a61..30a01d2 100755 --- a/tests/unittests/test_utils.py +++ b/tests/unittests/test_utils.py @@ -145,31 +145,31 @@ self.assertEqual(encoded.lower(), self.testfile_suci_calc_info.lower()) def testEnc_msisdn(self): - msisdn_encoded = utils.enc_msisdn("+4916012345678", npi=0x01, ton=0x03) + msisdn_encoded = legacy_utils.enc_msisdn("+4916012345678", npi=0x01, ton=0x03) self.assertEqual(msisdn_encoded, "0891946110325476f8ffffffffff") - msisdn_encoded = utils.enc_msisdn("123456", npi=0x01, ton=0x03) + msisdn_encoded = legacy_utils.enc_msisdn("123456", npi=0x01, ton=0x03) self.assertEqual(msisdn_encoded, "04b1214365ffffffffffffffffff") - msisdn_encoded = utils.enc_msisdn("12345678901234567890", npi=0x01, ton=0x03) + msisdn_encoded = legacy_utils.enc_msisdn("12345678901234567890", npi=0x01, ton=0x03) self.assertEqual(msisdn_encoded, "0bb121436587092143658709ffff") - msisdn_encoded = utils.enc_msisdn("+12345678901234567890", npi=0x01, ton=0x03) + msisdn_encoded = legacy_utils.enc_msisdn("+12345678901234567890", npi=0x01, ton=0x03) self.assertEqual(msisdn_encoded, "0b9121436587092143658709ffff") - msisdn_encoded = utils.enc_msisdn("", npi=0x01, ton=0x03) + msisdn_encoded = legacy_utils.enc_msisdn("", npi=0x01, ton=0x03) self.assertEqual(msisdn_encoded, "ffffffffffffffffffffffffffff") - msisdn_encoded = utils.enc_msisdn("+", npi=0x01, ton=0x03) + msisdn_encoded = legacy_utils.enc_msisdn("+", npi=0x01, ton=0x03) self.assertEqual(msisdn_encoded, "ffffffffffffffffffffffffffff") def testDec_msisdn(self): - msisdn_decoded = utils.dec_msisdn("0891946110325476f8ffffffffff") + msisdn_decoded = legacy_utils.dec_msisdn("0891946110325476f8ffffffffff") self.assertEqual(msisdn_decoded, (1, 1, "+4916012345678")) - msisdn_decoded = utils.dec_msisdn("04b1214365ffffffffffffffffff") + msisdn_decoded = legacy_utils.dec_msisdn("04b1214365ffffffffffffffffff") self.assertEqual(msisdn_decoded, (1, 3, "123456")) - msisdn_decoded = utils.dec_msisdn("0bb121436587092143658709ffff") + msisdn_decoded = legacy_utils.dec_msisdn("0bb121436587092143658709ffff") self.assertEqual(msisdn_decoded, (1, 3, "12345678901234567890")) - msisdn_decoded = utils.dec_msisdn("ffffffffffffffffffffffffffff") + msisdn_decoded = legacy_utils.dec_msisdn("ffffffffffffffffffffffffffff") self.assertEqual(msisdn_decoded, None) - msisdn_decoded = utils.dec_msisdn("00112233445566778899AABBCCDDEEFF001122330bb121436587092143658709ffff") + msisdn_decoded = legacy_utils.dec_msisdn("00112233445566778899AABBCCDDEEFF001122330bb121436587092143658709ffff") self.assertEqual(msisdn_decoded, (1, 3, "12345678901234567890")) - msisdn_decoded = utils.dec_msisdn("ffffffffffffffffffffffffffffffffffffffff0bb121436587092143658709ffff") + msisdn_decoded = legacy_utils.dec_msisdn("ffffffffffffffffffffffffffffffffffffffff0bb121436587092143658709ffff") self.assertEqual(msisdn_decoded, (1, 3, "12345678901234567890")) class TestLuhn(unittest.TestCase): -- To view, visit https://gerrit.osmocom.org/c/pysim/+/38118?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I19ec8ba14551ec282fc0cc12ae2f6d528bdfc527 Gerrit-Change-Number: 38118 Gerrit-PatchSet: 13 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org>

1 0

[M] Change in pysim[master]: utils: move enc_msisdn and dec_msisdn to legacy/utils.py
by laforge 30 Sep '24

30 Sep '24

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/38118?usp=email ) Change subject: utils: move enc_msisdn and dec_msisdn to legacy/utils.py ...................................................................... Patch Set 13: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/38118?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: I19ec8ba14551ec282fc0cc12ae2f6d528bdfc527 Gerrit-Change-Number: 38118 Gerrit-PatchSet: 13 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-Comment-Date: Mon, 30 Sep 2024 16:48:35 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in pysim[master]: ts_102_221: se _test_de_encode instead of _test_decode in EF.DIR unit...
by laforge 30 Sep '24

30 Sep '24

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/38201?usp=email ) Change subject: ts_102_221: se _test_de_encode instead of _test_decode in EF.DIR unittest ...................................................................... Patch Set 8: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/38201?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: If459073c6ff927c1cc1790d506e3979243b1fb4c Gerrit-Change-Number: 38201 Gerrit-PatchSet: 8 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-Comment-Date: Mon, 30 Sep 2024 16:48:33 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in pysim[master]: ts_51_011: use _test_de_encode instead of _test_decode in EF.CFIS uni...
by laforge 30 Sep '24

30 Sep '24

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/38200?usp=email ) Change subject: ts_51_011: use _test_de_encode instead of _test_decode in EF.CFIS unittest ...................................................................... Patch Set 8: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/38200?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: Ib876fd799f871fe64ced2a7b64847ffd09e16ed9 Gerrit-Change-Number: 38200 Gerrit-PatchSet: 8 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-Comment-Date: Mon, 30 Sep 2024 16:48:31 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[XS] Change in pysim[master]: ts_51_011: fix unittest for EF.ADN
by laforge 30 Sep '24

30 Sep '24

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/38199?usp=email ) Change subject: ts_51_011: fix unittest for EF.ADN ...................................................................... Patch Set 8: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/38199?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: I241792e66ee6167be6ddc076453344b6307d6265 Gerrit-Change-Number: 38199 Gerrit-PatchSet: 8 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-Comment-Date: Mon, 30 Sep 2024 16:48:26 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[L] Change in pysim[master]: filesystem: pass total_len to construct of when encoding file contents
by laforge 30 Sep '24

30 Sep '24

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/38195?usp=email ) Change subject: filesystem: pass total_len to construct of when encoding file contents ...................................................................... Patch Set 8: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/38195?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: I1b7a51594fbc5d9fe01132c39354a2fa88d53f9b Gerrit-Change-Number: 38195 Gerrit-PatchSet: 8 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-Comment-Date: Mon, 30 Sep 2024 16:48:22 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[S] Change in pysim[master]: cosmetic: use **kwargs instead of **_kwargs
by laforge 30 Sep '24

30 Sep '24

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/38300?usp=email ) Change subject: cosmetic: use **kwargs instead of **_kwargs ...................................................................... Patch Set 4: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/38300?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: I98857cc774185e55a604eb4fbfbf62ed4bd6ded7 Gerrit-Change-Number: 38300 Gerrit-PatchSet: 4 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-Comment-Date: Mon, 30 Sep 2024 16:48:20 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

[L] Change in pysim[master]: filesystem: pass total_len to construct of when encoding file contents
by laforge 30 Sep '24

30 Sep '24

Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/38195?usp=email ) Change subject: filesystem: pass total_len to construct of when encoding file contents ...................................................................... Patch Set 8: Code-Review+1 (1 comment) File pySim/filesystem.py: https://gerrit.osmocom.org/c/pysim/+/38195/comment/6b189e33_76586eb9?usp=em… : PS6, Line 1099: return b2h(t.to_tlv()) > I think we have overlooked files that use _tlv. […] yes, but let's do it in a follow-up patch to not drag this one on any longer. -- To view, visit https://gerrit.osmocom.org/c/pysim/+/38195?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: I1b7a51594fbc5d9fe01132c39354a2fa88d53f9b Gerrit-Change-Number: 38195 Gerrit-PatchSet: 8 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-Comment-Date: Mon, 30 Sep 2024 16:48:07 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Comment-In-Reply-To: dexter <pmaier(a)sysmocom.de>

1 0

[S] Change in osmo-bts[master]: l1sap: prevent buffer overflow in l1sap_rtp_rx_cb()
by laforge 30 Sep '24

30 Sep '24

Attention is currently required from: fixeria. laforge has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/osmo-bts/+/38313?usp=email ) Change subject: l1sap: prevent buffer overflow in l1sap_rtp_rx_cb() ...................................................................... Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/38313?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Change-Id: I214070ecf7458202922475505a8747950bedf930 Gerrit-Change-Number: 38313 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Mon, 30 Sep 2024 16:46:09 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

1 0

2025

2024

2023

2022

gerrit-log September 2024