Fix wrong type size

[platform/core/appfw/pkgmgr-info.git] / src / server / request_handler / create_db_request_handler.cc

diff --git a/src/server/request_handler/create_db_request_handler.cc b/src/server/request_handler/create_db_request_handler.cc
index 91dbf15..c6754e5 100644 (file)
--- a/src/server/request_handler/create_db_request_handler.cc
+++ b/src/server/request_handler/create_db_request_handler.cc
@@ -6,10 +6,11 @@
 
 #include <string>
 
-#include "parcelable_factory.hh"
 #include "create_db_parcelable.hh"
 #include "create_db_handler.hh"
 #include "db_type.hh"
+#include "parcelable_factory.hh"
+#include "utils/logging.hh"
 
 #include "pkgmgr_parser.h"
 
@@ -21,24 +22,24 @@ namespace psd = pkgmgr_server::database;
 namespace pkgmgr_server {
 namespace request_handler {
 
-bool CreateDBRequestHandler::HandleRequest(unsigned char* data, int size,
-                                               std::string locale) {
+bool CreateDBRequestHandler::HandleRequest(unsigned char* data, size_t size,
+    const std::string& locale) {
   auto abstract_parcel =
       pcp::ParcelableFactory::GetInst().CreateParcel(data, size);
 
   if (abstract_parcel == nullptr ||
       abstract_parcel->GetType() != pcp::ParcelableType::CreateDB) {
-    _LOGE("Invalid parcel or type");
+    LOG(ERROR) << "Invalid parcel or type";
     result_ = std::make_shared<pcp::ResultParcelable>(
-        PM_PARSER_R_ERROR, std::vector<std::vector<std::string>>{});
+        PM_PARSER_R_ERROR, std::vector<pcp::StrArgs>{});
     return false;
   }
 
   auto* parcel = dynamic_cast<pcp::CreateDBParcelable*>(abstract_parcel.get());
   if (parcel == nullptr) {
-    _LOGE("Parcel is empty");
+    LOG(ERROR) << "Parcel is empty";
     result_ = std::make_shared<pcp::ResultParcelable>(
-        PM_PARSER_R_ERROR, std::vector<std::vector<std::string>>{});
+        PM_PARSER_R_ERROR, std::vector<pcp::StrArgs>{});
     return false;
   }
 
@@ -50,21 +51,10 @@ bool CreateDBRequestHandler::HandleRequest(unsigned char* data, int size,
   ret = db.Execute();
 
   result_ = std::make_shared<pcp::ResultParcelable>(
-      ret, std::vector<std::vector<std::string>>{});
+      ret, std::vector<pcp::StrArgs>{});
 
   return true;
 }
 
-std::vector<uint8_t> CreateDBRequestHandler::ExtractResult() {
-  tizen_base::Parcel parcel;
-
-  parcel.WriteParcelable(*result_);
-  std::vector<uint8_t> raw = parcel.GetRaw();
-
-  result_.reset();
-
-  return raw;
-}
-
 }  // namespace request_handler
 }  // namespace pkgmgr_server