FIX: Coverity, Identical code for different branches, CID #121315, added a messa...

[proxmark3-svn] / client / uart.c

diff --git a/client/uart.c b/client/uart.c
index f7c5e35c3b0d7ad0a04c950c0ab55d328712933b..e3a6a57b7b68f16e77fb095ee3a79d2ee64fec5b 100644 (file)
--- a/client/uart.c
+++ b/client/uart.c
@@ -73,6 +73,7 @@ serial_port uart_open(const char* pcPortName)
   // Does the system allows us to place a lock on this file descriptor
   if (fcntl(sp->fd, F_SETLK, &fl) == -1) {
     // A conflicting lock is held by another process
   // Does the system allows us to place a lock on this file descriptor
   if (fcntl(sp->fd, F_SETLK, &fl) == -1) {
     // A conflicting lock is held by another process

+    free(sp);

     return CLAIMED_SERIAL_PORT;
   }
 
     return CLAIMED_SERIAL_PORT;
   }
 


@@ -369,9 +370,9 @@ serial_port uart_open(const char* pcPortName) {
   memset(&sp->dcb, 0, sizeof(DCB));
   sp->dcb.DCBlength = sizeof(DCB);
   if(!BuildCommDCBA("baud=9600 data=8 parity=N stop=1",&sp->dcb)) {
   memset(&sp->dcb, 0, sizeof(DCB));
   sp->dcb.DCBlength = sizeof(DCB);
   if(!BuildCommDCBA("baud=9600 data=8 parity=N stop=1",&sp->dcb)) {

-    uart_close(sp);
-    return INVALID_SERIAL_PORT;
-  }
+               uart_close(sp);
+               return INVALID_SERIAL_PORT;
+       }

   
   // Update the active serial port
   if(!SetCommState(sp->hPort,&sp->dcb)) {
   
   // Update the active serial port
   if(!SetCommState(sp->hPort,&sp->dcb)) {