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

nfc: m1k-based Aime (non-AIC) card support #3241

Merged
merged 2 commits into from
Dec 3, 2023
Merged

nfc: m1k-based Aime (non-AIC) card support #3241

merged 2 commits into from
Dec 3, 2023

Conversation

magicnat
Copy link
Contributor

@magicnat magicnat commented Nov 24, 2023

What's new

  • Added a new NFC supported card plugin for the old red-black colored Aime cards (MIFARE Classic 1K based)

Verification

Note: The example card above does not contain a valid access code and will not work on real arcades.

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

@dogtopus
Copy link
Contributor

Soccer game key finally online

Copy link
Member

@gornekich gornekich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@skotopes skotopes merged commit c477d13 into flipperdevices:dev Dec 3, 2023
@ry4000
Copy link
Contributor

ry4000 commented Dec 25, 2023

Tested this on my end - I was able to read my Aime Cards; many thanks!

That said, is there a way to provide similar support for Bandai Namco Passports?

The MFC BNP access keys are provided in PR #3314.

Ideally, the BNP plugin looks for both Sector 0 Keys:

  • Parse as a BNP if S0KA is 6090D00632F5 and S0KB is 574343467632; print the last 10 bytes in S0B2, which is the Access Code;

  • Parse as a BNP if S0KA is 6090D00632F5, but S0KB is 019761AA8082; read is still valid, but instead print "Please check the back of your Bandai Namco Passport for your Access Code" because S0B2 would instead be a Value Block.

Unlike the Aime plugin, I am unsure if BNPs have a checksum; however, I do know that the BNP readers look for all Sector Keys in their correct spots, but does not check for correct ACLs [which may be modified without issue].

@ry4000 ry4000 mentioned this pull request Dec 25, 2023
3 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants