Change mutex lock model to avoid dead lock and ensure locks are always released.

Additional cosmetic fixes:
 move 'name' as protected
 remove unnecessary local var
 Fix debug log

src/device/device.hpp

@@ -78,8 +78,10 @@ namespace hw {
            return false;
     }
 
-
     class device {
+    protected:
+        std::string  name;
+
     public:
 
         device()  {}
@@ -92,8 +94,6 @@ namespace hw {
         static const int SIGNATURE_FAKE = 1;
 
 
-        std::string  name;
-
         /* ======================================================================= */
         /*                              SETUP/TEARDOWN                             */
         /* ======================================================================= */
@@ -104,7 +104,15 @@ namespace hw {
         virtual bool release() = 0;
 
         virtual bool connect(void) = 0;
-        virtual bool disconnect() = 0;
+        virtual bool disconnect(void) = 0;
+
+        /* ======================================================================= */
+        /*  LOCKER                                                                 */
+        /* ======================================================================= */ 
+        virtual void lock(void) = 0;
+        virtual void unlock(void) = 0;
+        virtual bool try_lock(void) = 0;
+
 
         /* ======================================================================= */
         /*                             WALLET & ADDRESS                            */

src/device/device_default.cpp

@@ -82,6 +82,17 @@ namespace hw {
             dfns();
         }
 
+
+        /* ======================================================================= */
+        /*  LOCKER                                                                 */
+        /* ======================================================================= */ 
+    
+        void device_default::lock() { }
+
+        bool device_default::try_lock() { return true; }
+
+        void device_default::unlock() { }
+
         /* ======================================================================= */
         /*                             WALLET & ADDRESS                            */
         /* ======================================================================= */

src/device/device_default.hpp

@@ -59,6 +59,13 @@ namespace hw {
             bool connect(void) override;
             bool disconnect() override;
 
+            /* ======================================================================= */
+            /*  LOCKER                                                                 */
+            /* ======================================================================= */ 
+            void lock(void)  override;
+            void unlock(void) override;
+            bool try_lock(void) override;
+            
             /* ======================================================================= */
             /*                             WALLET & ADDRESS                            */
             /* ======================================================================= */

src/device/device_ledger.hpp

@@ -36,7 +36,8 @@
 #include "device.hpp"
 #include <PCSC/winscard.h>
 #include <PCSC/wintypes.h>
-
+#include <boost/thread/mutex.hpp>
+#include <boost/thread/recursive_mutex.hpp>
 
 namespace hw {
 
@@ -80,13 +81,11 @@ namespace hw {
 
     class device_ledger : public hw::device {
     private:
-        mutable std::mutex   device_locker;
+        // Locker for concurrent access
-        mutable std::mutex   tx_locker;
+        mutable boost::recursive_mutex   device_locker;
-        void lock_device() ;
+        mutable boost::mutex   command_locker;
-        void unlock_device() ;
+
-        void lock_tx() ;
+        //PCSC management 
-        void unlock_tx() ;
- 
         std::string  full_name;
         SCARDCONTEXT hContext;
         SCARDHANDLE  hCard;
@@ -130,6 +129,14 @@ namespace hw {
         bool connect(void) override;
         bool disconnect() override;
 
+
+        /* ======================================================================= */
+        /*  LOCKER                                                                 */
+        /* ======================================================================= */ 
+        void lock(void)  override;
+        void unlock(void) override;
+        bool try_lock(void) override;
+
         /* ======================================================================= */
         /*                             WALLET & ADDRESS                            */
         /* ======================================================================= */