From fe7bfa785696615de44fd896214182446a664646 Mon Sep 17 00:00:00 2001 From: "roel@libnfc.org" Date: Fri, 7 Dec 2012 12:51:19 +0000 Subject: [PATCH] Windows7 working now! ugly fix for inaccurate Windows timer, quickfix for receiving multiple frames --- client/cmdmain.c | 6 +++--- client/proxmark3.c | 15 ++++++++------- 2 files changed, 11 insertions(+), 10 deletions(-) diff --git a/client/cmdmain.c b/client/cmdmain.c index edff9f8e..434f9e6e 100644 --- a/client/cmdmain.c +++ b/client/cmdmain.c @@ -59,9 +59,9 @@ int CmdQuit(const char *Cmd) bool WaitForResponseTimeout(uint32_t cmd, UsbCommand* response, size_t ms_timeout) { // Wait until the command is received - for(size_t i=0; received_command != cmd && i < ms_timeout; i++) { - msleep(1); // XXX ugh - if (i == 2000) { + for(size_t i=0; received_command != cmd && i < ms_timeout/10; i++) { + msleep(10); // XXX ugh + if (i == 200) { // Two seconds elapsed PrintAndLog("Waiting for a response from the proxmark..."); PrintAndLog("Don't forget to cancel its operation first by pressing on the button"); } diff --git a/client/proxmark3.c b/client/proxmark3.c index 38a97716..949013ec 100644 --- a/client/proxmark3.c +++ b/client/proxmark3.c @@ -67,6 +67,7 @@ struct main_loop_arg { //} byte_t rx[0x1000000]; +byte_t* prx = rx; static void *uart_receiver(void *targ) { struct receiver_arg *arg = (struct receiver_arg*)targ; @@ -75,18 +76,19 @@ static void *uart_receiver(void *targ) { while (arg->run) { rxlen = sizeof(UsbCommand); - if (uart_receive(sp,rx,&rxlen)) { - if ((rxlen % sizeof(UsbCommand)) != 0) { - PrintAndLog("ERROR: received %03zd bytes, which does not seem to be one or more command(s)\n",rxlen ); + if (uart_receive(sp,prx,&rxlen)) { + prx += rxlen; + if (((prx-rx) % sizeof(UsbCommand)) != 0) { continue; } - cmd_count = rxlen / sizeof(UsbCommand); -// printf("received %zd bytes, which represents %zd commands\n",rxlen, cmd_count); + cmd_count = (prx-rx) / sizeof(UsbCommand); + // printf("received %d bytes, which represents %d commands\n",(prx-rx), cmd_count); for (size_t i=0; i