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.

Sign up for GitHub

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

Jump to bottom

Allow reading data group from eMRTD with npa-tool #2257

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Allow reading data group from eMRTD with npa-tool #2257

wants to merge 3 commits into from

Commits on Dec 12, 2021

  1. npa-tool: allow reading DGs from eMRTD 

    Adds the new options --application and --read-all-dgs

fixes OpenSC#2253
    @frankmorgner
    frankmorgner committed Dec 12, 2021
    Configuration menu
    Copy the full SHA
    42375da View commit details
    Browse the repository at this point in the history

  2. fixed bad error translation 

    a negative return value was incorrectly translated to a huge buffer
length, which then errors with "offset too large"
    @frankmorgner
    frankmorgner committed Dec 12, 2021
    Configuration menu
    Copy the full SHA
    635df88 View commit details
    Browse the repository at this point in the history

  3. iso7816: if index is beyond file, return EOF 

    commands for data unit handling may return 6B00 if the specified index
is too large (ISO 7816-4). Maybe SC_ERROR_OFFSET_TOO_LARGE would be more
appropriate, but this is (1) currently not used only for internal errors
and (2) this error code is not currently handled by sc_read_binary.
Hence, we stick to SC_ERROR_FILE_END_REACHED.
    @frankmorgner
    frankmorgner committed Dec 12, 2021
    Configuration menu
    Copy the full SHA
    60bf597 View commit details
    Browse the repository at this point in the history