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

Partial implementation of NTAG424 commands #2150

Merged
merged 11 commits into from
Oct 29, 2023
Merged

Partial implementation of NTAG424 commands #2150

merged 11 commits into from
Oct 29, 2023

Conversation

dankar
Copy link
Contributor

@dankar dankar commented Oct 26, 2023 •
edited
Loading

Missing stuff:

  • Shortcuts for CC file (set to read only etc)
  • Verification of MAC when using mac or encrypted channel.
  • LRP mode
  • APDU chaining

Stuff that works:

  • Read/write file (including over encrypted channel)
  • Get/change file settings
  • Change auth keys
  • Command to test auth keys
  • Configuring SDM/SUN feature

@dankar
Partial implementation of NTAG424 commands
7042293 
Noteworthy stuff that's missing:
* Encrypted read/write to file.
* Handling of CC and Proprietary file

Stuff that works:
* Read/write file
* Get/change file settings
* Change auth keys
* Configuring SDM/SUN feature
@github-actions
Copy link

You are welcome to add an entry to the CHANGELOG.md as well

dankar added 2 commits October 27, 2023 06:52
@dankar
Refactored MAC and IV calculations
011d1f6
@dankar
Cleanup
eb29e1a 
 * Added some comments
 * Changed name of offsets to sdm_data since this is more apt. Not all
   of them are describing offsets
 * Fixed dumb bug in MAC refactor
@dankar
Added encrypted comms to ReadData and WriteData commands
912c126 
This allows reading and writing to the "proprietary" file (number 3),
or file 2 if it's set to "full" communications mode.
@iceman1001 iceman1001 merged commit d22d6c4 into RfidResearchGroup:master Oct 29, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dankar @iceman1001