]> git.zerfleddert.de Git - proxmark3-svn/blobdiff - client/cmdhfmfu.c
CHG: extracted the UL_C & UL magic tests.
[proxmark3-svn] / client / cmdhfmfu.c
index 3f406ba811459dd7803b2be29be1e6a9256875ff..efdbec107dbaf99a78d6c5826912b3b48be399da 100644 (file)
 #define MAX_ULEV1b_BLOCKS      0x20
 #define MAX_NTAG_213           0x2c
 #define MAX_NTAG_215           0x86
-#define MAX_NTAG_216           0xE6
+#define MAX_NTAG_216      0xe6
 
 typedef enum TAGTYPE_UL {
-       UNKNOWN         = 0,
-       UL                      = 1,
-       UL_C            = 2,
-       UL_EV1_48       = 4,
-       UL_EV1_128      = 8,
-       NTAG            = 16,
-       NTAG_213        = 32,
-       NTAG_215        = 64,
-       NTAG_216        = 128,
-       MAGIC           = 256,
+       UNKNOWN     = 0x0000,
+       UL          = 0x0001,
+       UL_C        = 0x0002,
+       UL_EV1_48   = 0x0004,
+       UL_EV1_128  = 0x0008,
+       NTAG        = 0x0010,
+       NTAG_213    = 0x0020,
+       NTAG_215    = 0x0040,
+       NTAG_216    = 0x0080,
+       MAGIC       = 0x0100,
        UL_MAGIC        = UL | MAGIC,
        UL_C_MAGIC      = UL_C | MAGIC,
        UL_ERROR        = 0xFFFF,
-       
 } TagTypeUL_t;
 
 uint8_t default_3des_keys[7][16] = {
@@ -49,6 +48,11 @@ uint8_t default_3des_keys[7][16] = {
                { 0xFF,0xFF,0xFF,0xFF,0xFF,0xFF,0xFF,0xFF,0xFF,0xFF,0xFF,0xFF,0xFF,0xFF,0xFF,0xFF },// all FF
                { 0x00,0x11,0x22,0x33,0x44,0x55,0x66,0x77,0x88,0x99,0xAA,0xBB,0xCC,0xDD,0xEE,0xFF }     // 11 22 33
        };
+
+uint8_t default_pwd_pack[2][4] = {
+       {0xFF,0xFF,0xFF,0xFF}, // PACK 0x00,0x00 -- factory default
+       {0x4A,0xF8,0x4B,0x19} // PACK 0xE5 0xBE -- italian bus (sniffed.)
+};     
        
 static int CmdHelp(const char *Cmd);
 
@@ -86,9 +90,9 @@ char* getUlev1CardSizeStr( uint8_t fsize ){
        
        // is  LSB set?
        if (  fsize & 1 )
-               sprintf(retStr, "0x%02X (%d - %d bytes)",fsize, usize, lsize);
+               sprintf(retStr, "0x%02X (%u - %u bytes)",fsize, usize, lsize);
        else 
-               sprintf(retStr, "0x%02X (%d bytes)", fsize, lsize);             
+               sprintf(retStr, "0x%02X (%u bytes)", fsize, lsize);             
        return buf;
 }
 
@@ -107,15 +111,14 @@ static int ul_send_cmd_raw( uint8_t *cmd, uint8_t cmdlen, uint8_t *response, uin
        memcpy(c.d.asBytes, cmd, cmdlen);
        SendCommand(&c);
        UsbCommand resp;
-       if (!WaitForResponseTimeout(CMD_ACK, &resp, 1500)) return -1;
-
-       uint16_t resplen = (resp.arg[0] < responseLength) ? resp.arg[0] : responseLength;
-       if (resp.arg[0] > 0) 
-               memcpy(response, resp.d.asBytes, resplen);
-
+       if ( !WaitForResponseTimeout(CMD_ACK, &resp, 1500) ) return -1;
+       if ( !resp.arg[0] && responseLength) return -1;
+       
+       uint16_t resplen = (resp.arg[0] < responseLength) ? resp.arg[0] : responseLength;       
+       memcpy(response, resp.d.asBytes, resplen);
        return resplen;
 }
-
+/*
 static int ul_send_cmd_raw_crc( uint8_t *cmd, uint8_t cmdlen, uint8_t *response, uint16_t responseLength, bool append_crc ) {
        
        UsbCommand c = {CMD_READER_ISO_14443a, {ISO14A_RAW | ISO14A_NO_DISCONNECT , cmdlen, 0}};
@@ -128,25 +131,22 @@ static int ul_send_cmd_raw_crc( uint8_t *cmd, uint8_t cmdlen, uint8_t *response,
        if (!WaitForResponseTimeout(CMD_ACK, &resp, 1500)) return -1;
 
        uint16_t resplen = (resp.arg[0] < responseLength) ? resp.arg[0] : responseLength;
-       if (resp.arg[0] > 0)
+       if (resp.arg[0] > 0) {
                memcpy(response, resp.d.asBytes, resplen );
-
        return resplen;
+       } else return -1;
 }
-
+*/
 static int ul_select( iso14a_card_select_t *card ){
        
        ul_switch_on_field();
 
        UsbCommand resp;
-       if (!WaitForResponseTimeout(CMD_ACK, &resp, 1500)) {
-               ul_switch_off_field();
-               return -1;
-       }
+       if (!WaitForResponseTimeout(CMD_ACK, &resp, 1500)) return -1;
+       if (resp.arg[0] < 0) return -1;
+       
+       memcpy(card, resp.d.asBytes, sizeof(iso14a_card_select_t));
        
-       if (resp.arg[0] > 0) {
-               memcpy(card, resp.d.asBytes, sizeof(iso14a_card_select_t));
-       }
        return resp.arg[0];
 }
 
@@ -160,6 +160,21 @@ static int ul_read( uint8_t page, uint8_t *response, uint16_t responseLength ){
        return len;
 }
 
+static int ul_write( uint8_t page, uint8_t *data, uint8_t datalen ){
+       uint8_t cmd[] = {MIFARE_ULC_WRITE, page, 0,0,0,0};
+       memcpy(cmd+2, data, datalen);
+       uint8_t response[1] = {0xff};
+       int len = ul_send_cmd_raw(cmd, sizeof(cmd), response, sizeof(response));
+       if ( len == -1 )
+               ul_switch_off_field();
+       
+       PrintAndLog("RESP: %02x", response[0]);
+       // ACK
+       if ( response[0] == 0x0a ) return 0;
+       // NACK
+       return -1;      
+}
+
 static int ulc_requestAuthentication( uint8_t blockNo, uint8_t *nonce, uint16_t nonceLength ){
        
        uint8_t cmd[] = {MIFARE_ULC_AUTH_1, blockNo};
@@ -275,9 +290,9 @@ static int ntag_print_CC(uint8_t *data) {
        return 0;                               
 }
 
-static int ul_print_version(uint8_t *data){            
+static int ulev1_print_version(uint8_t *data){         
        PrintAndLog("\n--- UL-EV1 / NTAG Version");
-       PrintAndLog("Raw version bytes: %s", sprint_hex(data, 8) );
+       PrintAndLog("       Raw bytes : %s", sprint_hex(data, 8) );
        PrintAndLog("       Vendor ID : 0x%02X, Manufacturer: %s", data[1], getTagInfo(data[1]));
        PrintAndLog("    Product type : %s"             , getProductTypeStr(data[2]));
        PrintAndLog(" Product subtype : 0x%02X %s"      , data[3], (data[3]==1) ?"17 pF":"50pF");
@@ -292,11 +307,11 @@ static int ul_print_type(uint16_t tagtype){
        if ( tagtype & UL )     
                PrintAndLog("      TYPE : MIFARE Ultralight (MF0ICU1) %s", (tagtype & MAGIC)?"<magic>":"");
        else if ( tagtype & UL_C)
-               PrintAndLog("      TYPE : MIFARE Ultralight C (MF0ULC) %s", (tagtype & MAGIC)?"<magic>":"" );
+               PrintAndLog("      TYPE : MIFARE Ultralight C (MF0ULC) %s [%x]", (tagtype & MAGIC)?"<magic>":"", tagtype );
        else if ( tagtype & UL_EV1_48)
                PrintAndLog("      TYPE : MIFARE Ultralight EV1 48bytes (MF0UL1101)"); 
        else if ( tagtype & UL_EV1_128) 
-               PrintAndLog("      TYPE : MIFARE Ultralight EV1 128bytes (MF0UL2101");
+               PrintAndLog("      TYPE : MIFARE Ultralight EV1 128bytes (MF0UL2101)");
        else if ( tagtype & NTAG_213 )
                PrintAndLog("      TYPE : MIFARE NTAG 213 144bytes (NT2H1311G0DU)");
        else if ( tagtype & NTAG_215 )
@@ -371,39 +386,96 @@ static int ulev1_print_counters(){
        uint8_t counter[3] = {0,0,0};
        for ( uint8_t i = 0; i<3; ++i) {
                ulev1_readCounter(i,counter, sizeof(counter) );
-               PrintAndLog("Counter [%d] : %s", i, sprint_hex(counter,3));
+               PrintAndLog("     [0x%0d] : %s", i, sprint_hex(counter,3));
        }
        return 0;
 }
 
+static int ulev1_print_signature( uint8_t *data, uint8_t len){
+       PrintAndLog("\n--- UL-EV1 Signature");  
+       PrintAndLog("IC signature public key name  : NXP NTAG21x 2013");
+       PrintAndLog("IC signature public key value : 04494e1a386d3d3cfe3dc10e5de68a499b1c202db5b132393e89ed19fe5be8bc61");
+       PrintAndLog("    Elliptic curve parameters : secp128r1");
+       PrintAndLog("            Tag ECC Signature : %s", sprint_hex(data, len));
+       //to do:  verify if signature is valid
+       //PrintAndLog("IC signature status: %s valid", (iseccvalid() )?"":"not");
+       return 0;
+}
+
+static int ulc_magic_test(){
+       // Magic Ultralight test
+               // Magic UL-C, by observation,
+       // 1) it seems to have a static nonce response to 0x1A command.
+       // 2) the deskey bytes is not-zero:d out on as datasheet states.
+       // 3) UID - changeable, not only, but pages 0-1-2-3.
+       int returnValue = UL_ERROR;
+       iso14a_card_select_t card;
+       uint8_t nonce1[11] = {0x00};
+       uint8_t nonce2[11] = {0x00};
+       int status = ul_select(&card);
+       if ( status < 1 ){
+               PrintAndLog("Error: couldn't select ulc_magic_test");
+               ul_switch_off_field();
+               return UL_ERROR;
+       }
+       status = ulc_requestAuthentication(0, nonce1, sizeof(nonce1));
+       if ( status > 0 ) {
+               status = ulc_requestAuthentication(0, nonce2, sizeof(nonce2));
+               returnValue =  ( !memcmp(nonce1, nonce2, 11) ) ? UL_C_MAGIC : UL_C;
+       } else {
+               returnValue = UL;
+       }       
+       ul_switch_off_field();
+       return returnValue;
+}
+
+static int ul_magic_test(){
+       
+       // Magic Ultralight test
+       // It takes present UID, and tries to write it back.
+       iso14a_card_select_t card;
+       int status = ul_select(&card);
+       if ( status < 1 ){
+               PrintAndLog("Error: couldn't select ul_magic_test");
+               ul_switch_off_field();
+               return UL_ERROR;
+       }
+       uint8_t page0 = 0;
+       uint8_t data[4] = {card.uid[0], card.uid[1], card.uid[2], 0};
+       data[3] = 0x88 ^ card.uid[0] ^ card.uid[1] ^  card.uid[2];
+
+       status = ul_write( page0, data, sizeof(data));
+       ul_switch_off_field();
+       if ( status == 0) 
+               return UL_MAGIC;
+       return UL;
+}
+
 uint16_t GetHF14AMfU_Type(void){
 
        TagTypeUL_t tagtype = UNKNOWN;
        iso14a_card_select_t card;
        uint8_t version[10] = {0x00};
-       uint8_t nonce1[11] = {0x00};
-       uint8_t nonce2[11] = {0x00};
        int status = 0;
        int len;
 
        status = ul_select(&card);
        if ( status < 1 ){
                PrintAndLog("Error: couldn't select");
+               ul_switch_off_field();
                return UL_ERROR;
        }
-
        // Ultralight - ATQA / SAK 
-       if ( card.atqa[1] != 0x00 && card.atqa[0] != 0x44 && card.sak != 0x00 ) {
+       if ( card.atqa[1] != 0x00 || card.atqa[0] != 0x44 || card.sak != 0x00 ) {
+               PrintAndLog("Tag is not UL or NTAG. [ATQA: %02X %02x SAK: %02X]", card.atqa[1],card.atqa[0],card.sak);
                ul_switch_off_field();
                return UL_ERROR;
        }
        
        len  = ulev1_getVersion(version, sizeof(version));
+       ul_switch_off_field();
        
        switch (len) {
-               case -1:
-                       tagtype = (UL | UL_C);
-                       break;
                case 0x0A: {
                
                        if ( version[2] == 0x03 && version[6] == 0x0B ) 
@@ -421,61 +493,31 @@ uint16_t GetHF14AMfU_Type(void){
                        
                        break;
                        }
-               case 0x01:{
-                       tagtype = UL_C; 
-                       break;
-               }
-               case 0x00: {
-                       tagtype = UL;
-                       break;
-               }
-               default :{
-                       tagtype = UNKNOWN;
-                       break;
-               }
-       }
-
-       if ((tagtype & ( UL_C | UL ))) {
-               // Magic UL-C, by observation,
-               // it seems to have a static nonce response to 0x1A command.
-               status = ul_select(&card);
-               status = ulc_requestAuthentication(0, nonce1, sizeof(nonce1));
-               if ( status > 0 ) {
-
-                       status = ulc_requestAuthentication(0, nonce2, sizeof(nonce2));
-                       
-                       tagtype =( !memcmp(nonce1, nonce2, 11) ) ? UL_C_MAGIC : UL_C;
-                               
-               } else {
-                       tagtype = UL;
-               }
+               case 0x01: tagtype = UL_C; break;
+               case 0x00: tagtype = UL; break;
+               case -1:   tagtype = (UL | UL_C); break;
+               default:   tagtype = UNKNOWN; break;
        }
 
-       //PrintAndLog("ICE %d", tagtype);
-       //Magic Ultralight test here.  It takes present UID, and tries to write it back.
-       if ( (tagtype & UL) ){
-               // read 3des key or PWD,  
-               // if response bytes == all zeros its a NORMAL tag.
-               //return UL_MAGIC;
-       }
+       if ((tagtype & ( UL_C | UL ))) tagtype = ulc_magic_test();
+       if ( (tagtype & UL) ) tagtype = ul_magic_test();
 
-       ul_switch_off_field();
        return tagtype;
 }      
 
 int CmdHF14AMfUInfo(const char *Cmd){
-       
+
+       uint8_t authlim = 0xff;
        uint8_t data[16] = {0x00};
        iso14a_card_select_t card;
        uint8_t *key;
        int status;     
 
-       PrintAndLog("\n--- Tag Information ---------");
-       PrintAndLog("-------------------------------------------------------------");
-       
        TagTypeUL_t tagtype = GetHF14AMfU_Type();
        if (tagtype == UL_ERROR) return -1;
-       
+
+       PrintAndLog("\n--- Tag Information ---------");
+       PrintAndLog("-------------------------------------------------------------");
        ul_print_type(tagtype);
        
        status = ul_select(&card);
@@ -488,33 +530,32 @@ int CmdHF14AMfUInfo(const char *Cmd){
        // read pages 0,1,2,4 (should read 4pages)
        status = ul_read(0, data, sizeof(data));
        if ( status == -1 ){
-               PrintAndLog("Error: tag didn't answer to READ");
+               PrintAndLog("Error: tag didn't answer to READ A");
                ul_switch_off_field();
                return status;
        }
        
        ul_print_default(data);
 
-       
        if ((tagtype & UL_C)){
        
+               // lookup lockbits before?
                // read pages 0x28, 0x29, 0x2A, 0x2B
+               // tag may be locked
                uint8_t ulc_conf[16] = {0x00};
                status = ul_read(0x28, ulc_conf, sizeof(ulc_conf));
                if ( status == -1 ){
-                       PrintAndLog("Error: tag didn't answer to READ");
-                       ul_switch_off_field();
-                       return status;
+                       PrintAndLog("Error: tag didn't answer to READ - possibly locked");
+               } else {
+                       ulc_print_configuration(ulc_conf);
                }
 
-               ulc_print_configuration(ulc_conf);
-
-               if ((tagtype & UL_C_MAGIC)){
+               if ((tagtype & MAGIC)){
 
                        uint8_t ulc_deskey[16] = {0x00};
                        status = ul_read(0x2C, ulc_deskey, sizeof(ulc_deskey));
                        if ( status == -1 ){
-                               PrintAndLog("Error: tag didn't answer to READ");
+                               PrintAndLog("Error: tag didn't answer to READ B");
                                ul_switch_off_field();
                                return status;
                        }
@@ -524,6 +565,7 @@ int CmdHF14AMfUInfo(const char *Cmd){
                }
                else {
                        PrintAndLog("Trying some default 3des keys");
+                       ul_switch_off_field();
                        for (uint8_t i = 0; i < 7; ++i ){
                                key = default_3des_keys[i];
                                if (try3DesAuthentication(key) == 1){
@@ -535,17 +577,17 @@ int CmdHF14AMfUInfo(const char *Cmd){
        }
        
        if ((tagtype & (UL_EV1_48 | UL_EV1_128))) {
-               
-               ulev1_print_counters();
-               
+                       
                uint8_t startconfigblock = (tagtype & UL_EV1_48) ? 0x10 : 0x25;
                uint8_t ulev1_conf[16] = {0x00};
                status = ul_read(startconfigblock, ulev1_conf, sizeof(ulev1_conf));
                if ( status == -1 ){
-                       PrintAndLog("Error: tag didn't answer to READ");
+                       PrintAndLog("Error: tag didn't answer to READ C");
                        ul_switch_off_field();
                        return status;
                }
+               // save AUTHENTICATION LIMITS for later:
+               authlim = (ulev1_conf[4] & 0x07);
                
                ulev1_print_configuration(ulev1_conf);
                
@@ -555,11 +597,10 @@ int CmdHF14AMfUInfo(const char *Cmd){
                        PrintAndLog("Error: tag didn't answer to READ SIGNATURE");
                        ul_switch_off_field();
                        return status;
-               }
-               
-               PrintAndLog(" ECC Signature : %s", sprint_hex(ulev1_signature, sizeof(ulev1_signature)));
-               PrintAndLog(" Verify it with the following 'secp128r1' for Elliptic curve cryptograghy in OpenSSL");
+               }               
+               ulev1_print_signature( ulev1_signature, sizeof(ulev1_signature));
                
+               ulev1_print_counters();
        }
        
        if ((tagtype & (UL_EV1_48 | UL_EV1_128 | NTAG_213 | NTAG_215 | NTAG_216))) {
@@ -571,22 +612,24 @@ int CmdHF14AMfUInfo(const char *Cmd){
                        ul_switch_off_field();
                        return status;
                }
-               ul_print_version(version);
+               ulev1_print_version(version);
                
-               //********** TODO ********************************
-               // --problem, there is a failed pwd tries counter in UL-EV1
-               PrintAndLog("\nTrying some known EV1/NTAG passwords.");
-                               
-               uint8_t password[4] ={0xff,0xff,0xff,0xff};
-               uint8_t pack[4] = {0,0,0,0};
-               status = ulev1_requestAuthentication(password, pack, sizeof(pack));
-               if ( status == -1 ){
-                       PrintAndLog("Error: tag didn't answer to AUTHENTICATE");
+               // AUTHLIMIT, (number of failed authentications)
+               // 0 = limitless.
+               // 1-7 = ...  should we even try then?
+               if ( authlim == 0 ){
+                       PrintAndLog("\n--- Known EV1/NTAG passwords.");
+               
+                       uint8_t pack[4] = {0,0,0,0};
+                       
+                       for (uint8_t i = 0; i < 2; ++i ){
+                               key = default_pwd_pack[i];
+                               if ( ulev1_requestAuthentication(key, pack, sizeof(pack)) > -1 ){
+                                       PrintAndLog("Found a default password: %s || Pack: %02X %02X",sprint_hex(key, 4), pack[0], pack[1]);
+                               }
+                       }
                        ul_switch_off_field();
-                       return status;
                }
-               PrintAndLog("Found default password: %s",sprint_hex(password, sizeof(password)));
-               PrintAndLog("Got PACK : %s", sprint_hex(pack,sizeof(pack)));
        }
        
        if ((tagtype & (NTAG_213 | NTAG_215 | NTAG_216))){
@@ -595,7 +638,7 @@ int CmdHF14AMfUInfo(const char *Cmd){
                uint8_t cc[16] = {0x00};
                status = ul_read(2, cc, sizeof(cc));
                if ( status == -1 ){
-                       PrintAndLog("Error: tag didn't answer to READ");
+                       PrintAndLog("Error: tag didn't answer to READ D");
                        ul_switch_off_field();
                        return status;
                }
@@ -723,6 +766,11 @@ int usage_hf_mfu_dump(void)
        PrintAndLog("and saves binary dump into the file `filename.bin` or `cardUID.bin`");
        PrintAndLog("It autodetects card type.\n");     
        PrintAndLog("Usage:  hf mfu dump k <key> n <filename w/o .bin>");
+       PrintAndLog("  Options : ");
+       PrintAndLog("  k <key> : Enter key for authentication");
+       PrintAndLog("  n <FN > : Enter filename w/o .bin to save the dump as"); 
+       PrintAndLog("        s : Swap entered key's endianness for auth");
+       PrintAndLog("");
        PrintAndLog("   sample : hf mfu dump");
        PrintAndLog("          : hf mfu dump n myfile");
        return 0;
@@ -751,10 +799,11 @@ int CmdHF14AMfUDump(const char *Cmd){
        bool tmplockbit = false;
        uint8_t dataLen=0;
        uint8_t cmdp =0;
-       //uint8_t key[16]= {0x00};
-       uint8_t *key = NULL;
+       uint8_t key[16] = {0x00};
+       uint8_t *keyPtr = key;
        size_t fileNlen = 0;
-       bool errors = FALSE;
+       bool errors = false;
+       bool swapEndian = false;
 
        while(param_getchar(Cmd, cmdp) != 0x00)
        {
@@ -769,9 +818,7 @@ int CmdHF14AMfUDump(const char *Cmd){
                        if (dataLen) {
                                errors = true; 
                        } else {
-                               //memcpy(key, data, 16);
-                               key = SwapEndian64(data, 16);
-                               PrintAndLog("3des key: %s",sprint_hex(key, 16));
+                               memcpy(key, data, 16);
                        }   
                        cmdp += 2;
                        hasPwd = true;
@@ -783,6 +830,9 @@ int CmdHF14AMfUDump(const char *Cmd){
                        if (fileNlen > FILE_PATH_SIZE-5) fileNlen = FILE_PATH_SIZE-5;
                        cmdp += 2;
                        break;
+               case 's':
+                       swapEndian = true;
+                       cmdp++;
                default:
                        PrintAndLog("Unknown parameter '%c'", param_getchar(Cmd, cmdp));
                        errors = true;
@@ -794,6 +844,9 @@ int CmdHF14AMfUDump(const char *Cmd){
        //Validations
        if(errors) return usage_hf_mfu_dump();
        
+       if (swapEndian)
+               keyPtr = SwapEndian64(data, 16);
+
        TagTypeUL_t tagtype = GetHF14AMfU_Type();
        if (tagtype == UL_ERROR) return -1;
        
@@ -828,7 +881,7 @@ int CmdHF14AMfUDump(const char *Cmd){
                PrintAndLog("Command execute time-out");
                return 1;
        }
-       
+       PrintAndLog     ("%u,%u",resp.arg[0],resp.arg[1]);
        uint8_t isOK = resp.arg[0] & 0xff;
        if (isOK) {
                memcpy(data, resp.d.asBytes, resp.arg[1]);
@@ -1159,11 +1212,11 @@ int CmdHF14AMfUCRdBl(const char *Cmd)
        UsbCommand resp;
        bool hasPwd = FALSE;
        uint8_t blockNo = -1;
-       unsigned char key[16];
+       uint8_t key[16];
        char cmdp = param_getchar(Cmd, 0);
        
        if (strlen(Cmd) < 1 || cmdp == 'h' || cmdp == 'H') {
-               PrintAndLog("Usage:  hf mfu crdbl  <block number> <password>");
+               PrintAndLog("Usage:  hf mfu crdbl  <block number> <key>");
                PrintAndLog("");
                PrintAndLog("sample: hf mfu crdbl 0");
                PrintAndLog("        hf mfu crdbl 0 00112233445566778899AABBCCDDEEFF");
@@ -1255,6 +1308,7 @@ int CmdHF14AMfUCWrBl(const char *Cmd){
        if ( blockNo <= 3 ) {
                if (!chinese_card){
                         PrintAndLog("Access Denied");  
+                       return 1;
                } else {
                        PrintAndLog("--Special block no: 0x%02x", blockNo);
                        PrintAndLog("--Data: %s", sprint_hex(bldata, 4));
@@ -1266,6 +1320,7 @@ int CmdHF14AMfUCWrBl(const char *Cmd){
                                PrintAndLog("isOk:%02x", isOK);
                        } else {
                                PrintAndLog("Command execute timeout");
+                               return 1;
                        }  
                }       
        } else {
@@ -1279,6 +1334,7 @@ int CmdHF14AMfUCWrBl(const char *Cmd){
                                PrintAndLog("isOk : %02x", isOK);
                        } else {
                                PrintAndLog("Command execute timeout");
+                               return 1;
                        }
        }
        return 0;
@@ -1532,4 +1588,4 @@ int CmdHFMFUltra(const char *Cmd){
 int CmdHelp(const char *Cmd){
        CmdsHelp(CommandTable);
        return 0;
-}
\ No newline at end of file
+}
Impressum, Datenschutz