X-Git-Url: http://cvs.zerfleddert.de/cgi-bin/gitweb.cgi/proxmark3-svn/blobdiff_plain/52ab55ab0da1a34f4ce62d2f730e39ac099d0555..13629a71d3f77223281a2dcec5375e038419a808:/client/proxmark3.c

diff --git a/client/proxmark3.c b/client/proxmark3.c
index 8a967c7f..0fa55425 100644
--- a/client/proxmark3.c
+++ b/client/proxmark3.c
@@ -16,7 +16,7 @@
 #include <unistd.h>
 #include <readline/readline.h>
 #include <readline/history.h>
-//#include "proxusb.h"
+
 #include "proxmark3.h"
 #include "proxgui.h"
 #include "cmdmain.h"
@@ -24,7 +24,7 @@
 #include "ui.h"
 #include "sleep.h"
 #include "cmdparser.h"
-#include "cmdmain.h"
+#include "cmdhw.h"
 
 // a global mutex to prevent interlaced printing from different threads
 pthread_mutex_t print_lock;
@@ -35,30 +35,31 @@ volatile static bool txcmd_pending = false;
 
 void SendCommand(UsbCommand *c) {
 	#if 0
-		printf("Sending %d bytes\n", sizeof(UsbCommand));
+	printf("Sending %d bytes\n", sizeof(UsbCommand));
 	#endif
 
 	if (offline) {
-      PrintAndLog("Sending bytes to proxmark failed - offline");
-      return;
-    }
-  /**
+		PrintAndLog("Sending bytes to proxmark failed - offline");
+		return;
+	}
+	/**
 	The while-loop below causes hangups at times, when the pm3 unit is unresponsive
 	or disconnected. The main console thread is alive, but comm thread just spins here.
 	Not good.../holiman
 	**/
 	while(txcmd_pending);
+
 	txcmd = *c;
 	txcmd_pending = true;
 }
 
 struct receiver_arg {
-	int run;
+  int run;
 };
 
 struct main_loop_arg {
-	int usb_present;
-	char *script_cmds_file;
+  int usb_present;
+  char *script_cmds_file;
 };
 
 byte_t rx[0x1000000];
@@ -70,22 +71,24 @@ static void *uart_receiver(void *targ) {
 	size_t cmd_count;
 
 	while (arg->run) {
+
 		rxlen = sizeof(UsbCommand);
+
 		if (uart_receive(sp, prx, &rxlen)) {
 			prx += rxlen;
-			if (((prx-rx) % sizeof(UsbCommand)) != 0) {
+			if (((prx-rx) % sizeof(UsbCommand)) != 0)
 				continue;
-			}
+			
 			cmd_count = (prx-rx) / sizeof(UsbCommand);
 
-			for (size_t i = 0; i < cmd_count; i++) {
-				UsbCommandReceived((UsbCommand*)(rx+(i*sizeof(UsbCommand))));
-			}
+			for (size_t i = 0; i < cmd_count; i++)
+				UsbCommandReceived((UsbCommand*)( rx + ( i * sizeof(UsbCommand))));
+
 		}
 		prx = rx;
 
-		if(txcmd_pending) {
-			if (!uart_send(sp, (byte_t*) &txcmd, sizeof(UsbCommand))) {
+		if (txcmd_pending) {
+			if ( !uart_send(sp, (byte_t*) &txcmd, sizeof(UsbCommand))) {
 				PrintAndLog("Sending bytes to proxmark failed");
 			}
 			txcmd_pending = false;
@@ -105,16 +108,18 @@ static void *main_loop(void *targ) {
 	if (arg->usb_present == 1) {
 		rarg.run = 1;
 		pthread_create(&reader_thread, NULL, &uart_receiver, &rarg);
+		// cache Version information now:
+		CmdVersion(NULL);
 	}
 
 	FILE *script_file = NULL;
-	char script_cmd_buf[256];  // iceman, needs lua script the same file_path_buffer as the rest
+	char script_cmd_buf[256] = {0x00};  // iceman, needs lua script the same file_path_buffer as the rest
 
 	if (arg->script_cmds_file) {
 		script_file = fopen(arg->script_cmds_file, "r");
-		if (script_file) {
+		
+		if (script_file)
 			printf("using 'scripting' commands file %s\n", arg->script_cmds_file);
-		}
 	}
 
 	read_history(".history");
@@ -122,62 +127,70 @@ static void *main_loop(void *targ) {
 	while(1)  {
 
 		// If there is a script file
-		if (script_file)
-		{
+		if (script_file) {
+			
 			if (!fgets(script_cmd_buf, sizeof(script_cmd_buf), script_file)) {
 				fclose(script_file);
 				script_file = NULL;
 			} else {
 				char *nl;
 				nl = strrchr(script_cmd_buf, '\r');
-				if (nl) *nl = '\0';
+				if (nl)
+					*nl = '\0';
 				
 				nl = strrchr(script_cmd_buf, '\n');
-				if (nl) *nl = '\0';
-
-				if ((cmd = (char*) malloc(strlen(script_cmd_buf) + 1)) != NULL) {
-					memset(cmd, 0, strlen(script_cmd_buf));
+				
+				if (nl)
+					*nl = '\0';
+				
+				int newlen = strlen(script_cmd_buf);
+				if ((cmd = (char*) malloc( newlen + 1)) != NULL) {
+					memset(cmd, 0x00, newlen);
 					strcpy(cmd, script_cmd_buf);
 					printf("%s\n", cmd);
 				}
 			}
-		}
-		
-		if (!script_file) {
+		} else {
 			cmd = readline(PROXPROMPT);
 		}
 		
+		// this one should pick up all non-null cmd...
+		// why is there a 
 		if (cmd) {
 
 			while(cmd[strlen(cmd) - 1] == ' ')
 				cmd[strlen(cmd) - 1] = 0x00;
-			
+
 			if (cmd[0] != 0x00) {
-				if (strncmp(cmd, "quit", 4) == 0) {
-					exit(0);
-					break;
-				}
-				CommandReceived(cmd);
+				int ret = CommandReceived(cmd);
 				add_history(cmd);
+				
+				// exit or quit
+				if (ret == 99) 
+					break;
 			}
 			free(cmd);
+			cmd = 0;
 		} else {
 			printf("\n");
 			break;
 		}
 	}
-  
-	write_history(".history");
-  
-	if (arg->usb_present == 1) {
-		rarg.run = 0;
-		pthread_join(reader_thread, NULL);
-	}
 
 	if (script_file) {
 		fclose(script_file);
 		script_file = NULL;
 	}
+	
+	write_history(".history");
+
+	free(cmd);
+	cmd = 0;
+			
+	if (arg->usb_present == 1) {
+		rarg.run = 0;
+		pthread_join(reader_thread, NULL);
+	}
 
 	ExitGraphics();
 	pthread_exit(NULL);
@@ -186,12 +199,12 @@ static void *main_loop(void *targ) {
 
 static void dumpAllHelp(int markdown)
 {
-  printf("\n%sProxmark3 command dump%s\n\n",markdown?"# ":"",markdown?"":"\n======================");
-  printf("Some commands are available only if a Proxmark is actually connected.%s\n",markdown?"  ":"");
-  printf("Check column \"offline\" for their availability.\n");
-  printf("\n");
-  command_t *cmds = getTopLevelCommandTable();
-  dumpCommandsRecursive(cmds, markdown);
+	printf("\n%sProxmark3 command dump%s\n\n",markdown?"# ":"",markdown?"":"\n======================");
+	printf("Some commands are available only if a Proxmark is actually connected.%s\n",markdown?"  ":"");
+	printf("Check column \"offline\" for their availability.\n");
+	printf("\n");
+	command_t *cmds = getTopLevelCommandTable();
+	dumpCommandsRecursive(cmds, markdown);
 }
 
 int main(int argc, char* argv[]) {
@@ -221,7 +234,8 @@ int main(int argc, char* argv[]) {
 		.usb_present = 0,
 		.script_cmds_file = NULL
 	};
-	pthread_t main_loop_t;
+	
+	pthread_t main_loop_threat;
 
   
 	sp = uart_open(argv[1]);
@@ -249,25 +263,27 @@ int main(int argc, char* argv[]) {
 			printf("Output will be flushed after every print.\n");
 			flushAfterWrite = 1;
 		}
-		else
-		marg.script_cmds_file = argv[2];
+		else {
+			marg.script_cmds_file = argv[2];
+		}
 	}
 
 	// create a mutex to avoid interlacing print commands from our different threads
 	pthread_mutex_init(&print_lock, NULL);
 
-	pthread_create(&main_loop_t, NULL, &main_loop, &marg);
+	pthread_create(&main_loop_threat, NULL, &main_loop, &marg);
 	InitGraphics(argc, argv);
 
 	MainGraphics();
 
-	pthread_join(main_loop_t, NULL);
+	pthread_join(main_loop_threat, NULL);
 
 	// Clean up the port
-	uart_close(sp);
+	if (offline == 0)
+		uart_close(sp);
   
 	// clean up mutex
 	pthread_mutex_destroy(&print_lock);
-  
-  return 0;
+	
+	exit(0);
 }