Hello, Thanks again for your work on this. I suspect this is a relatively easy fix

Same on Sierra MC7710. <a class="user-mention notranslate" data-hove

<a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/us

This should be fixed in commit <a class="commit-link" data-hovercard-type="commit" dat

When run with --info, script crashes out when trying to parse version_strings about qcsuper HOT 4 CLOSED

p1sec commented on June 5, 2024

When run with --info, script crashes out when trying to parse version_strings

from qcsuper.

Comments (4)

StefanBruens commented on June 5, 2024

Same on Sierra MC7710.

@eriksejr - can you change line 88 of info.py as follows:

-            build_id, model_string, _ = ver_strings.split(b'\x00')
+            print_row('ver_strings', ver_strings)
+            build_id, model_string, *_ = ver_strings.split(b'\x00')

For the MC7710, output is:
[+] ver_strings: b'M9200B-SCAQWBZD-X3052903\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00'

As can be seen, the output is padded with b'\x00', each creating an empty segment.

from qcsuper.

eriksejr commented on June 5, 2024

@StefanBruens yes that fixes the issue. The version string is similar padded with NULLs:

[+] Compilation date:    May 19 2017 06:37:30
[+] Release date:        Apr 21 2017 10:00:00
[+] Version directory:   C2240506

[+] Common air interface information: 
[+]   Station classmark: 58
[+]   Common air interface revision: 6
[+]   Mobile model:      255
[+]   Mobile firmware revision: 100
[+]   Slot cycle index:  2
[+]   Hardware revision: 0x922 (9.34)

[+] ver_strings:         b'C2240506-M9635TAAAANAZM-1.96273.1\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00'
[+] Mobile model ID:     0xffd
[+] Chip version:        2
[+] Firmware build ID:   C2240506-M9635TAAAANAZM-1.96273.1

[+] Diag version:        8

[+] Serial number:       0

Thanks!

from qcsuper.

StefanBruens commented on June 5, 2024

@eriksejr - you shouldn´t close this, as long as a fix is not committed to the repo.

from qcsuper.