CHG: minor code cleaning in 'hf 14a reader'

This commit is contained in:
iceman1001 2016-08-10 16:24:49 +02:00
commit 823ad2e186

View file

@ -179,6 +179,7 @@ int CmdHF14AList(const char *Cmd) {
}
int CmdHF14AReader(const char *Cmd) {
UsbCommand cDisconnect = {CMD_READER_ISO_14443a, {0,0,0}};
UsbCommand c = {CMD_READER_ISO_14443a, {ISO14A_CONNECT | ISO14A_NO_DISCONNECT, 0, 0}};
clearCommandBuffer();
SendCommand(&c);
@ -192,22 +193,14 @@ int CmdHF14AReader(const char *Cmd) {
if(select_status == 0) {
if (Cmd[0] != 's') PrintAndLog("iso14443a card select failed");
// disconnect
c.arg[0] = 0;
c.arg[1] = 0;
c.arg[2] = 0;
SendCommand(&c);
SendCommand(&cDisconnect);
return 0;
}
if(select_status == 3) {
PrintAndLog("Card doesn't support standard iso14443-3 anticollision");
PrintAndLog("ATQA : %02x %02x", card.atqa[1], card.atqa[0]);
// disconnect
c.arg[0] = 0;
c.arg[1] = 0;
c.arg[2] = 0;
SendCommand(&c);
SendCommand(&cDisconnect);
return 0;
}
@ -218,13 +211,13 @@ int CmdHF14AReader(const char *Cmd) {
switch (card.sak) {
case 0x00:
//***************************************test****************
// ******** is card of the MFU type (UL/ULC/NTAG/ etc etc)
ul_switch_off_field();
uint32_t tagT = GetHF14AMfU_Type();
ul_print_type(tagT, 0);
//reconnect for further tests
// reconnect for further tests
c.arg[0] = ISO14A_CONNECT | ISO14A_NO_DISCONNECT;
c.arg[1] = 0;
c.arg[2] = 0;
@ -233,7 +226,7 @@ int CmdHF14AReader(const char *Cmd) {
SendCommand(&c);
UsbCommand resp;
WaitForResponse(CMD_ACK,&resp);
WaitForResponse(CMD_ACK, &resp);
memcpy(&card, (iso14a_card_select_t *)resp.d.asBytes, sizeof(iso14a_card_select_t));
@ -243,27 +236,6 @@ int CmdHF14AReader(const char *Cmd) {
ul_switch_off_field();
return 0;
}
/* orig
// check if the tag answers to GETVERSION (0x60)
c.arg[0] = ISO14A_RAW | ISO14A_APPEND_CRC | ISO14A_NO_DISCONNECT;
c.arg[1] = 1;
c.arg[2] = 0;
c.d.asBytes[0] = 0x60;
SendCommand(&c);
WaitForResponse(CMD_ACK,&resp);
uint8_t version[10] = {0};
memcpy(version, resp.d.asBytes, resp.arg[0] < sizeof(version) ? resp.arg[0] : sizeof(version));
uint8_t len = resp.arg[0] & 0xff;
switch ( len){
// todo, identify "Magic UL-C tags". // they usually have a static nonce response to 0x1A command.
// UL-EV1, size, check version[6] == 0x0b (smaller) 0x0b * 4 == 48
case 0x0A:PrintAndLog("TYPE : NXP MIFARE Ultralight EV1 %d bytes", (version[6] == 0xB) ? 48 : 128); break;
case 0x01:PrintAndLog("TYPE : NXP MIFARE Ultralight C");break;
case 0x00:PrintAndLog("TYPE : NXP MIFARE Ultralight");break;
}
*/
break;
case 0x01: PrintAndLog("TYPE : NXP TNP3xxx Activision Game Appliance"); break;
case 0x04: PrintAndLog("TYPE : NXP MIFARE (various !DESFire !DESFire EV1)"); break;
@ -379,59 +351,29 @@ int CmdHF14AReader(const char *Cmd) {
PrintAndLog(" %02x -> Length is %d bytes",
card.ats[pos + 1], card.ats[pos + 1]);
switch (card.ats[pos + 2] & 0xf0) {
case 0x10:
PrintAndLog(" 1x -> MIFARE DESFire");
break;
case 0x20:
PrintAndLog(" 2x -> MIFARE Plus");
break;
case 0x10: PrintAndLog(" 1x -> MIFARE DESFire"); break;
case 0x20: PrintAndLog(" 2x -> MIFARE Plus"); break;
}
switch (card.ats[pos + 2] & 0x0f) {
case 0x00:
PrintAndLog(" x0 -> <1 kByte");
break;
case 0x01:
PrintAndLog(" x1 -> 1 kByte");
break;
case 0x02:
PrintAndLog(" x2 -> 2 kByte");
break;
case 0x03:
PrintAndLog(" x3 -> 4 kByte");
break;
case 0x04:
PrintAndLog(" x4 -> 8 kByte");
break;
case 0x00: PrintAndLog(" x0 -> <1 kByte"); break;
case 0x01: PrintAndLog(" x1 -> 1 kByte"); break;
case 0x02: PrintAndLog(" x2 -> 2 kByte"); break;
case 0x03: PrintAndLog(" x3 -> 4 kByte"); break;
case 0x04: PrintAndLog(" x4 -> 8 kByte"); break;
}
switch (card.ats[pos + 3] & 0xf0) {
case 0x00:
PrintAndLog(" 0x -> Engineering sample");
break;
case 0x20:
PrintAndLog(" 2x -> Released");
break;
case 0x00: PrintAndLog(" 0x -> Engineering sample"); break;
case 0x20: PrintAndLog(" 2x -> Released"); break;
}
switch (card.ats[pos + 3] & 0x0f) {
case 0x00:
PrintAndLog(" x0 -> Generation 1");
break;
case 0x01:
PrintAndLog(" x1 -> Generation 2");
break;
case 0x02:
PrintAndLog(" x2 -> Generation 3");
break;
case 0x00: PrintAndLog(" x0 -> Generation 1"); break;
case 0x01: PrintAndLog(" x1 -> Generation 2"); break;
case 0x02: PrintAndLog(" x2 -> Generation 3"); break;
}
switch (card.ats[pos + 4] & 0x0f) {
case 0x00:
PrintAndLog(" x0 -> Only VCSL supported");
break;
case 0x01:
PrintAndLog(" x1 -> VCS, VCSL, and SVC supported");
break;
case 0x0E:
PrintAndLog(" xE -> no VCS command supported");
break;
case 0x00: PrintAndLog(" x0 -> Only VCSL supported"); break;
case 0x01: PrintAndLog(" x1 -> VCS, VCSL, and SVC supported"); break;
case 0x0E: PrintAndLog(" xE -> no VCS command supported"); break;
}
}
}
@ -454,11 +396,7 @@ int CmdHF14AReader(const char *Cmd) {
PrintAndLog("Answers to magic commands (GEN1): %s", (isOK ? "YES" : "NO") );
// disconnect
c.cmd = CMD_READER_ISO_14443a;
c.arg[0] = 0;
c.arg[1] = 0;
c.arg[2] = 0;
SendCommand(&c);
SendCommand(&cDisconnect);
return select_status;
}