iceman1001 / proxmark3

[Deprecated] Iceman Fork, the most totally wicked fork around if you are into proxmark3
http://www.icedev.se/pm3.aspx
GNU General Public License v2.0
465 stars 116 forks source link

MIFARE_ULEV1_INCR_CNT wrong behavior #110

Closed iceman1001 closed 7 years ago

iceman1001 commented 7 years ago

The default behavior for MIFARE_ULEV1_INCR_CNT is wrong;

According to offical documents, the INCR_CNT command takes a 24bits value, not 32bits. If called with a number bigger than 24bits, it should send a NACK.

https://github.com/iceman1001/proxmark3/blob/master/armsrc/iso14443a.c#L1125

Thanks to @Vyacheslav for pointing out that the INCR_CNT simulation doesn't work properly.

ref: http://www.nxp.com/documents/data_sheet/MF0ULX1.pdf (see end of page 17)

iceman1001 commented 7 years ago

This issue has been addressed. You can test it again and se if this fixes your problem

iceman1001 commented 7 years ago

Since no reply, I will assume the problem is resolved and close this issue