Attention is currently required from: fixeria, laforge.

View Change

2 comments:

• Patchset:

  • Patch Set #1:

    I have tried this out. and fed it some APDUs (the testcases from current master, /tests/pySim-shell_test/apdu/test_apdu.script). Case 1-3 seem to work fine but when I enter 00a40004023f0000 (case 4) it seems to have problems. Leaving out the Le field also does not help.

    Here is what I tried:

    ./pySim-trace.py stdin-hex
    INFO     root: Opening source stdin-hex...
    Detected UICC Add-on "SIM"
    Detected UICC Add-on "GSM-R"
    Detected UICC Add-on "RUIM"
    Can't read AIDs from SIM -- 'list' object has no attribute 'lower'
    warning: EF.DIR seems to be empty!
     ADF.ECASD: a0000005591010ffffffff8900000200
     ADF.ISD-R: a0000005591010ffffffff8900000100
     ISIM: a0000000871004
     USIM: a0000000871002
    INFO     root: Entering main loop...
    C-APDU >00a40004023f0000
    C-APDU >
    (nothing happens, so I just press enter...)
    Traceback (most recent call last):
      File "/home/user/work/git_master/pysim/./pySim-trace.py", line 220, in <module>
        tracer.main()
      File "/home/user/work/git_master/pysim/./pySim-trace.py", line 117, in main
        apdu = self.source.read()
               ^^^^^^^^^^^^^^^^^^
      File "/home/user/work/git_master/pysim/pySim/apdu_source/__init__.py", line 23, in read
        r = self.read_packet()
            ^^^^^^^^^^^^^^^^^^
      File "/home/user/work/git_master/pysim/pySim/apdu_source/stdin_hex.py", line 40, in read_packet
        return ApduCommands.parse_cmd_bytes(h2b(command) + h2b(response))
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      File "/home/user/work/git_master/pysim/pySim/apdu/__init__.py", line 405, in parse_cmd_bytes
        raise ValueError('Unknown CLA=%02X INS=%02X' % (cla, ins))
    ValueError: Unknown CLA=90 INS=00

    I also wonder what is with the responses? Those aren't intended to be analyzed?. Maybe I am just using it wrong. As I said, some more info in the commit message would be helpful.

• Commit Message:

  • Patch Set #1, Line 7: pySim.apdu_source.stdin_hex

    I think it would be good to write a sentence or two what this is patch is about and how it can be useful. I can see it is for pySim-trace, but I do not get the usecase.

To view, visit change 37925. To unsubscribe, or for help writing mail filters, visit settings.