Fix memory bounds error

[proxmark3-svn] / client / cmdmain.c

diff --git a/client/cmdmain.c b/client/cmdmain.c
index c1d730ee636f13cf92b01f59b71b607ad1791a20..739d68e1cbc0beb609f0d858dae78a3ad1438b95 100644 (file)
--- a/client/cmdmain.c
+++ b/client/cmdmain.c
@@ -8,11 +8,13 @@
 // Main command parser entry point
 //-----------------------------------------------------------------------------
 
 // Main command parser entry point
 //-----------------------------------------------------------------------------
 

+#include "cmdmain.h"
+
+#include <pthread.h>

 #include <stdio.h>
 #include <stdlib.h>
 #include <unistd.h>
 #include <string.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <unistd.h>
 #include <string.h>

-#include "sleep.h"

 #include "cmdparser.h"
 #include "proxmark3.h"
 #include "data.h"
 #include "cmdparser.h"
 #include "proxmark3.h"
 #include "data.h"


@@ -22,15 +24,17 @@
 #include "cmddata.h"
 #include "cmdhw.h"
 #include "cmdlf.h"
 #include "cmddata.h"
 #include "cmdhw.h"
 #include "cmdlf.h"

-#include "cmdmain.h"

 #include "util.h"
 #include "util.h"

+#include "util_posix.h"

 #include "cmdscript.h"
 #include "cmdscript.h"

+#include "cmdcrc.h"

 
 
 unsigned int current_command = CMD_UNKNOWN;
 
 static int CmdHelp(const char *Cmd);
 static int CmdQuit(const char *Cmd);
 
 
 unsigned int current_command = CMD_UNKNOWN;
 
 static int CmdHelp(const char *Cmd);
 static int CmdQuit(const char *Cmd);

+static int CmdRev(const char *Cmd);

 
 //For storing command that are received from the device
 #define CMD_BUFFER_SIZE 50
 
 //For storing command that are received from the device
 #define CMD_BUFFER_SIZE 50


@@ -39,18 +43,21 @@ static UsbCommand cmdBuffer[CMD_BUFFER_SIZE];
 static int cmd_head;//Starts as 0
 //Points to the position of the last unread command
 static int cmd_tail;//Starts as 0
 static int cmd_head;//Starts as 0
 //Points to the position of the last unread command
 static int cmd_tail;//Starts as 0

+// to lock cmdBuffer operations from different threads
+static pthread_mutex_t cmdBufferMutex = PTHREAD_MUTEX_INITIALIZER;

 
 static command_t CommandTable[] = 
 {
 
 static command_t CommandTable[] = 
 {

-       {"help",        CmdHelp,        1, "This help. Use '<command> help' for details of a particular command."},
-       {"data",        CmdData,        1, "{ Plot window / data buffer manipulation... }"},
-       {"hf",          CmdHF,          1, "{ High Frequency commands... }"},
-       {"hw",          CmdHW,          1, "{ Hardware commands... }"},
-       {"lf",          CmdLF,          1, "{ Low Frequency commands... }"},
-       {"script",      CmdScript,      1, "{ Scripting commands }"},
-       {"quit",        CmdQuit,        1, "Exit program"},
-       {"exit",        CmdQuit,        1, "Exit program"},
-       {NULL, NULL, 0, NULL}
+  {"help",  CmdHelp,  1, "This help. Use '<command> help' for details of a particular command."},
+  {"data",  CmdData,  1, "{ Plot window / data buffer manipulation... }"},
+  {"hf",    CmdHF,    1, "{ High Frequency commands... }"},
+  {"hw",    CmdHW,    1, "{ Hardware commands... }"},
+  {"lf",    CmdLF,    1, "{ Low Frequency commands... }"},
+  {"reveng",CmdRev,   1, "Crc calculations from the software reveng1-30"},
+  {"script",CmdScript,1, "{ Scripting commands }"},
+  {"quit",  CmdQuit,  1, "Exit program"},
+  {"exit",  CmdQuit,  1, "Exit program"},
+  {NULL, NULL, 0, NULL}

 };
 
 command_t* getTopLevelCommandTable()
 };
 
 command_t* getTopLevelCommandTable()


@@ -67,6 +74,13 @@ int CmdQuit(const char *Cmd)
 {
   return 99;
 }
 {
   return 99;
 }

+
+int CmdRev(const char *Cmd)
+{
+  CmdCrc(Cmd);
+  return 0;
+}
+

 /**
  * @brief This method should be called when sending a new command to the pm3. In case any old
  *  responses from previous commands are stored in the buffer, a call to this method should clear them.
 /**
  * @brief This method should be called when sending a new command to the pm3. In case any old
  *  responses from previous commands are stored in the buffer, a call to this method should clear them.


@@ -76,7 +90,9 @@ int CmdQuit(const char *Cmd)
 void clearCommandBuffer()
 {
     //This is a very simple operation
 void clearCommandBuffer()
 {
     //This is a very simple operation

+       pthread_mutex_lock(&cmdBufferMutex);

     cmd_tail = cmd_head;
     cmd_tail = cmd_head;

+       pthread_mutex_unlock(&cmdBufferMutex);

 }
 
 /**
 }
 
 /**


@@ -85,6 +101,7 @@ void clearCommandBuffer()
  */
 void storeCommand(UsbCommand *command)
 {
  */
 void storeCommand(UsbCommand *command)
 {

+       pthread_mutex_lock(&cmdBufferMutex);

     if( ( cmd_head+1) % CMD_BUFFER_SIZE == cmd_tail)
     {
         //If these two are equal, we're about to overwrite in the
     if( ( cmd_head+1) % CMD_BUFFER_SIZE == cmd_tail)
     {
         //If these two are equal, we're about to overwrite in the


@@ -96,6 +113,7 @@ void storeCommand(UsbCommand *command)
     memcpy(destination, command, sizeof(UsbCommand));
 
     cmd_head = (cmd_head +1) % CMD_BUFFER_SIZE; //increment head and wrap
     memcpy(destination, command, sizeof(UsbCommand));
 
     cmd_head = (cmd_head +1) % CMD_BUFFER_SIZE; //increment head and wrap

+       pthread_mutex_unlock(&cmdBufferMutex);

 }
 
 
 }
 
 


@@ -106,8 +124,10 @@ void storeCommand(UsbCommand *command)
  */
 int getCommand(UsbCommand* response)
 {
  */
 int getCommand(UsbCommand* response)
 {

+       pthread_mutex_lock(&cmdBufferMutex);

     //If head == tail, there's nothing to read, or if we just got initialized
     if(cmd_head == cmd_tail){
     //If head == tail, there's nothing to read, or if we just got initialized
     if(cmd_head == cmd_tail){

+               pthread_mutex_unlock(&cmdBufferMutex);

         return 0;
     }
     //Pick out the next unread command
         return 0;
     }
     //Pick out the next unread command


@@ -115,7 +135,7 @@ int getCommand(UsbCommand* response)
     memcpy(response, last_unread, sizeof(UsbCommand));
     //Increment tail - this is a circular buffer, so modulo buffer size
     cmd_tail = (cmd_tail +1 ) % CMD_BUFFER_SIZE;
     memcpy(response, last_unread, sizeof(UsbCommand));
     //Increment tail - this is a circular buffer, so modulo buffer size
     cmd_tail = (cmd_tail +1 ) % CMD_BUFFER_SIZE;

-
+       pthread_mutex_unlock(&cmdBufferMutex);

     return 1;
 }
 
     return 1;
 }
 


@@ -129,7 +149,7 @@ int getCommand(UsbCommand* response)
  * @param ms_timeout
  * @return true if command was returned, otherwise false
  */
  * @param ms_timeout
  * @return true if command was returned, otherwise false
  */

-bool WaitForResponseTimeout(uint32_t cmd, UsbCommand* response, size_t ms_timeout) {
+bool WaitForResponseTimeoutW(uint32_t cmd, UsbCommand* response, size_t ms_timeout, bool show_warning) {

   
        UsbCommand resp;
        
   
        UsbCommand resp;
        


@@ -137,25 +157,34 @@ bool WaitForResponseTimeout(uint32_t cmd, UsbCommand* response, size_t ms_timeou
                response = &resp;
        }
 
                response = &resp;
        }
 

+       uint64_t start_time = msclock();
+       

        // Wait until the command is received
        // Wait until the command is received

-       for(size_t dm_seconds=0; dm_seconds < ms_timeout/10; dm_seconds++) {
+       while (true) {

                while(getCommand(response)) {
                        if(response->cmd == cmd){
                                return true;
                        }
                }
                while(getCommand(response)) {
                        if(response->cmd == cmd){
                                return true;
                        }
                }

-               msleep(10); // XXX ugh
-               if (dm_seconds == 200) { // Two seconds elapsed
+               if (msclock() - start_time > ms_timeout) {
+                       break;
+               }
+               if (msclock() - start_time > 2000 && show_warning) {

                        PrintAndLog("Waiting for a response from the proxmark...");
                        PrintAndLog("Don't forget to cancel its operation first by pressing on the button");
                        PrintAndLog("Waiting for a response from the proxmark...");
                        PrintAndLog("Don't forget to cancel its operation first by pressing on the button");

+                       break;

                }
        }
        return false;
 }
 
 
                }
        }
        return false;
 }
 
 

+bool WaitForResponseTimeout(uint32_t cmd, UsbCommand* response, size_t ms_timeout) {
+       return WaitForResponseTimeoutW(cmd, response, ms_timeout, true);
+}
+

 bool WaitForResponse(uint32_t cmd, UsbCommand* response) {
 bool WaitForResponse(uint32_t cmd, UsbCommand* response) {

-       return WaitForResponseTimeout(cmd,response,-1);
+       return WaitForResponseTimeoutW(cmd, response, -1, true);

 }
 
 
 }