More error messages 51/65851/4
authorZbigniew Jasinski <z.jasinski@samsung.com>
Wed, 13 Apr 2016 14:05:05 +0000 (16:05 +0200)
committerGerrit Code Review <gerrit@review.vlan103.tizen.org>
Wed, 13 Apr 2016 14:55:07 +0000 (07:55 -0700)
Some error messages were missing. One could get misleading error messages.
For example, during app installation, if app directory doesn't exist,
one get "Failed getting app dir for user uid: ..."

* added more error messages
* added errno info for realpath()

Change-Id: I1cddc007b53417ca664e40a08fd60cf05adb9654
Signed-off-by: Zbigniew Jasinski <z.jasinski@samsung.com>
src/common/service_impl.cpp

index 884b37d..9d3da26 100755 (executable)
@@ -212,24 +212,32 @@ bool ServiceImpl::getUserAppDir(const uid_t &uid, std::string &userAppDir)
 {
     struct tzplatform_context *tz_ctx = nullptr;
 
-    if (tzplatform_context_create(&tz_ctx))
-            return false;
+    if (tzplatform_context_create(&tz_ctx)) {
+        LogError("Error in tzplatform_context_create()");
+        return false;
+    }
 
     std::unique_ptr<struct tzplatform_context, decltype(tzplatform_context_destroy)*> tz_ctxPtr(
         tz_ctx, &tzplatform_context_destroy);
 
-    if (tzplatform_context_set_user(tz_ctxPtr.get(), uid))
+    if (tzplatform_context_set_user(tz_ctxPtr.get(), uid)) {
+        LogError("Error in tzplatform_context_set_user()");
         return false;
+    }
 
     enum tzplatform_variable id =
             (uid == getGlobalUserId()) ? TZ_SYS_RW_APP : TZ_USER_APP;
     const char *appDir = tzplatform_context_getenv(tz_ctxPtr.get(), id);
-    if (!appDir)
+    if (!appDir) {
+        LogError("Error in tzplatform_context_getenv()");
         return false;
+    }
 
     std::unique_ptr<char, decltype(free)*> real_pathPtr(realpath(appDir, NULL), free);
-    if (!real_pathPtr.get())
+    if (!real_pathPtr.get()) {
+        LogError("Error in realpath(): " << GetErrnoString(errno) << " for: " << appDir);
         return false;
+    }
 
     userAppDir.assign(real_pathPtr.get());