Skip to content
New issue

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

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

Already on GitHub? # to your account

[FL-3675] Ntag21x write #3246

Merged
merged 23 commits into from
Dec 2, 2023
Merged

Conversation

RebornedBrain
Copy link
Contributor

@RebornedBrain RebornedBrain commented Nov 27, 2023

What's new

Now Flipper can write user data from one stored NTAG21x to another presented card.

There are several checks to be passed in order to write successfully:

  • Write and presented cards must be of the same type NTAG213 - NTAG213, NTAG215 - NTAG215, NTAG216 - NTAG216
  • Presented card must have default PWD, PACK and disabled AUTH0
  • Static and dynamic lock bits on the presented card must be 0, which guaranties that all user memory is unlocked.

Verification

Case 1:

  • On some stored or fresh read NTAG213 choose "Write" button
  • Then present another NTAG213 fresh to flipper.
    Result: Writting success screen and data written to the presented card.

Case 2:

  • On some stored or fresh read NTAG213 choose "Write" button
  • Then present another NOT NTAG213 fresh to flipper.
    Result: "This is wrong card" screen and no data written to the presented card.

Case 3:

  • On some stored or fresh read NTAG213 choose "Write" button
  • Then present another NTAG213 with pwd set to flipper.
    Result: "Writting gone wrong" screen and no data written to the presented card.

Case 4:

  • On some stored or fresh read NTAG213 choose "Write" button
  • Then present another NTAG213 with static or dynamic lock bits set to flipper.
    Result: "Writting gone wrong" screen and no data written to the presented card.

Checklist (For Reviewer)

  • PR has description of feature/bug or link to Confluence/Jira task
  • Description contains actions to verify feature/bugfix
  • I've built this code, uploaded it to the device and verified feature/bugfix

@gornekich gornekich added the NFC NFC-related label Nov 30, 2023
@gornekich gornekich marked this pull request as draft November 30, 2023 15:25
@gornekich gornekich self-assigned this Nov 30, 2023
@RebornedBrain RebornedBrain marked this pull request as ready for review December 1, 2023 09:47
gornekich
gornekich previously approved these changes Dec 1, 2023
@skotopes skotopes merged commit 6a5d638 into flipperdevices:dev Dec 2, 2023
@gornekich gornekich mentioned this pull request Dec 5, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
NFC NFC-related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants