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

Rework hitag2 read/write help #310

Merged
merged 3 commits into from
Aug 9, 2019
Merged

Rework hitag2 read/write help #310

merged 3 commits into from
Aug 9, 2019

Conversation

ViRb3
Copy link
Contributor

@ViRb3 ViRb3 commented Aug 5, 2019

And add default keys

@iceman1001
Copy link
Collaborator

Hm.. so there is no write / read on hitag S?

@ViRb3
Copy link
Contributor Author

ViRb3 commented Aug 5, 2019

Sure there is? I only reworded the help text.

@iceman1001
Copy link
Collaborator

Because your rewording removes "read all pages" etc from the help text..
So it that not true anymore?

@ViRb3
Copy link
Contributor Author

ViRb3 commented Aug 5, 2019

Oh, no, it's still valid. I just assumed it is obvious that all commands under read will read, and under write will write. Think I should add that back to the help?

@iceman1001
Copy link
Collaborator

Not sure its much clearer with your suggested changes.
hitag reader... acting as a reader doesn't mean read , read all pages, etc..

@ViRb3
Copy link
Contributor Author

ViRb3 commented Aug 5, 2019

Sure, I'll re-write it to be clearer.

@ViRb3
Copy link
Contributor Author

ViRb3 commented Aug 6, 2019

How does that look like? :)

@doegox
Copy link
Contributor

doegox commented Aug 9, 2019

looks good, thanks

@doegox doegox merged commit ceda9f9 into RfidResearchGroup:master Aug 9, 2019
@iceman1001
Copy link
Collaborator

Great!

@pwpiwi
Copy link
Contributor

pwpiwi commented Aug 13, 2019

Are there other default passwords? It doesn't work for me on a brand new Hitag2 tag:

[usb] pm3 --> lf hitag reader
Hitag reader functions
Usage: lf hitag reader [h] <reader function #>
Options:
       h               This help
   HitagS (0*)
      01 <nr> <ar>     Read all pages, challenge mode
      02 <key>         Read all pages, crypto mode. Set key=0 for no auth
   Hitag1 (1*)
      Not implemented
   Hitag2 (2*)
      21 <password>    Read all pages, password mode. Default: 4D494B52 ("MIKR")
      22 <nr> <ar>     Read all pages, challenge mode
      23 <key>         Read all pages, crypto mode. Key format: ISK high + ISK low. Default: 4F4E4D494B52 ("ONMIKR")
      25               Test recorded authentications
      26               Just read UID
[usb] pm3 --> lf hitag reader 21 4d494b52
#db# List identifier in password mode
#db# Password failed!
[usb] pm3 --> lf hitag reader 26
[+] Valid Hitag2 tag found - UID: aa8e6b11

@iceman1001
Copy link
Collaborator

Better to use forum for questions.

@ViRb3
Copy link
Contributor Author

ViRb3 commented Aug 13, 2019

@pwpiwi those are the only default keys I'm afraid. I can confirm they work on my new Hitag2 cards. The distance and placement of the tag to the antenna is incredibly important, try moving it around and retrying the read.

@ViRb3 ViRb3 deleted the rework-help branch August 13, 2019 21:30
@pwpiwi
Copy link
Contributor

pwpiwi commented Aug 14, 2019

Indeed, I could make it work after some fiddling! While lf hitag 26 works at different distances, lf hitag 23 is more sensitive to distance (which makes me think that sending is the issue rather than receiving). In general, possible distances are quite low compared to commercial LF readers.

@doegox
Copy link
Contributor

doegox commented Aug 14, 2019

Indeed @pwpiwi lf hitag suffers from two problems: reading distance (while manually I could achieve much large distances) and last bit wrong/missing, see #124 and #240

@ViRb3
Copy link
Contributor Author

ViRb3 commented Aug 14, 2019

@pwpiwi any chance you could send that upstream hitag love (PR) back to this repo? I could use your refactor if I'm doing more work on hitag.

@pwpiwi
Copy link
Contributor

pwpiwi commented Aug 14, 2019

No need to send the latest changes from official to RRG. The PR adds latest additions (password write) formatting and refactoring from RRG to official. 😃

@ViRb3
Copy link
Contributor Author

ViRb3 commented Aug 14, 2019

I meant your touches, e.g. the commands extracted to the protocols header. That's good stuff :)

@pwpiwi
Copy link
Contributor

pwpiwi commented Aug 14, 2019

I had added this more or less for documentation only. Feel free to copy & paste. As discussed in #18 there is no easy way to reuse PRs and/or commits.

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

4 participants