FIX: introduced a bug in a loop by uint8_t inside crapto1.c

[proxmark3-svn] / client / util.c

diff --git a/client/util.c b/client/util.c
index fcda9bde0ae7815bf46d1e2c0567c564f43cb526..586d930b263d68bebb653d889a75aea281969fba 100644 (file)
--- a/client/util.c
+++ b/client/util.c
@@ -97,14 +97,26 @@ void FillFileNameByUID(char *fileName, uint8_t * uid, char *ext, int byteCount)
 }
 
 // printing and converting functions
 }
 
 // printing and converting functions

-
-void print_hex(const uint8_t * data, const size_t len)
-{
+void print_hex(const uint8_t * data, const size_t len) {

        size_t i;
        size_t i;

-
-       for (i=0; i < len; i++)
+       for (i=0; i < len; ++i)

                printf("%02x ", data[i]);
                printf("%02x ", data[i]);

+       printf("\n");
+}
+void print_hex_break(const uint8_t *data, const size_t len, uint8_t breaks) {
+
+       int rownum = 0;
+       printf("[%02d] | ", rownum);
+       for (int i = 0; i < len; ++i) {

 
 

+               printf("%02X ", data[i]);
+               
+               // check if a line break is needed
+               if ( breaks > 0 && !((i+1) % breaks) && (i+1 < len) ) {
+                       ++rownum;
+                       printf("\n[%02d] | ", rownum);
+               }
+       }

        printf("\n");
 }
 
        printf("\n");
 }
 


@@ -158,9 +170,10 @@ char *sprint_bin(const uint8_t *data, const size_t len) {
 
 char *sprint_hex_ascii(const uint8_t *data, const size_t len) {
        static char buf[1024];
 
 char *sprint_hex_ascii(const uint8_t *data, const size_t len) {
        static char buf[1024];

-       memset(buf, 0x00, 1024);

        char *tmp = buf;
        char *tmp = buf;

-       sprintf(tmp, "%s| %s", sprint_hex(data, len) , data);   
+       memset(buf, 0x00, 1024);
+       size_t max_len = (len > 1010) ? 1010 : len;
+       sprintf(tmp, "%s| %s", sprint_hex(data, max_len) , data);       

        return buf;
 }
 void num_to_bytes(uint64_t n, size_t len, uint8_t* dest)
        return buf;
 }
 void num_to_bytes(uint64_t n, size_t len, uint8_t* dest)