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

possible miscoding? #85

Open
stylejuya opened this issue Dec 7, 2023 · 2 comments
Open

possible miscoding? #85

stylejuya opened this issue Dec 7, 2023 · 2 comments

Comments

@stylejuya
Copy link

issue here:

if (responseDataExpected) {

lline 5161 code must be below line 5174 if-condition "if (responseDataExpected)". Le is not set in more blocks use case.

@koh-osug
Copy link

koh-osug commented Dec 8, 2023

Not sure if this is an issue. Only the last STORE command should return response data was my understanding. For the other cases Le is not set since they ares no case 4 APDUs. Do you have a test where this behaviour is causing a bug to be sure?

@koh-osug
Copy link

Please provide some input here, since this not a known issue so far, I would like to verify this.

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