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

Added lfrfid protocol EM4100/40/RAW #722

Closed

Conversation

hippocache
Copy link

@hippocache hippocache commented Feb 28, 2024

This protocol allows users to send 8 bytes (64 bits) of data directly without any encoding.

Added documentation/LFRFID_EM4100_RAW.md

What's new

Sometimes it is useful to send eight bytes of your choosing over, and over again using ASK at 40cs/bit-rate.

Verification

A device such as a proxmark3 should be able to demodulate data as em4100. Try emulating FF80000001234567 while reading with a device like proxmark3.

Checklist (For Reviewer)

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

This protocol allows users to send 8 bytes (64 bits) of data
directly without any encoding.

Added documentation/LFRFID_EM4100_RAW.md
@xMasterX xMasterX marked this pull request as draft March 11, 2024 00:59
@DarkFlippers DarkFlippers deleted a comment from vaidasnet Mar 18, 2024
@hippocache
Copy link
Author

The SecuraKey LFRFID implementation on OFW (Release 0.103.1, unleashed release 75) , and the LFRFID fuzzer, make this patch irrelevant / unnecessary.

@hippocache hippocache closed this Jul 4, 2024
@xMasterX
Copy link
Member

xMasterX commented Jul 4, 2024

The SecuraKey LFRFID implementation on OFW (Release 0.103.1, unleashed release 75) , and the LFRFID fuzzer, make this patch irrelevant / unnecessary.

Not sure what you mean about lfrfid fuzzer,
but the idea of RAW option itself in lfrfid sounds interesting, this one exact implementation was specifically tuned for that system and wasn't universal, so it wasn't accepted in first place, also we do not have that cards to test it with
I asked @Leptopt1los couple times to respond here and explain the issues of this PR
but it never got anywhere than our local messages, sorry for no response here!

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

Successfully merging this pull request may close these issues.

None yet

2 participants