Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error when uninstalling an applet #165

Closed
rben-dev opened this issue May 9, 2019 · 3 comments
Closed

Error when uninstalling an applet #165

rben-dev opened this issue May 9, 2019 · 3 comments

Comments

@rben-dev
Copy link

rben-dev commented May 9, 2019

Describe the bug

It seems that a regression might have been introduced with commit c8dad37. The '--uninstall' option for gp.jar provides a '0x6A80' error when trying to remove an existing applet.

Bisecting to the previous commit 5fd2424 allows to get back a working '--uninstall' option on the same card and with the same applet.

The error log (full trace provided below):

$ java -jar ./externals/globalplatformpro/target/gp.jar --uninstall ex.cap -v 
GlobalPlatformPro 19.01.22-5-gc8dad37
Running on Linux, Java 1.8.0_212 by Oracle Corporation
Reader: OMNIKEY AG CardMan 3121 00 00
ATR: 3BF91300008031FE454A434F503234325232A2
More information about your card:
    http://smartcard-atr.appspot.com/parse?ATR=3BF91300008031FE454A434F503234325232A2

[DEBUG] GlobalPlatform - Auto-detected ISD: A000000151000000
...
[DEBUG] DMTokenGenerator - Adding tag 0x9E before Delete Token
[DEBUG] DMTokenGenerator - No private key for token generation provided
Deletion failed: 0x6A80 (Wrong data/incorrect values in data)

Information about your card

As much information as you have:

  1. Vendor = NXP
  2. Product = JCOP (J3D081)
  3. Version = J3D081

Expected behavior

The applet should be properly deleted with '--uninstall'.

Full log

gp_log.txt

Thanks in advance for your valuable help,

@martinpaljak
Copy link
Owner

Thanks for noticing.
That PR was merged with the clause that it will be re-written a bit, to support static tokens. Will be fixed with that change.

@rben-dev
Copy link
Author

@martinpaljak,

Thank you very much for the follow-up and the confirmation!
Waiting for the fix to be merged in master then!

Regards,

@martinpaljak martinpaljak mentioned this issue May 16, 2019
martinpaljak added a commit that referenced this issue May 16, 2019
- Sanitize key handling, splitting card keys and session keys
- Merge registry elements into single class
- Rename things to be more uniform
- Remove a lot of rot
- Release a snapshot with updated dependencies

closes #118 #165 #153 #9
@martinpaljak
Copy link
Owner

This should be fixed in latest release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants