Fix minor errors in code. 37/24437/2
authorBartlomiej Grzelewski <b.grzelewski@samsung.com>
Wed, 9 Jul 2014 12:59:56 +0000 (14:59 +0200)
committerBartlomiej Grzelewski <b.grzelewski@samsung.com>
Mon, 14 Jul 2014 10:11:40 +0000 (12:11 +0200)
* m_maxDesc was used without initialization.
* client-common module passed wrong value to poll if
  connect returns EINPROGRESS (was POLLIN, should be  POLLOUT)

Change-Id: I814a2b3779cd3fbb5a9d5db1a34f4f644c82fa94

src/server/client/client-common.cpp
src/server/main/socket-manager.cpp

index 00f099fcced4434797f38d41d29714683ded3105..5acb08be6bffa1e0488e5628ac5a3e2ea9c09bc7 100644 (file)
@@ -116,7 +116,7 @@ public:
 
         int retval = TEMP_FAILURE_RETRY(connect(m_sock, (struct sockaddr*)&clientAddr, SUN_LEN(&clientAddr)));
         if ((retval == -1) && (errno == EINPROGRESS)) {
-            if (0 >= waitForSocket(m_sock, POLLIN, POLL_TIMEOUT)) {
+            if (0 >= waitForSocket(m_sock, POLLOUT, POLL_TIMEOUT)) {
                 LogError("Error in waitForSocket.");
                 return SECURITY_SERVER_API_ERROR_SOCKET;
             }
index e7d2f3f9dd08dc4c26cb806c5f6d14bb10f4368d..7a94561864e9bb41097dbbf6615265af7fd01a68 100644 (file)
@@ -136,7 +136,8 @@ SocketManager::CreateDefaultReadSocketDescription(int sock, bool timeout)
 }
 
 SocketManager::SocketManager()
-  : m_counter(0)
+  : m_maxDesc(0)
+  , m_counter(0)
 {
     FD_ZERO(&m_readSet);
     FD_ZERO(&m_writeSet);