Neither the Bu::Client class or the Bu::ProtocolTelnet class were threadsafe

and both really need to be.  Bu::Client is used in multi-threaded applications
and has been for a while, so it's a no brainer.  It may need a little bit more
safety added around variable access, but all the buffers are safe now.

ProtocolTelnet has it's own issus, and I should probably rework a few parts of
it before too long so we can better accomidate things like line editing.

1 files changed, 3 insertions, 0 deletions

diff --git a/src/stable/protocoltelnet.h b/src/stable/protocoltelnet.h
index 60faef4..19298dc 100644
--- a/src/stable/protocoltelnet.h
+++ b/src/stable/protocoltelnet.h
@@ -11,6 +11,7 @@
 #include "bu/protocol.h"
 #include "bu/hash.h"
 #include "bu/string.h"
+#include "bu/mutex.h"
 
 // #define __TELNET_DEBUG
 
@@ -210,6 +211,8 @@ namespace Bu
 #endif
 
     private:
+        Mutex mRead;
+        Mutex mWrite;
         Client *pClient;
 
         Bu::String sDataBuf;    /**< Buffer for regular line data. */