Merge pull request #39 from RfidResearchGroup/master

Update from master

armsrc/Standalone/hf_legic.c

@@ -60,7 +60,7 @@ void save_dump_to_file(legic_card_select_t *p_card) {
     // legic functions puts it memory in Emulator reserved memory.
     uint8_t *mem = BigBuf_get_EM_addr();
 
-    char *preferredName = (char*)BigBuf_malloc(30);
+    char *preferredName = (char *)BigBuf_malloc(30);
     if (preferredName == NULL) {
         goto OUT;
     }
@@ -68,7 +68,7 @@ void save_dump_to_file(legic_card_select_t *p_card) {
     sprintf(preferredName, "hf-legic-%02X%02X%02X%02X-dump", p_card->uid[0], p_card->uid[1], p_card->uid[2], p_card->uid[3]);
     uint16_t preferredNameLen = strlen(preferredName);
 
-    char *filename = (char*)BigBuf_malloc(preferredNameLen + 4 + 1 + 10);
+    char *filename = (char *)BigBuf_malloc(preferredNameLen + 4 + 1 + 10);
     if (filename == NULL) {
         goto OUT;
     }
@@ -146,8 +146,8 @@ void RunMod() {
 
             LED_D_ON();
             uint8_t ct;
-            switch(p_card->tagtype) {
-               case 0x0D:
+            switch (p_card->tagtype) {
+                case 0x0D:
                     ct = 0;
                     break;
                 case 0x1D:

armsrc/hitagS.c

@@ -977,7 +977,7 @@ void SimulateHitagSTag(bool tag_mem_supplied, uint8_t *data) {
     // TC1: Capture mode, default timer source = MCK/2 (TIMER_CLOCK1), TIOA is external trigger,
     // external trigger rising edge, load RA on rising edge of TIOA.
     AT91C_BASE_TC1->TC_CMR = AT91C_TC_CLKS_TIMER_DIV1_CLOCK
-                              | AT91C_TC_ETRGEDG_RISING | AT91C_TC_ABETRG | AT91C_TC_LDRA_RISING;
+                             | AT91C_TC_ETRGEDG_RISING | AT91C_TC_ABETRG | AT91C_TC_LDRA_RISING;
 
     // Enable and reset counter
     AT91C_BASE_TC0->TC_CCR = AT91C_TC_CLKEN | AT91C_TC_SWTRG;

armsrc/legicrf.c

@@ -404,7 +404,7 @@ bool write_byte(uint16_t index, uint8_t byte, uint8_t addr_sz) {
 //
 // Only this functions are public / called from appmain.c
 //-----------------------------------------------------------------------------
-legic_card_select_t* getLegicCardInfo(void) {
+legic_card_select_t *getLegicCardInfo(void) {
     return &card;
 }
 

armsrc/legicrf.h

@@ -21,5 +21,5 @@ int LegicRfReaderEx(uint16_t offset, uint16_t len, uint8_t iv);
 void LegicRfReader(uint16_t offset, uint16_t len, uint8_t iv);
 void LegicRfWriter(uint16_t offset, uint16_t len, uint8_t iv, uint8_t *data);
 
-legic_card_select_t* getLegicCardInfo(void);
+legic_card_select_t *getLegicCardInfo(void);
 #endif /* __LEGICRF_H */

armsrc/lfops.c

@@ -2405,7 +2405,7 @@ void SendForward(uint8_t fwd_bit_count) {
             WaitUS(32 * 8);
         } else {
             TurnReadLF_off(23 * 8);
-            TurnReadLFOn((32-23) * 8);
+            TurnReadLFOn((32 - 23) * 8);
         }
     }
 }

armsrc/mifaredesfire.c

@@ -334,16 +334,16 @@ void MifareDES_Auth1(uint8_t *datain) {
         return;
     }
 
-    int rndlen=8;
+    int rndlen = 8;
     int expectedlen = 1 + 8 + 2 + 2;
     if (payload->algo == MFDES_ALGO_AES || payload->algo == MFDES_ALGO_3K3DES) {
         expectedlen = 1 + 16 + 2 + 2;
-        rndlen=16;
+        rndlen = 16;
     }
 
     if (payload->mode == MFDES_AUTH_PICC) {
         expectedlen = 1 + 1 + 8 + 2;
-        rndlen=8;
+        rndlen = 8;
     }
 
     if (len != expectedlen) {

armsrc/util.h

@@ -15,7 +15,7 @@
 
 // PRIx64 definition missing with gcc-arm-none-eabi v8?
 #ifndef PRIx64
-    #define PRIx64 "llx"
+#define PRIx64 "llx"
 #endif
 
 // Basic macros

client/Makefile

@@ -17,7 +17,7 @@ include ../Makefile.defs
 INSTALLBIN = proxmark3
 INSTALLSHARE = cmdscripts lualibs luascripts resources dictionaries
 
-VPATH = ../common uart
+VPATH =  ../common src/uart src deps
 vpath %.dic dictionaries
 OBJDIR = obj
 
@@ -32,15 +32,15 @@ ifneq ($(platform),Darwin)
 endif
 
 # local libraries
-LUALIBPATH = ./liblua
+LUALIBPATH = ./deps/liblua
 LUALIB = $(LUALIBPATH)/liblua.a
-JANSSONLIBPATH = ./jansson
+JANSSONLIBPATH = ./deps/jansson
 JANSSONLIB = $(JANSSONLIBPATH)/libjansson.a
-CBORLIBPATH = ./tinycbor
+CBORLIBPATH = ./deps/tinycbor
 CBORLIB = $(CBORLIBPATH)/tinycbor.a
-REVENGPATH = ./reveng
+REVENGPATH = ./deps/reveng
 REVENGLIB = $(REVENGPATH)/libreveng.a
-AMIIBOLIBPATH = ./amiitool
+AMIIBOLIBPATH = ./deps/amiitool
 AMIIBOLIB = $(AMIIBOLIBPATH)/libamiibo.a
 
 # common libraries
@@ -50,7 +50,7 @@ ZLIBPATH = ../common/zlib
 ZLIB = $(OBJDIR)/libz.a
 
 LIBS = -I$(LUALIBPATH) -I$(MBEDTLSLIBPATH) -I$(JANSSONLIBPATH) -I$(CBORLIBPATH) -I$(ZLIBPATH) -I$(REVENGPATH) -I$(AMIIBOLIBPATH)
-INCLUDES_CLIENT = -I. -I../include -I../common -Iuart $(LIBS)
+INCLUDES_CLIENT = -I./src -I./deps -I../include -I../common -I./deps/cliparser -I./src/uart $(LIBS)
 CFLAGS ?= -Wall -Werror -g -O3
 # We cannot just use CFLAGS+=... because it has impact on sub-makes if CFLAGS is defined in env:
 PM3CFLAGS = $(CFLAGS) -std=c99 -D_ISOC99_SOURCE $(INCLUDES_CLIENT)
@@ -59,7 +59,7 @@ ifneq (,$(findstring MINGW,$(platform)))
     PM3CFLAGS += -mno-ms-bitfields -fexec-charset=cp850
 endif
 CXXFLAGS ?= -Wall -Werror -O3
-PM3CXXFLAGS = $(CXXFLAGS) -I../include
+PM3CXXFLAGS = $(CXXFLAGS) -I../include -I/.deps/cliparser
 
 LUAPLATFORM = generic
 ifneq (,$(findstring MINGW,$(platform)))
@@ -130,6 +130,7 @@ CORESRCS =  uart_posix.c \
             util_posix.c \
             scandir.c \
             crc16.c \
+            crc32.c \
             comms.c
 
 CMDSRCS =   crapto1/crapto1.c \
@@ -200,7 +201,7 @@ CMDSRCS =   crapto1/crapto1.c \
             cmdhfmfu.c \
             cmdhfmfp.c \
             cmdhfmfhard.c \
-            hardnested/hardnested_bruteforce.c \
+            deps/hardnested/hardnested_bruteforce.c \
             cmdhfmfdes.c \
             cmdhftopaz.c \
             cmdhffido.c \
@@ -258,13 +259,13 @@ CMDSRCS =   crapto1/crapto1.c \
 
 cpu_arch = $(shell uname -m)
 ifneq ($(findstring 86, $(cpu_arch)), )
-    MULTIARCHSRCS = hardnested/hardnested_bf_core.c hardnested/hardnested_bitarray_core.c
+    MULTIARCHSRCS = deps/hardnested/hardnested_bf_core.c deps/hardnested/hardnested_bitarray_core.c
 endif
 ifneq ($(findstring amd64, $(cpu_arch)), )
-    MULTIARCHSRCS = hardnested/hardnested_bf_core.c hardnested/hardnested_bitarray_core.c
+    MULTIARCHSRCS = deps/hardnested/hardnested_bf_core.c deps/hardnested/hardnested_bitarray_core.c
 endif
 ifeq ($(MULTIARCHSRCS), )
-    CMDSRCS += hardnested/hardnested_bf_core.c hardnested/hardnested_bitarray_core.c
+    CMDSRCS += deps/hardnested/hardnested_bf_core.c deps/hardnested/hardnested_bitarray_core.c
 endif
 
 
@@ -310,13 +311,13 @@ proxmark3: $(OBJDIR)/proxmark3.o $(COREOBJS) $(CMDOBJS) $(OBJCOBJS) $(QTGUIOBJS)
 	$(info [=] LD $@)
 	$(Q)$(LD) $(LDFLAGS) $(OBJDIR)/proxmark3.o $(COREOBJS) $(CMDOBJS) $(OBJCOBJS) $(QTGUIOBJS) $(MULTIARCHOBJS) $(LDLIBS)  -o $@
 
-proxgui.cpp: ui/ui_overlays.h
+src/proxgui.cpp: src/ui/ui_overlays.h
 
-proxguiqt.moc.cpp: proxguiqt.h
+src/proxguiqt.moc.cpp: src/proxguiqt.h
 	$(info [-] MOC $@)
 	$(Q)$(MOC) -o$@ $^
 
-ui/ui_overlays.h: ui/overlays.ui
+src/ui/ui_overlays.h: src/ui/overlays.ui
 	$(info [-] UIC $@)
 	$(Q)$(UIC) $^ > $@
 

client/deps/amiitool/Makefile

@@ -1,5 +1,5 @@
 MYSRCPATHS =
-MYINCLUDES = -I. -I.. -I../jansson -I../../common/ -I../../include/
+MYINCLUDES = -I. -I.. -I../jansson -I../../../common -I../../../include
 MYCFLAGS = -std=c99 -D_ISOC99_SOURCE
 MYDEFS =
 MYSRCS = \
@@ -9,11 +9,11 @@ MYSRCS = \
 
 LIB_A = libamiibo.a
 
-include ../../Makefile.host
+include ../../../Makefile.host
 
 # just for testing amiitool before complete migration into a lib:
 
 amiitool:
 	gcc $(CFLAGS) \
-	  amiitool.c $(MYSRCS) ../../common/commonutil.c ../ui.c -lreadline -lm ../../common/mbedtls/libmbedtls.a \
+	  amiitool.c $(MYSRCS) ../../../common/commonutil.c ../ui.c -lreadline -lm ../../../common/mbedtls/libmbedtls.a \
 	  -o amiitool

client/deps/amiitool/amiitool.c

@@ -7,7 +7,7 @@
 
 #include <stdio.h>
 #include <string.h>
-#include "fileutils.h"
+#include "src/fileutils.h"
 #include "amiibo.h"
 #include "getopt.h"
 

client/deps/hardnested/hardnested_bf_core.c

@@ -59,8 +59,8 @@ THE SOFTWARE.
 #include <string.h>
 #include "crapto1/crapto1.h"
 #include "parity.h"
-#include "util.h"
-#include "common.h"
+//#include "util.h"
+//#include "common.h"
 
 // bitslice type
 // while AVX supports 256 bit vector floating point operations, we need integer operations for boolean logic

client/deps/hardnested/hardnested_bruteforce.h

@@ -19,7 +19,33 @@
 
 #include <stdint.h>
 #include <stdbool.h>
-#include "cmdhfmfhard.h"
+
+#define NUM_SUMS 19 // number of possible sum property values
+
+typedef struct guess_sum_a8 {
+    float prob;
+    uint64_t num_states;
+    uint8_t sum_a8_idx;
+} guess_sum_a8_t;
+
+typedef struct noncelistentry {
+    uint32_t nonce_enc;
+    uint8_t par_enc;
+    void *next;
+} noncelistentry_t;
+
+typedef struct noncelist {
+    uint16_t num;
+    uint16_t Sum;
+    guess_sum_a8_t sum_a8_guess[NUM_SUMS];
+    bool sum_a8_guess_dirty;
+    float expected_num_brute_force;
+    uint8_t BitFlips[0x400];
+    uint32_t *states_bitarray[2];
+    uint32_t num_states_bitarray[2];
+    bool all_bitflips_dirty[2];
+    noncelistentry_t *first;
+} noncelist_t;
 
 typedef struct {
     uint32_t *states[2];

client/deps/jansson/Makefile

@@ -18,4 +18,4 @@ MYSRCS = \
 
 LIB_A = libjansson.a
 
-include ../../Makefile.host
+include ../../../Makefile.host

client/deps/jansson/hashtable.c

@@ -16,7 +16,7 @@
 #include <stdint.h>
 #endif
 
-#include <jansson_config.h>   /* for JSON_INLINE */
+#include "jansson_config.h"   /* for JSON_INLINE */
 #include "jansson_private.h"  /* for container_of() */
 #include "hashtable.h"
 

client/deps/jansson/path.c

@@ -10,7 +10,7 @@
 #include <string.h>
 #include <assert.h>
 
-#include <jansson.h>
+#include "jansson.h"
 #include "jansson_private.h"
 
 json_t *json_path_get(const json_t *json, const char *path) {

client/deps/liblua/Makefile

@@ -21,7 +21,7 @@ PLATS= aix ansi bsd freebsd generic linux macosx mingw posix solaris
 
 default: $(PLAT)
 
-include ../../Makefile.host
+include ../../../Makefile.host
 
 # Convenience targets for popular platforms
 ALL= all