Renamed class SocketBuffer to MessageBuffer and all its usages.
authorAdam Malinowski <a.malinowsk2@samsung.com>
Thu, 19 Sep 2013 12:42:26 +0000 (14:42 +0200)
committerBartlomiej Grzelewski <b.grzelewski@samsung.com>
Thu, 6 Feb 2014 16:13:22 +0000 (17:13 +0100)
[Issue#] SSDWSSP-480 /1
[Feature] SocketBuffer class needed to be renamed to MessageBuffer.
[Cause] SocketBuffer is not right name for this class.
[Solution] Class was renamed.
[Verification] Build security-server and run security server tests on target.

Change-Id: I8fce6aef14308389e1148599e89b3e11939a36fd

29 files changed:
src/server2/CMakeLists.txt
src/server2/client/client-app-permissions.cpp
src/server2/client/client-common.cpp
src/server2/client/client-common.h
src/server2/client/client-cookie.cpp
src/server2/client/client-get-gid.cpp
src/server2/client/client-get-object-name.cpp
src/server2/client/client-privilege-by-pid.cpp
src/server2/client/client-shared-memory.cpp
src/server2/client/client-socket-privilege.cpp
src/server2/common/message-buffer.cpp [new file with mode: 0644]
src/server2/common/message-buffer.h [new file with mode: 0644]
src/server2/common/socket-buffer.cpp [deleted file]
src/server2/common/socket-buffer.h [deleted file]
src/server2/service/app-permissions.cpp
src/server2/service/app-permissions.h
src/server2/service/cookie.cpp
src/server2/service/cookie.h
src/server2/service/data-share.cpp
src/server2/service/data-share.h
src/server2/service/echo.h
src/server2/service/exec-path.cpp
src/server2/service/exec-path.h
src/server2/service/get-gid.cpp
src/server2/service/get-gid.h
src/server2/service/get-object-name.cpp
src/server2/service/get-object-name.h
src/server2/service/privilege-by-pid.cpp
src/server2/service/privilege-by-pid.h

index 6bdceb478fe5ef70d8cc1c02486caf1bce650ed2..154bf18fc6c6338cb1b793308d0b2a29fafb525b 100644 (file)
@@ -7,7 +7,7 @@ SET(COMMON_PATH ${PROJECT_SOURCE_DIR}/src/server2)
 
 SET(COMMON_SOURCES
     ${COMMON_PATH}/common/protocols.cpp
-    ${COMMON_PATH}/common/socket-buffer.cpp
+    ${COMMON_PATH}/common/message-buffer.cpp
     ${COMMON_PATH}/dpl/log/src/abstract_log_provider.cpp
     ${COMMON_PATH}/dpl/log/src/dlog_log_provider.cpp
     ${COMMON_PATH}/dpl/log/src/log.cpp
index 764ee092888732a31957e32e2b1b7c3b805c99d8..9a0703b8e067bfe89d3312b1a41914cff3dc7101 100644 (file)
@@ -29,7 +29,7 @@
 #include <dpl/log/log.h>
 #include <dpl/exception.h>
 
-#include <socket-buffer.h>
+#include <message-buffer.h>
 #include <client-common.h>
 #include <protocols.h>
 
@@ -42,7 +42,7 @@ SECURITY_SERVER_API
 int security_server_app_enable_permissions(const char *app_id, app_type_t app_type, const char **perm_list, int persistent)
 {
     using namespace SecurityServer;
-    SocketBuffer send, recv;
+    MessageBuffer send, recv;
     std::vector<std::string> permissions_list;
 
     LogDebug("security_server_app_enable_permissions() called");
@@ -85,8 +85,8 @@ int security_server_app_enable_permissions(const char *app_id, app_type_t app_ty
         Deserialization::Deserialize(recv, result);
         return result;
 
-    } catch (SocketBuffer::Exception::Base &e) {
-        LogDebug("SecurityServer::SocketBuffer::Exception " << e.DumpToString());
+    } catch (MessageBuffer::Exception::Base &e) {
+        LogDebug("SecurityServer::MessageBuffer::Exception " << e.DumpToString());
     } catch (std::exception &e) {
         LogDebug("STD exception " << e.what());
     } catch (...) {
@@ -101,7 +101,7 @@ SECURITY_SERVER_API
 int security_server_app_disable_permissions(const char *app_id, app_type_t app_type, const char **perm_list)
 {
     using namespace SecurityServer;
-    SocketBuffer send, recv;
+    MessageBuffer send, recv;
     std::vector<std::string> permissions_list;
 
     LogDebug("security_server_app_disable_permissions() called");
@@ -142,8 +142,8 @@ int security_server_app_disable_permissions(const char *app_id, app_type_t app_t
         Deserialization::Deserialize(recv, result);
         return result;
 
-    } catch (SocketBuffer::Exception::Base &e) {
-        LogDebug("SecurityServer::SocketBuffer::Exception " << e.DumpToString());
+    } catch (MessageBuffer::Exception::Base &e) {
+        LogDebug("SecurityServer::MessageBuffer::Exception " << e.DumpToString());
     } catch (std::exception &e) {
         LogDebug("STD exception " << e.what());
     } catch (...) {
index 10d017e7725492f553f496d36c4f67883e5c06e9..9534ad37463025ad5fe0b8595e47df77bb61cb4c 100644 (file)
@@ -34,7 +34,7 @@
 #include <dpl/singleton.h>
 #include <dpl/singleton_safe_impl.h>
 
-#include <socket-buffer.h>
+#include <message-buffer.h>
 
 #include <security-server.h>
 
@@ -167,7 +167,7 @@ private:
 namespace SecurityServer {
 
 
-int sendToServer(char const * const interface, const RawBuffer &send, SocketBuffer &recv) {
+int sendToServer(char const * const interface, const RawBuffer &send, MessageBuffer &recv) {
     int ret;
     SockRAII sock;
     ssize_t done = 0;
index 667bd9d1fb3f028e19fffdd47703988207eee40d..fd8e07d8b55285416a581b8dfcc81e45fc6d9741 100644 (file)
 
 #include <vector>
 
-#include <socket-buffer.h>
+#include <message-buffer.h>
 
 namespace SecurityServer {
 
 typedef std::vector<unsigned char> RawBuffer;
 
-int sendToServer(char const * const interface, const RawBuffer &send, SocketBuffer &recv);
+int sendToServer(char const * const interface, const RawBuffer &send, MessageBuffer &recv);
 
 } // namespace SecuritySever
 
index 7ca07d210e065fd031beb7c235fdd18630a3d4ad..efa373dc8238bb5ec346f1fc45e3a8cd1e9a335c 100644 (file)
@@ -28,7 +28,7 @@
 #include <dpl/log/log.h>
 #include <dpl/exception.h>
 
-#include <socket-buffer.h>
+#include <message-buffer.h>
 #include <client-common.h>
 #include <protocols.h>
 
@@ -46,7 +46,7 @@ SECURITY_SERVER_API
 int security_server_request_cookie(char *cookie, size_t bufferSize)
 {
     using namespace SecurityServer;
-    SocketBuffer send, recv;
+    MessageBuffer send, recv;
     std::vector<char> receivedCookie;
 
     LogDebug("security_server_request_cookie() called");
@@ -86,8 +86,8 @@ int security_server_request_cookie(char *cookie, size_t bufferSize)
         memcpy(cookie, &receivedCookie[0], receivedCookie.size());
         return retval;
 
-    } catch (SocketBuffer::Exception::Base &e) {
-        LogDebug("SecurityServer::SocketBuffer::Exception " << e.DumpToString());
+    } catch (MessageBuffer::Exception::Base &e) {
+        LogDebug("SecurityServer::MessageBuffer::Exception " << e.DumpToString());
     } catch (std::exception &e) {
         LogDebug("STD exception " << e.what());
     } catch (...) {
@@ -101,7 +101,7 @@ SECURITY_SERVER_API
 int security_server_get_cookie_pid(const char *cookie)
 {
     using namespace SecurityServer;
-    SocketBuffer send, recv;
+    MessageBuffer send, recv;
     int pid;
     int retval = SECURITY_SERVER_API_ERROR_UNKNOWN;
 
@@ -133,8 +133,8 @@ int security_server_get_cookie_pid(const char *cookie)
         Deserialization::Deserialize(recv, pid);
         return pid;
 
-    } catch (SocketBuffer::Exception::Base &e) {
-        LogDebug("SecurityServer::SocketBuffer::Exception " << e.DumpToString());
+    } catch (MessageBuffer::Exception::Base &e) {
+        LogDebug("SecurityServer::MessageBuffer::Exception " << e.DumpToString());
     } catch (std::exception &e) {
         LogDebug("STD exception " << e.what());
     } catch (...) {
@@ -148,7 +148,7 @@ SECURITY_SERVER_API
 char * security_server_get_smacklabel_cookie(const char *cookie)
 {
     using namespace SecurityServer;
-    SocketBuffer send, recv;
+    MessageBuffer send, recv;
     int retval = SECURITY_SERVER_API_ERROR_UNKNOWN;
     std::string label;
 
@@ -181,8 +181,8 @@ char * security_server_get_smacklabel_cookie(const char *cookie)
 
         return strdup(label.c_str());
 
-    } catch (SocketBuffer::Exception::Base &e) {
-        LogDebug("SecurityServer::SocketBuffer::Exception " << e.DumpToString());
+    } catch (MessageBuffer::Exception::Base &e) {
+        LogDebug("SecurityServer::MessageBuffer::Exception " << e.DumpToString());
     } catch (std::exception &e) {
         LogDebug("STD exception " << e.what());
     } catch (...) {
@@ -196,7 +196,7 @@ SECURITY_SERVER_API
 int security_server_check_privilege(const char *cookie, gid_t privilege)
 {
     using namespace SecurityServer;
-    SocketBuffer send, recv;
+    MessageBuffer send, recv;
     int retval = SECURITY_SERVER_API_ERROR_UNKNOWN;
 
     LogDebug("security_server_check_privilege() called");
@@ -224,8 +224,8 @@ int security_server_check_privilege(const char *cookie, gid_t privilege)
         Deserialization::Deserialize(recv, retval);
         return retval;
 
-    } catch (SocketBuffer::Exception::Base &e) {
-        LogDebug("SecurityServer::SocketBuffer::Exception " << e.DumpToString());
+    } catch (MessageBuffer::Exception::Base &e) {
+        LogDebug("SecurityServer::MessageBuffer::Exception " << e.DumpToString());
     } catch (std::exception &e) {
         LogDebug("STD exception " << e.what());
     } catch (...) {
@@ -239,7 +239,7 @@ SECURITY_SERVER_API
 int security_server_check_privilege_by_cookie(const char *cookie, const char *object, const char *access_rights)
 {
     using namespace SecurityServer;
-    SocketBuffer send, recv;
+    MessageBuffer send, recv;
     int retval = SECURITY_SERVER_API_ERROR_UNKNOWN;
 
     LogDebug("security_server_check_privilege_by_cookie() called");
@@ -271,8 +271,8 @@ int security_server_check_privilege_by_cookie(const char *cookie, const char *ob
         Deserialization::Deserialize(recv, retval);
         return retval;
 
-    } catch (SocketBuffer::Exception::Base &e) {
-        LogDebug("SecurityServer::SocketBuffer::Exception " << e.DumpToString());
+    } catch (MessageBuffer::Exception::Base &e) {
+        LogDebug("SecurityServer::MessageBuffer::Exception " << e.DumpToString());
     } catch (std::exception &e) {
         LogDebug("STD exception " << e.what());
     } catch (...) {
@@ -286,7 +286,7 @@ SECURITY_SERVER_API
 int security_server_get_uid_by_cookie(const char *cookie, uid_t *uid)
 {
     using namespace SecurityServer;
-    SocketBuffer send, recv;
+    MessageBuffer send, recv;
     int retval = SECURITY_SERVER_API_ERROR_UNKNOWN;
 
     LogDebug("security_server_get_uid_by_cookie() called");
@@ -319,8 +319,8 @@ int security_server_get_uid_by_cookie(const char *cookie, uid_t *uid)
 
         return retval;
 
-    } catch (SocketBuffer::Exception::Base &e) {
-        LogDebug("SecurityServer::SocketBuffer::Exception " << e.DumpToString());
+    } catch (MessageBuffer::Exception::Base &e) {
+        LogDebug("SecurityServer::MessageBuffer::Exception " << e.DumpToString());
     } catch (std::exception &e) {
         LogDebug("STD exception " << e.what());
     } catch (...) {
@@ -334,7 +334,7 @@ SECURITY_SERVER_API
 int security_server_get_gid_by_cookie(const char *cookie, gid_t *gid)
 {
     using namespace SecurityServer;
-    SocketBuffer send, recv;
+    MessageBuffer send, recv;
     int retval = SECURITY_SERVER_API_ERROR_UNKNOWN;
 
     LogDebug("security_server_get_uid_by_cookie() called");
@@ -367,8 +367,8 @@ int security_server_get_gid_by_cookie(const char *cookie, gid_t *gid)
 
         return retval;
 
-    } catch (SocketBuffer::Exception::Base &e) {
-        LogDebug("SecurityServer::SocketBuffer::Exception " << e.DumpToString());
+    } catch (MessageBuffer::Exception::Base &e) {
+        LogDebug("SecurityServer::MessageBuffer::Exception " << e.DumpToString());
     } catch (std::exception &e) {
         LogDebug("STD exception " << e.what());
     } catch (...) {
index 1decca80309cdbac9459c197e163ec30c462c5d6..96ac70ee1af5fb0013a26366aabd2097530f019c 100644 (file)
@@ -27,7 +27,7 @@
 #include <dpl/log/log.h>
 #include <dpl/exception.h>
 
-#include <socket-buffer.h>
+#include <message-buffer.h>
 #include <client-common.h>
 #include <protocols.h>
 
@@ -49,7 +49,7 @@ int security_server_get_gid(const char *objectName) {
             return SECURITY_SERVER_API_ERROR_INPUT_PARAM;
         }
 
-        SocketBuffer send, recv;
+        MessageBuffer send, recv;
         Serialization::Serialize(send, std::string(objectName));
 
         int retCode = sendToServer(
@@ -70,8 +70,8 @@ int security_server_get_gid(const char *objectName) {
         gid_t gid;
         Deserialization::Deserialize(recv, gid);
         return gid;
-    } catch (SocketBuffer::Exception::Base &e) {
-        LogDebug("SecurityServer::SocketBuffer::Exception " << e.DumpToString());
+    } catch (MessageBuffer::Exception::Base &e) {
+        LogDebug("SecurityServer::MessageBuffer::Exception " << e.DumpToString());
     } catch (std::exception &e) {
         LogDebug("STD exception " << e.what());
     } catch (...) {
index c259a135360989b3c8ddc8e6f31b5771089f29e4..f7df55213104d49c770476d217b067578fd47dce 100644 (file)
@@ -27,7 +27,7 @@
 #include <dpl/log/log.h>
 #include <dpl/exception.h>
 
-#include <socket-buffer.h>
+#include <message-buffer.h>
 #include <client-common.h>
 #include <protocols.h>
 
@@ -44,7 +44,7 @@ int security_server_get_object_name(gid_t gid, char *pObjectName, size_t maxObje
             return SECURITY_SERVER_API_ERROR_INPUT_PARAM;
         }
 
-        SocketBuffer send, recv;
+        MessageBuffer send, recv;
         Serialization::Serialize(send, gid);
 
         int result = sendToServer(
@@ -70,8 +70,8 @@ int security_server_get_object_name(gid_t gid, char *pObjectName, size_t maxObje
 
         return result;
 
-    } catch (SocketBuffer::Exception::Base &e) {
-        LogDebug("SecurityServer::SocketBuffer::Exception " << e.DumpToString());
+    } catch (MessageBuffer::Exception::Base &e) {
+        LogDebug("SecurityServer::MessageBuffer::Exception " << e.DumpToString());
     } catch (std::exception &e) {
         LogDebug("STD exception " << e.what());
     } catch (...) {
index cc351226a31f2a74e98d485faa8b48fcf17509e4..931548746dcaf60a99f95d4595fd5bdb0924e401 100644 (file)
@@ -28,7 +28,7 @@
 #include <dpl/log/log.h>
 #include <dpl/exception.h>
 
-#include <socket-buffer.h>
+#include <message-buffer.h>
 #include <client-common.h>
 #include <protocols.h>
 #include <smack-check.h>
@@ -63,7 +63,7 @@ int security_server_check_privilege_by_pid(
             return SECURITY_SERVER_API_ERROR_INPUT_PARAM;
         }
 
-        SocketBuffer send, recv;
+        MessageBuffer send, recv;
         Serialization::Serialize(send, pid);
         Serialization::Serialize(send, std::string(object));
         Serialization::Serialize(send, std::string(access_rights));
@@ -78,8 +78,8 @@ int security_server_check_privilege_by_pid(
 
         Deserialization::Deserialize(recv, result);
         return result;
-    } catch (SocketBuffer::Exception::Base &e) {
-        LogDebug("SecurityServer::SocketBuffer::Exception " << e.DumpToString());
+    } catch (MessageBuffer::Exception::Base &e) {
+        LogDebug("SecurityServer::MessageBuffer::Exception " << e.DumpToString());
     } catch (std::exception &e) {
         LogDebug("STD exception " << e.what());
     } catch (...) {
index a71da767486fb2c5dc71ed980980a0ad21892f5c..66e6fc7d7c65f83866a54804f0b40ba7aec63437 100644 (file)
@@ -27,7 +27,7 @@
 #include <dpl/log/log.h>
 #include <dpl/exception.h>
 
-#include <socket-buffer.h>
+#include <message-buffer.h>
 #include <client-common.h>
 #include <protocols.h>
 #include <smack-check.h>
@@ -48,7 +48,7 @@ int security_server_app_give_access(const char *customer_label, int customer_pid
             return SECURITY_SERVER_API_ERROR_INPUT_PARAM;
         }
 
-        SocketBuffer send, recv;
+        MessageBuffer send, recv;
         Serialization::Serialize(send, std::string(customer_label));
         Serialization::Serialize(send, customer_pid);
 
@@ -62,8 +62,8 @@ int security_server_app_give_access(const char *customer_label, int customer_pid
 
         Deserialization::Deserialize(recv, result);
         return result;
-    } catch (SocketBuffer::Exception::Base &e) {
-        LogDebug("SecurityServer::SocketBuffer::Exception " << e.DumpToString());
+    } catch (MessageBuffer::Exception::Base &e) {
+        LogDebug("SecurityServer::MessageBuffer::Exception " << e.DumpToString());
     } catch (std::exception &e) {
         LogDebug("STD exception " << e.what());
     } catch (...) {
index bca7c3d8036795859da68ac441ac4f1a3f359b15..4358b075a7db3a2fc39ac77ab3ef40c8360332b6 100644 (file)
@@ -29,7 +29,7 @@
 #include <dpl/log/log.h>
 #include <dpl/exception.h>
 
-#include <socket-buffer.h>
+#include <message-buffer.h>
 #include <client-common.h>
 #include <protocols.h>
 #include <smack-check.h>
@@ -42,7 +42,7 @@ static int get_exec_path(pid_t pid, std::string &exe)
     using namespace SecurityServer;
 
     try{
-        SocketBuffer send, recv;
+        MessageBuffer send, recv;
         Serialization::Serialize(send, pid);
 
         int result = sendToServer(
@@ -58,8 +58,8 @@ static int get_exec_path(pid_t pid, std::string &exe)
 
         Deserialization::Deserialize(recv, exe);
         return result;
-    } catch (SocketBuffer::Exception::Base &e) {
-        LogDebug("SecurityServer::SocketBuffer::Exception " << e.DumpToString());
+    } catch (MessageBuffer::Exception::Base &e) {
+        LogDebug("SecurityServer::MessageBuffer::Exception " << e.DumpToString());
     } catch (std::exception &e) {
         LogDebug("STD exception " << e.what());
     } catch (...) {
diff --git a/src/server2/common/message-buffer.cpp b/src/server2/common/message-buffer.cpp
new file mode 100644 (file)
index 0000000..3fc090b
--- /dev/null
@@ -0,0 +1,69 @@
+/*
+ *  Copyright (c) 2000 - 2013 Samsung Electronics Co., Ltd All Rights Reserved
+ *
+ *  Contact: Bumjin Im <bj.im@samsung.com>
+ *
+ *  Licensed under the Apache License, Version 2.0 (the "License");
+ *  you may not use this file except in compliance with the License.
+ *  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *  Unless required by applicable law or agreed to in writing, software
+ *  distributed under the License is distributed on an "AS IS" BASIS,
+ *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  See the License for the specific language governing permissions and
+ *  limitations under the License
+ */
+/*
+ * @file        message-buffer.cpp
+ * @author      Bartlomiej Grzelewski (b.grzelewski@samsung.com)
+ * @version     1.0
+ * @brief       Implementation of MessageBuffer.
+ */
+
+#include <message-buffer.h>
+
+#include <dpl/log/log.h>
+
+namespace SecurityServer {
+
+void MessageBuffer::Push(const RawBuffer &data) {
+    m_buffer.AppendCopy(&data[0], data.size());
+}
+
+RawBuffer MessageBuffer::Pop() {
+    size_t size = m_buffer.Size();
+    RawBuffer buffer;
+    buffer.resize(size + sizeof(size_t));
+    memcpy(&buffer[0], &size, sizeof(size_t));
+    m_buffer.FlattenConsume(&buffer[sizeof(size_t)], size);
+    return buffer;
+}
+
+bool MessageBuffer::Ready() {
+    CountBytesLeft();
+    if (m_bytesLeft == 0)
+        return false;
+    if (m_bytesLeft > m_buffer.Size())
+        return false;
+    return true;
+}
+
+void MessageBuffer::Read(size_t num, void *bytes) {
+    CountBytesLeft();
+    if (num > m_bytesLeft) {
+        LogDebug("Protocol broken. OutOfData. Asked for: " << num << " Ready: " << m_bytesLeft << " Buffer.size(): " << m_buffer.Size());
+        Throw(Exception::OutOfData);
+    }
+
+    m_buffer.FlattenConsume(bytes, num);
+    m_bytesLeft -= num;
+}
+
+void MessageBuffer::Write(size_t num, const void *bytes) {
+    m_buffer.AppendCopy(bytes, num);
+}
+
+} // namespace SecurityServer
+
diff --git a/src/server2/common/message-buffer.h b/src/server2/common/message-buffer.h
new file mode 100644 (file)
index 0000000..850c7de
--- /dev/null
@@ -0,0 +1,79 @@
+/*
+ *  Copyright (c) 2000 - 2013 Samsung Electronics Co., Ltd All Rights Reserved
+ *
+ *  Contact: Bumjin Im <bj.im@samsung.com>
+ *
+ *  Licensed under the Apache License, Version 2.0 (the "License");
+ *  you may not use this file except in compliance with the License.
+ *  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *  Unless required by applicable law or agreed to in writing, software
+ *  distributed under the License is distributed on an "AS IS" BASIS,
+ *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  See the License for the specific language governing permissions and
+ *  limitations under the License
+ */
+/*
+ * @file        secket-buffer.h
+ * @author      Bartlomiej Grzelewski (b.grzelewski@samsung.com)
+ * @version     1.0
+ * @brief       Implementatin of MessageBuffer.
+ */
+
+#ifndef _SECURITY_SERVER_SOCKET_BUFFER_
+#define _SECURITY_SERVER_SOCKET_BUFFER_
+
+#include <vector>
+
+#include <dpl/binary_queue.h>
+#include <dpl/exception.h>
+#include <dpl/serialization.h>
+
+namespace SecurityServer {
+
+typedef std::vector<unsigned char> RawBuffer;
+
+class MessageBuffer : public SecurityServer::IStream {
+public:
+    class Exception
+    {
+    public:
+        DECLARE_EXCEPTION_TYPE(SecurityServer::Exception, Base)
+        DECLARE_EXCEPTION_TYPE(Base, OutOfData)
+    };
+
+    MessageBuffer()
+      : m_bytesLeft(0)
+    {}
+
+    void Push(const RawBuffer &data);
+
+    RawBuffer Pop();
+
+    bool Ready();
+
+    virtual void Read(size_t num, void *bytes);
+
+    virtual void Write(size_t num, const void *bytes);
+
+protected:
+
+    inline void CountBytesLeft() {
+        if (m_bytesLeft > 0)
+            return;  // we already counted m_bytesLeft nothing to do
+
+        if (m_buffer.Size() < sizeof(size_t))
+            return;  // we cannot count m_bytesLeft because buffer is too small
+
+        m_buffer.FlattenConsume(&m_bytesLeft, sizeof(size_t));
+    }
+
+    size_t m_bytesLeft;
+    SecurityServer::BinaryQueue m_buffer;
+};
+
+} // namespace SecurityServer
+
+#endif // _SECURITY_SERVER_SOCKET_BUFFER_
diff --git a/src/server2/common/socket-buffer.cpp b/src/server2/common/socket-buffer.cpp
deleted file mode 100644 (file)
index 682547a..0000000
+++ /dev/null
@@ -1,69 +0,0 @@
-/*
- *  Copyright (c) 2000 - 2013 Samsung Electronics Co., Ltd All Rights Reserved
- *
- *  Contact: Bumjin Im <bj.im@samsung.com>
- *
- *  Licensed under the Apache License, Version 2.0 (the "License");
- *  you may not use this file except in compliance with the License.
- *  You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- *  Unless required by applicable law or agreed to in writing, software
- *  distributed under the License is distributed on an "AS IS" BASIS,
- *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- *  See the License for the specific language governing permissions and
- *  limitations under the License
- */
-/*
- * @file        socket-buffer.cpp
- * @author      Bartlomiej Grzelewski (b.grzelewski@samsung.com)
- * @version     1.0
- * @brief       Implementation of SocketBuffer.
- */
-
-#include <socket-buffer.h>
-
-#include <dpl/log/log.h>
-
-namespace SecurityServer {
-
-void SocketBuffer::Push(const RawBuffer &data) {
-    m_buffer.AppendCopy(&data[0], data.size());
-}
-
-RawBuffer SocketBuffer::Pop() {
-    size_t size = m_buffer.Size();
-    RawBuffer buffer;
-    buffer.resize(size + sizeof(size_t));
-    memcpy(&buffer[0], &size, sizeof(size_t));
-    m_buffer.FlattenConsume(&buffer[sizeof(size_t)], size);
-    return buffer;
-}
-
-bool SocketBuffer::Ready() {
-    CountBytesLeft();
-    if (m_bytesLeft == 0)
-        return false;
-    if (m_bytesLeft > m_buffer.Size())
-        return false;
-    return true;
-}
-
-void SocketBuffer::Read(size_t num, void *bytes) {
-    CountBytesLeft();
-    if (num > m_bytesLeft) {
-        LogDebug("Protocol broken. OutOfData. Asked for: " << num << " Ready: " << m_bytesLeft << " Buffer.size(): " << m_buffer.Size());
-        Throw(Exception::OutOfData);
-    }
-
-    m_buffer.FlattenConsume(bytes, num);
-    m_bytesLeft -= num;
-}
-
-void SocketBuffer::Write(size_t num, const void *bytes) {
-    m_buffer.AppendCopy(bytes, num);
-}
-
-} // namespace SecurityServer
-
diff --git a/src/server2/common/socket-buffer.h b/src/server2/common/socket-buffer.h
deleted file mode 100644 (file)
index 34b58fa..0000000
+++ /dev/null
@@ -1,79 +0,0 @@
-/*
- *  Copyright (c) 2000 - 2013 Samsung Electronics Co., Ltd All Rights Reserved
- *
- *  Contact: Bumjin Im <bj.im@samsung.com>
- *
- *  Licensed under the Apache License, Version 2.0 (the "License");
- *  you may not use this file except in compliance with the License.
- *  You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- *  Unless required by applicable law or agreed to in writing, software
- *  distributed under the License is distributed on an "AS IS" BASIS,
- *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- *  See the License for the specific language governing permissions and
- *  limitations under the License
- */
-/*
- * @file        secket-buffer.h
- * @author      Bartlomiej Grzelewski (b.grzelewski@samsung.com)
- * @version     1.0
- * @brief       Implementatin of SocketBuffer.
- */
-
-#ifndef _SECURITY_SERVER_SOCKET_BUFFER_
-#define _SECURITY_SERVER_SOCKET_BUFFER_
-
-#include <vector>
-
-#include <dpl/binary_queue.h>
-#include <dpl/exception.h>
-#include <dpl/serialization.h>
-
-namespace SecurityServer {
-
-typedef std::vector<unsigned char> RawBuffer;
-
-class SocketBuffer : public SecurityServer::IStream {
-public:
-    class Exception
-    {
-    public:
-        DECLARE_EXCEPTION_TYPE(SecurityServer::Exception, Base)
-        DECLARE_EXCEPTION_TYPE(Base, OutOfData)
-    };
-
-    SocketBuffer()
-      : m_bytesLeft(0)
-    {}
-
-    void Push(const RawBuffer &data);
-
-    RawBuffer Pop();
-
-    bool Ready();
-
-    virtual void Read(size_t num, void *bytes);
-
-    virtual void Write(size_t num, const void *bytes);
-
-protected:
-
-    inline void CountBytesLeft() {
-        if (m_bytesLeft > 0)
-            return;  // we already counted m_bytesLeft nothing to do
-
-        if (m_buffer.Size() < sizeof(size_t))
-            return;  // we cannot count m_bytesLeft because buffer is too small
-
-        m_buffer.FlattenConsume(&m_bytesLeft, sizeof(size_t));
-    }
-
-    size_t m_bytesLeft;
-    SecurityServer::BinaryQueue m_buffer;
-};
-
-} // namespace SecurityServer
-
-#endif // _SECURITY_SERVER_SOCKET_BUFFER_
index d34ac5baef52b60b32eb6fe7dd7a967c569352aa..1c10030ded3afb26fc10616b5ecd665e0c242506 100644 (file)
@@ -79,7 +79,7 @@ void AppPermissionsService::write(const WriteEvent &event) {
 
 void AppPermissionsService::read(const ReadEvent &event) {
     LogDebug("Read event for counter: " << event.connectionID.counter);
-    auto &buffer = m_socketBufferMap[event.connectionID.counter];
+    auto &buffer = m_messageBufferMap[event.connectionID.counter];
     buffer.Push(event.rawBuffer);
 
     // We can get several requests in one package.
@@ -89,7 +89,7 @@ void AppPermissionsService::read(const ReadEvent &event) {
 
 void AppPermissionsService::close(const CloseEvent &event) {
     LogDebug("CloseEvent. ConnectionID: " << event.connectionID.sock);
-    m_socketBufferMap.erase(event.connectionID.counter);
+    m_messageBufferMap.erase(event.connectionID.counter);
 }
 
 void AppPermissionsService::error(const ErrorEvent &event) {
@@ -97,10 +97,10 @@ void AppPermissionsService::error(const ErrorEvent &event) {
     m_serviceManager->Close(event.connectionID);
 }
 
-bool AppPermissionsService::readOne(const ConnectionID &conn, SocketBuffer &buffer)
+bool AppPermissionsService::readOne(const ConnectionID &conn, MessageBuffer &buffer)
 {
     LogDebug("Iteration begin");
-    SocketBuffer send, recv;
+    MessageBuffer send, recv;
     std::vector<std::string> permissions_list;
     std::string app_id;
     int persistent;
@@ -129,7 +129,7 @@ bool AppPermissionsService::readOne(const ConnectionID &conn, SocketBuffer &buff
         app_type = (app_type_t)temp;
         Deserialization::Deserialize(buffer, app_id);
         Deserialization::Deserialize(buffer, permissions_list);
-    } Catch (SocketBuffer::Exception::Base) {
+    } Catch (MessageBuffer::Exception::Base) {
         LogDebug("Broken protocol. Closing socket.");
         m_serviceManager->Close(conn);
         return false;
index 2030ca64152807424031f4ecc6c8499fd715d274..301f2beaeb39c6e25a29ba7b6960dd0850ca860c 100644 (file)
@@ -29,7 +29,7 @@
 #include <service-thread.h>
 #include <generic-socket-manager.h>
 #include <dpl/serialization.h>
-#include <socket-buffer.h>
+#include <message-buffer.h>
 #include <security-server-common.h>
 
 namespace SecurityServer {
@@ -39,7 +39,7 @@ class AppPermissionsService  :
   , public SecurityServer::ServiceThread<AppPermissionsService>
 {
 public:
-    typedef std::map<int, SocketBuffer> SocketBufferMap;
+    typedef std::map<int, MessageBuffer> MessageBufferMap;
 
     ServiceDescriptionVector GetServiceDescription();
 
@@ -56,9 +56,9 @@ public:
     void error(const ErrorEvent &event);
 
 private:
-    bool readOne(const ConnectionID &conn, SocketBuffer &buffer);
+    bool readOne(const ConnectionID &conn, MessageBuffer &buffer);
 
-    SocketBufferMap m_socketBufferMap;
+    MessageBufferMap m_messageBufferMap;
 };
 
 } // namespace SecurityServer
index 1b28f8595d87dfd9ebc59d5277ce1e629f6f5483..07c6b112d2015b7b70aae2fb2e96fbe1c8522e07 100644 (file)
@@ -98,10 +98,10 @@ void CookieService::error(const ErrorEvent &event) {
     m_serviceManager->Close(event.connectionID);
 }
 
-bool CookieService::readOne(const ConnectionID &conn, SocketBuffer &buffer, int interfaceID)
+bool CookieService::readOne(const ConnectionID &conn, MessageBuffer &buffer, int interfaceID)
 {
     LogDebug("Iteration begin");
-    SocketBuffer send, recv;
+    MessageBuffer send, recv;
     int msgType;
     bool removeGarbage = false;
 
@@ -113,7 +113,7 @@ bool CookieService::readOne(const ConnectionID &conn, SocketBuffer &buffer, int
     //receive data from buffer and check MSG_ID
     Try {
         Deserialization::Deserialize(buffer, msgType);  //receive MSG_ID
-    } Catch (SocketBuffer::Exception::Base) {
+    } Catch (MessageBuffer::Exception::Base) {
         LogDebug("Broken protocol. Closing socket.");
         m_serviceManager->Close(conn);
         return false;
@@ -200,7 +200,7 @@ bool CookieService::readOne(const ConnectionID &conn, SocketBuffer &buffer, int
     return retval;
 }
 
-bool CookieService::cookieRequest(SocketBuffer &send, int socket)
+bool CookieService::cookieRequest(MessageBuffer &send, int socket)
 {
     struct ucred cr;
     unsigned len = sizeof(cr);
@@ -221,13 +221,13 @@ bool CookieService::cookieRequest(SocketBuffer &send, int socket)
     return true;
 }
 
-bool CookieService::pidByCookieRequest(SocketBuffer &buffer, SocketBuffer &send)
+bool CookieService::pidByCookieRequest(MessageBuffer &buffer, MessageBuffer &send)
 {
     std::vector<char> cookieKey;
 
     Try {
         Deserialization::Deserialize(buffer, cookieKey);
-    } Catch (SocketBuffer::Exception::Base) {
+    } Catch (MessageBuffer::Exception::Base) {
         LogDebug("Broken protocol. Closing socket.");
         return false;
     }
@@ -247,13 +247,13 @@ bool CookieService::pidByCookieRequest(SocketBuffer &buffer, SocketBuffer &send)
     return true;
 }
 
-bool CookieService::smackLabelByCookieRequest(SocketBuffer &buffer, SocketBuffer &send)
+bool CookieService::smackLabelByCookieRequest(MessageBuffer &buffer, MessageBuffer &send)
 {
     std::vector<char> cookieKey;
 
     Try {
         Deserialization::Deserialize(buffer, cookieKey);
-    } Catch (SocketBuffer::Exception::Base) {
+    } Catch (MessageBuffer::Exception::Base) {
         LogDebug("Broken protocol. Closing socket.");
         return false;
     }
@@ -273,7 +273,7 @@ bool CookieService::smackLabelByCookieRequest(SocketBuffer &buffer, SocketBuffer
     return true;
 }
 
-bool CookieService::privilegeByCookieGidRequest(SocketBuffer &buffer, SocketBuffer &send)
+bool CookieService::privilegeByCookieGidRequest(MessageBuffer &buffer, MessageBuffer &send)
 {
     std::vector<char> cookieKey;
     int gid;
@@ -281,7 +281,7 @@ bool CookieService::privilegeByCookieGidRequest(SocketBuffer &buffer, SocketBuff
     Try {
         Deserialization::Deserialize(buffer, cookieKey);
         Deserialization::Deserialize(buffer, gid);
-    } Catch (SocketBuffer::Exception::Base) {
+    } Catch (MessageBuffer::Exception::Base) {
         LogDebug("Broken protocol. Closing socket.");
         return false;
     }
@@ -304,7 +304,7 @@ bool CookieService::privilegeByCookieGidRequest(SocketBuffer &buffer, SocketBuff
     return true;
 }
 
-bool CookieService::privilegeByCookieRequest(SocketBuffer &buffer, SocketBuffer &send)
+bool CookieService::privilegeByCookieRequest(MessageBuffer &buffer, MessageBuffer &send)
 {
     std::vector<char> cookieKey;
     std::string subject;
@@ -315,7 +315,7 @@ bool CookieService::privilegeByCookieRequest(SocketBuffer &buffer, SocketBuffer
         Deserialization::Deserialize(buffer, cookieKey);
         Deserialization::Deserialize(buffer, object);
         Deserialization::Deserialize(buffer, access);
-    } Catch (SocketBuffer::Exception::Base) {
+    } Catch (MessageBuffer::Exception::Base) {
         LogDebug("Broken protocol. Closing socket.");
         return false;
     }
@@ -339,13 +339,13 @@ bool CookieService::privilegeByCookieRequest(SocketBuffer &buffer, SocketBuffer
     return true;
 }
 
-bool CookieService::uidByCookieRequest(SocketBuffer &buffer, SocketBuffer &send)
+bool CookieService::uidByCookieRequest(MessageBuffer &buffer, MessageBuffer &send)
 {
     std::vector<char> cookieKey;
 
     Try {
         Deserialization::Deserialize(buffer, cookieKey);
-    } Catch (SocketBuffer::Exception::Base) {
+    } Catch (MessageBuffer::Exception::Base) {
         LogDebug("Broken protocol. Closing socket.");
         return false;
     }
@@ -365,13 +365,13 @@ bool CookieService::uidByCookieRequest(SocketBuffer &buffer, SocketBuffer &send)
     return true;
 }
 
-bool CookieService::gidByCookieRequest(SocketBuffer &buffer, SocketBuffer &send)
+bool CookieService::gidByCookieRequest(MessageBuffer &buffer, MessageBuffer &send)
 {
     std::vector<char> cookieKey;
 
     Try {
         Deserialization::Deserialize(buffer, cookieKey);
-    } Catch (SocketBuffer::Exception::Base) {
+    } Catch (MessageBuffer::Exception::Base) {
         LogDebug("Broken protocol. Closing socket.");
         return false;
     }
index 78ee0fd4281c28b57416c7e124985339524f2b07..ad4ee3bdfacfbc9fc343032ced06a65a92aee96e 100644 (file)
@@ -28,7 +28,7 @@
 #include <service-thread.h>
 #include <generic-socket-manager.h>
 #include <dpl/serialization.h>
-#include <socket-buffer.h>
+#include <message-buffer.h>
 #include <security-server-common.h>
 #include <cookie-jar.h>
 
@@ -42,7 +42,7 @@ public:
     struct SocketInfo
     {
         int interfaceID;
-        SocketBuffer buffer;
+        MessageBuffer buffer;
     };
 
     typedef std::map<int, SocketInfo> SocketInfoMap;
@@ -62,17 +62,17 @@ public:
     void error(const ErrorEvent &event);
 
 private:
-    bool readOne(const ConnectionID &conn, SocketBuffer &buffer, int interfaceID);
+    bool readOne(const ConnectionID &conn, MessageBuffer &buffer, int interfaceID);
 
-    bool cookieRequest(SocketBuffer &send, int socket);
+    bool cookieRequest(MessageBuffer &send, int socket);
 
-    bool pidByCookieRequest(SocketBuffer &buffer, SocketBuffer &send);
-    bool smackLabelByCookieRequest(SocketBuffer &buffer, SocketBuffer &send);
-    bool privilegeByCookieGidRequest(SocketBuffer &buffer, SocketBuffer &send);
-    bool privilegeByCookieRequest(SocketBuffer &buffer, SocketBuffer &send);
+    bool pidByCookieRequest(MessageBuffer &buffer, MessageBuffer &send);
+    bool smackLabelByCookieRequest(MessageBuffer &buffer, MessageBuffer &send);
+    bool privilegeByCookieGidRequest(MessageBuffer &buffer, MessageBuffer &send);
+    bool privilegeByCookieRequest(MessageBuffer &buffer, MessageBuffer &send);
 
-    bool uidByCookieRequest(SocketBuffer &buffer, SocketBuffer &send);
-    bool gidByCookieRequest(SocketBuffer &buffer, SocketBuffer &send);
+    bool uidByCookieRequest(MessageBuffer &buffer, MessageBuffer &send);
+    bool gidByCookieRequest(MessageBuffer &buffer, MessageBuffer &send);
 
     CookieJar m_cookieJar;
 
index 2a9b820748d4759a4816416f5403747e8173eea1..6f6200c5b7282b3f636dfa80f0d868a8043fb466 100644 (file)
@@ -77,7 +77,7 @@ void SharedMemoryService::write(const WriteEvent &event) {
         m_serviceManager->Close(event.connectionID);
 }
 
-bool SharedMemoryService::readOne(const ConnectionID &conn, SocketBuffer &buffer) {
+bool SharedMemoryService::readOne(const ConnectionID &conn, MessageBuffer &buffer) {
     LogDebug("Iteration begin");
     static const char * const revoke = "-----";
     static const char * const permissions = "rwxat";
@@ -94,7 +94,7 @@ bool SharedMemoryService::readOne(const ConnectionID &conn, SocketBuffer &buffer
     Try {
         Deserialization::Deserialize(buffer, clientLabel);
         Deserialization::Deserialize(buffer, clientPid);
-     } Catch (SocketBuffer::Exception::Base) {
+     } Catch (MessageBuffer::Exception::Base) {
         LogDebug("Broken protocol. Closing socket.");
         m_serviceManager->Close(conn);
         return false;
@@ -137,7 +137,7 @@ end:
     free(providerLabel);
     smack_accesses_free(smack);
 
-    SocketBuffer sendBuffer;
+    MessageBuffer sendBuffer;
     Serialization::Serialize(sendBuffer, retCode);
     m_serviceManager->Write(conn, sendBuffer.Pop());
     return true;
@@ -145,7 +145,7 @@ end:
 
 void SharedMemoryService::read(const ReadEvent &event) {
     LogDebug("Read event for counter: " << event.connectionID.counter);
-    auto &buffer = m_socketBufferMap[event.connectionID.counter];
+    auto &buffer = m_messageBufferMap[event.connectionID.counter];
     buffer.Push(event.rawBuffer);
 
     // We can get several requests in one package.
@@ -155,7 +155,7 @@ void SharedMemoryService::read(const ReadEvent &event) {
 
 void SharedMemoryService::close(const CloseEvent &event) {
     LogDebug("CloseEvent. ConnectionID: " << event.connectionID.sock);
-    m_socketBufferMap.erase(event.connectionID.counter);
+    m_messageBufferMap.erase(event.connectionID.counter);
 }
 
 void SharedMemoryService::error(const ErrorEvent &event) {
index e035978a316d57ad884fbc20318a198896e57a53..019b05a74331c41343351561d6297c5411956acd 100644 (file)
@@ -28,7 +28,7 @@
 #include <service-thread.h>
 #include <generic-socket-manager.h>
 
-#include <socket-buffer.h>
+#include <message-buffer.h>
 
 namespace SecurityServer {
 
@@ -37,7 +37,7 @@ class SharedMemoryService
   , public SecurityServer::ServiceThread<SharedMemoryService>
 {
 public:
-    typedef std::map<int, SocketBuffer> SocketBufferMap;
+    typedef std::map<int, MessageBuffer> MessageBufferMap;
 
     ServiceDescriptionVector GetServiceDescription();
 
@@ -53,9 +53,9 @@ public:
     void close(const CloseEvent &event);
     void error(const ErrorEvent &event);
 private:
-    bool readOne(const ConnectionID &conn, SocketBuffer &buffer);
+    bool readOne(const ConnectionID &conn, MessageBuffer &buffer);
 
-    SocketBufferMap m_socketBufferMap;
+    MessageBufferMap m_messageBufferMap;
 };
 
 } // namespace SecurityServer
index 093537ffac56e75f71d4ef7ac0c03c76a36b3cb9..9446f3b8fac900f14f2e40758ab9e77636b2fdc9 100644 (file)
@@ -30,7 +30,7 @@
 
 #include <dpl/serialization.h>
 
-#include <socket-buffer.h>
+#include <message-buffer.h>
 
 namespace SecurityServer {
 
index 6948134f46d17ca253ef3d9e78816b4be2e8e335..262ea2a926ec0a737906e9085f8e45be32125843 100644 (file)
@@ -80,7 +80,7 @@ void ExecPathService::write(const WriteEvent &event) {
         m_serviceManager->Close(event.connectionID);
 }
 
-bool ExecPathService::processOne(const ConnectionID &conn, SocketBuffer &buffer) {
+bool ExecPathService::processOne(const ConnectionID &conn, MessageBuffer &buffer) {
     LogDebug("Processing message");
 
     int pid = 0;
@@ -92,13 +92,13 @@ bool ExecPathService::processOne(const ConnectionID &conn, SocketBuffer &buffer)
 
     Try {
         Deserialization::Deserialize(buffer, pid);
-     } Catch (SocketBuffer::Exception::Base) {
+     } Catch (MessageBuffer::Exception::Base) {
         LogDebug("Broken protocol. Closing socket.");
         m_serviceManager->Close(conn);
         return false;
     }
 
-    SocketBuffer sendBuffer;
+    MessageBuffer sendBuffer;
     int retVal;
 
     // get executable path
@@ -126,7 +126,7 @@ bool ExecPathService::processOne(const ConnectionID &conn, SocketBuffer &buffer)
 
 void ExecPathService::read(const ReadEvent &event) {
     LogDebug("Read event for counter: " << event.connectionID.counter);
-    auto &buffer = m_socketBufferMap[event.connectionID.counter];
+    auto &buffer = m_messageBufferMap[event.connectionID.counter];
     buffer.Push(event.rawBuffer);
 
     // We can get several requests in one package.
@@ -136,7 +136,7 @@ void ExecPathService::read(const ReadEvent &event) {
 
 void ExecPathService::close(const CloseEvent &event) {
     LogDebug("CloseEvent. ConnectionID: " << event.connectionID.sock);
-    m_socketBufferMap.erase(event.connectionID.counter);
+    m_messageBufferMap.erase(event.connectionID.counter);
 }
 
 void ExecPathService::error(const ErrorEvent &event) {
index 0832acb48f4b7973ce82f68643b86a45bfae9424..89ad1b2aedd7a84a927810049630817bd095da7f 100644 (file)
@@ -28,7 +28,7 @@
 #include <service-thread.h>
 #include <generic-socket-manager.h>
 
-#include <socket-buffer.h>
+#include <message-buffer.h>
 
 namespace SecurityServer {
 
@@ -37,7 +37,7 @@ class ExecPathService
   , public SecurityServer::ServiceThread<ExecPathService>
 {
 public:
-    typedef std::map<int, SocketBuffer> SocketBufferMap;
+    typedef std::map<int, MessageBuffer> MessageBufferMap;
 
     ServiceDescriptionVector GetServiceDescription();
 
@@ -53,8 +53,8 @@ public:
     void close(const CloseEvent &event);
     void error(const ErrorEvent &event);
 private:
-    bool processOne(const ConnectionID &conn, SocketBuffer &buffer);
-    SocketBufferMap m_socketBufferMap;
+    bool processOne(const ConnectionID &conn, MessageBuffer &buffer);
+    MessageBufferMap m_messageBufferMap;
 };
 
 } // namespace SecurityServer
index 298133b09a5ebff1c20ee98bb9340203a5b0bd66..58a8500c2f91b8f7435c6a2386f48152a165c4db 100644 (file)
@@ -116,7 +116,7 @@ int GetGidService::setGid(std::string& obj)
 }
 
 
-bool GetGidService::readOne(const ConnectionID &conn, SocketBuffer &buffer) {
+bool GetGidService::readOne(const ConnectionID &conn, MessageBuffer &buffer) {
     LogDebug("Iteration begin");
     std::string objectName;
     int retCode = SECURITY_SERVER_API_ERROR_SERVER_ERROR;
@@ -128,7 +128,7 @@ bool GetGidService::readOne(const ConnectionID &conn, SocketBuffer &buffer) {
     // Get objects name:
     Try {
         Deserialization::Deserialize(buffer, objectName);
-     } Catch (SocketBuffer::Exception::Base) {
+     } Catch (MessageBuffer::Exception::Base) {
         LogDebug("Broken protocol. Closing socket.");
         m_serviceManager->Close(conn);
         return false;
@@ -138,7 +138,7 @@ bool GetGidService::readOne(const ConnectionID &conn, SocketBuffer &buffer) {
     retCode = setGid(objectName);
 
     // Send the result
-    SocketBuffer sendBuffer;
+    MessageBuffer sendBuffer;
     Serialization::Serialize(sendBuffer, retCode);
     Serialization::Serialize(sendBuffer, m_gid);
     m_serviceManager->Write(conn, sendBuffer.Pop());
@@ -147,7 +147,7 @@ bool GetGidService::readOne(const ConnectionID &conn, SocketBuffer &buffer) {
 
 void GetGidService::read(const ReadEvent &event) {
     LogDebug("Read event for counter: " << event.connectionID.counter);
-    auto &buffer = m_socketBufferMap[event.connectionID.counter];
+    auto &buffer = m_messageBufferMap[event.connectionID.counter];
     buffer.Push(event.rawBuffer);
 
     // We can get several requests in one package.
@@ -157,7 +157,7 @@ void GetGidService::read(const ReadEvent &event) {
 
 void GetGidService::close(const CloseEvent &event) {
     LogDebug("CloseEvent. ConnectionID: " << event.connectionID.sock);
-    m_socketBufferMap.erase(event.connectionID.counter);
+    m_messageBufferMap.erase(event.connectionID.counter);
 }
 
 void GetGidService::error(const ErrorEvent &event) {
index fd691247e68f169427cf224648eb8606354d9dd9..c8c5e0a731dfa613889f1f872c4a76a3498c39cb 100644 (file)
@@ -29,7 +29,7 @@
 #include <generic-socket-manager.h>
 
 #include <dpl/serialization.h>
-#include <socket-buffer.h>
+#include <message-buffer.h>
 
 #include <security-server-common.h>
 
@@ -40,7 +40,7 @@ class GetGidService  :
   , public SecurityServer::ServiceThread<GetGidService>
 {
 public:
-    typedef std::map<int, SocketBuffer> SocketBufferMap;
+    typedef std::map<int, MessageBuffer> MessageBufferMap;
 
     ServiceDescriptionVector GetServiceDescription();
 
@@ -57,9 +57,9 @@ public:
     void error(const ErrorEvent &event);
 private:
     gid_t m_gid;
-    bool readOne(const ConnectionID &conn, SocketBuffer &buffer);
+    bool readOne(const ConnectionID &conn, MessageBuffer &buffer);
     int  setGid(std::string& objectName);
-    SocketBufferMap m_socketBufferMap;
+    MessageBufferMap m_messageBufferMap;
 };
 
 } // namespace SecurityServer
index d973335d86684cacee3d9fc99725d050c57a1f0d..882c1dd695d4eefb1fddfa5d7c076f6eaa5bd335 100644 (file)
@@ -119,7 +119,7 @@ int GetObjectNameService::setName(const gid_t gid)
 }
 
 
-bool GetObjectNameService::readOne(const ConnectionID &conn, SocketBuffer &buffer) {
+bool GetObjectNameService::readOne(const ConnectionID &conn, MessageBuffer &buffer) {
     LogDebug("Iteration begin");
     gid_t gid;
     int retCode = SECURITY_SERVER_API_ERROR_SERVER_ERROR;
@@ -131,7 +131,7 @@ bool GetObjectNameService::readOne(const ConnectionID &conn, SocketBuffer &buffe
     // Get objects GID:
     Try {
         Deserialization::Deserialize(buffer, gid);
-     } Catch (SocketBuffer::Exception::Base) {
+     } Catch (MessageBuffer::Exception::Base) {
         LogDebug("Broken protocol. Closing socket.");
         m_serviceManager->Close(conn);
         return false;
@@ -141,7 +141,7 @@ bool GetObjectNameService::readOne(const ConnectionID &conn, SocketBuffer &buffe
     retCode = setName(gid);
 
     // Send the result
-    SocketBuffer sendBuffer;
+    MessageBuffer sendBuffer;
     Serialization::Serialize(sendBuffer, retCode);
     Serialization::Serialize(sendBuffer, m_name);
     m_serviceManager->Write(conn, sendBuffer.Pop());
@@ -150,7 +150,7 @@ bool GetObjectNameService::readOne(const ConnectionID &conn, SocketBuffer &buffe
 
 void GetObjectNameService::read(const ReadEvent &event) {
     LogDebug("Read event for counter: " << event.connectionID.counter);
-    auto &buffer = m_socketBufferMap[event.connectionID.counter];
+    auto &buffer = m_messageBufferMap[event.connectionID.counter];
     buffer.Push(event.rawBuffer);
 
     // We can get several requests in one package.
@@ -160,7 +160,7 @@ void GetObjectNameService::read(const ReadEvent &event) {
 
 void GetObjectNameService::close(const CloseEvent &event) {
     LogDebug("CloseEvent. ConnectionID: " << event.connectionID.sock);
-    m_socketBufferMap.erase(event.connectionID.counter);
+    m_messageBufferMap.erase(event.connectionID.counter);
 }
 
 void GetObjectNameService::error(const ErrorEvent &event) {
index 5083788392c514a8d71d11fb160f9553e4d113bc..c78871da168d3f71745b658ba27c5748d3c617a2 100644 (file)
@@ -29,7 +29,7 @@
 #include <generic-socket-manager.h>
 
 #include <dpl/serialization.h>
-#include <socket-buffer.h>
+#include <message-buffer.h>
 
 #include <security-server-common.h>
 
@@ -40,7 +40,7 @@ class GetObjectNameService  :
   , public SecurityServer::ServiceThread<GetObjectNameService>
 {
 public:
-    typedef std::map<int, SocketBuffer> SocketBufferMap;
+    typedef std::map<int, MessageBuffer> MessageBufferMap;
 
     ServiceDescriptionVector GetServiceDescription();
 
@@ -58,11 +58,11 @@ public:
 
 private:
     std::string m_name;
-    bool readOne(const ConnectionID &conn, SocketBuffer &buffer);
+    bool readOne(const ConnectionID &conn, MessageBuffer &buffer);
     int  setName(gid_t gid);
-    SocketBufferMap m_socketBufferMap;
+    MessageBufferMap m_messageBufferMap;
 };
 
 } // namespace SecurityServer
 
-#endif // _SECURITY_SERVER_SERV_GET_OBJECT_NAME_
\ No newline at end of file
+#endif // _SECURITY_SERVER_SERV_GET_OBJECT_NAME_
index baea49b159723e05e8cfa1a1a5cf421026754a61..878409c0184eed87bac5fcc47ccbffa7a8d82a32 100644 (file)
@@ -81,7 +81,7 @@ void PrivilegeByPidService::write(const WriteEvent &event) {
         m_serviceManager->Close(event.connectionID);
 }
 
-bool PrivilegeByPidService::readOne(const ConnectionID &conn, SocketBuffer &buffer) {
+bool PrivilegeByPidService::readOne(const ConnectionID &conn, MessageBuffer &buffer) {
     LogDebug("Iteration begin");
 
     int retval;
@@ -101,7 +101,7 @@ bool PrivilegeByPidService::readOne(const ConnectionID &conn, SocketBuffer &buff
         Deserialization::Deserialize(buffer, pid);
         Deserialization::Deserialize(buffer, object);
         Deserialization::Deserialize(buffer, access_rights);
-    } Catch (SocketBuffer::Exception::Base) {
+    } Catch (MessageBuffer::Exception::Base) {
         LogDebug("Broken protocol. Closing socket.");
         m_serviceManager->Close(conn);
         return false;
@@ -149,7 +149,7 @@ bool PrivilegeByPidService::readOne(const ConnectionID &conn, SocketBuffer &buff
     else                //there is no permission
         retCode = SECURITY_SERVER_API_ERROR_ACCESS_DENIED;
 
-    SocketBuffer sendBuffer;
+    MessageBuffer sendBuffer;
     Serialization::Serialize(sendBuffer, retCode);
     m_serviceManager->Write(conn, sendBuffer.Pop());
     return true;
@@ -157,7 +157,7 @@ bool PrivilegeByPidService::readOne(const ConnectionID &conn, SocketBuffer &buff
 
 void PrivilegeByPidService::read(const ReadEvent &event) {
     LogDebug("Read event for counter: " << event.connectionID.counter);
-    auto &buffer = m_socketBufferMap[event.connectionID.counter];
+    auto &buffer = m_messageBufferMap[event.connectionID.counter];
     buffer.Push(event.rawBuffer);
 
     // We can get several requests in one package.
@@ -167,7 +167,7 @@ void PrivilegeByPidService::read(const ReadEvent &event) {
 
 void PrivilegeByPidService::close(const CloseEvent &event) {
     LogDebug("CloseEvent. ConnectionID: " << event.connectionID.sock);
-    m_socketBufferMap.erase(event.connectionID.counter);
+    m_messageBufferMap.erase(event.connectionID.counter);
 }
 
 void PrivilegeByPidService::error(const ErrorEvent &event) {
index a4083765170d329aa81451abe8eda4413cc03e24..386812ee14e7e21f4d1b57c2e38a4230be5eaa90 100644 (file)
@@ -27,7 +27,7 @@
 #include <service-thread.h>
 #include <generic-socket-manager.h>
 
-#include <socket-buffer.h>
+#include <message-buffer.h>
 
 namespace SecurityServer {
 
@@ -36,7 +36,7 @@ class PrivilegeByPidService
   , public SecurityServer::ServiceThread<PrivilegeByPidService>
 {
 public:
-    typedef std::map<int, SocketBuffer> SocketBufferMap;
+    typedef std::map<int, MessageBuffer> MessageBufferMap;
 
     ServiceDescriptionVector GetServiceDescription();
 
@@ -52,9 +52,9 @@ public:
     void close(const CloseEvent &event);
     void error(const ErrorEvent &event);
 private:
-    bool readOne(const ConnectionID &conn, SocketBuffer &buffer);
+    bool readOne(const ConnectionID &conn, MessageBuffer &buffer);
 
-    SocketBufferMap m_socketBufferMap;
+    MessageBufferMap m_messageBufferMap;
 };
 
 } // namespace SecurityServer