From: Ossama Othman <ossama.othman@intel.com>
Date: Mon, 7 Oct 2013 04:16:37 +0000 (-0700)
Subject: [manager] Return immediately on empty request.
X-Git-Tag: accepted/tizen/20131106.223055~17
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6bc8120b12fc2eff49679e3d18a9ae3054aa8399;p=profile%2Fivi%2Fsettings-daemon.git

[manager] Return immediately on empty request.

Change-Id: Ib2b2b6743d119666757c79347a15142d94053ec1
Signed-off-by: Ossama Othman <ossama.othman@intel.com>
---

diff --git a/lib/manager.cpp b/lib/manager.cpp
index 0b497c9..8fd64b0 100644
--- a/lib/manager.cpp
+++ b/lib/manager.cpp
@@ -105,7 +105,8 @@ ivi::settings::manager::load_settings(std::string const & dir)
          * @todo I really hate catching an exception like this, but I
          *       really don't want to resort to a construct/init()
          *       style of object initialization.  Fix.
-         */std::cerr << "Error loading plugin: " << e.what() << "\n";
+         */
+        std::cerr << "Error loading plugin: " << e.what() << "\n";
       }
     }
 
@@ -130,6 +131,7 @@ ivi::settings::manager::dispatch(std::string request,
   if (request.empty()) {
     response_callback response(wsi, std::string(), std::string());
     response.send_error("Empty settings request.");
+    return;
   }
 
   smart_ptr<JsonParser> const parser(json_parser_new());