core: add more verbose error info
authorDeng Zhengrong <dzrongg@gmail.com>
Tue, 10 Jul 2012 00:32:54 +0000 (08:32 +0800)
committerArun Raghavan <arun.raghavan@collabora.co.uk>
Tue, 17 Jul 2012 05:42:53 +0000 (11:12 +0530)
I'm debugging an issue, and find that it's more useful for users to know which
directory hasn't been created instead of just knowing error code.

src/pulsecore/core-util.c

index e435cc4..1e40ba0 100644 (file)
@@ -1508,7 +1508,7 @@ char *pa_get_state_dir(void) {
      * dir then this will break. */
 
     if (pa_make_secure_dir(d, 0700U, (uid_t) -1, (gid_t) -1, TRUE) < 0) {
-        pa_log_error("Failed to create secure directory: %s", pa_cstrerror(errno));
+        pa_log_error("Failed to create secure directory (%s): %s", d, pa_cstrerror(errno));
         pa_xfree(d);
         return NULL;
     }
@@ -1650,7 +1650,7 @@ char *pa_get_runtime_dir(void) {
     if (d) {
 
         if (pa_make_secure_dir(d, m, (uid_t) -1, (gid_t) -1, TRUE) < 0) {
-            pa_log_error("Failed to create secure directory: %s", pa_cstrerror(errno));
+            pa_log_error("Failed to create secure directory (%s): %s", d, pa_cstrerror(errno));
             goto fail;
         }
 
@@ -1664,7 +1664,7 @@ char *pa_get_runtime_dir(void) {
 
         if (pa_make_secure_dir(k, m, (uid_t) -1, (gid_t) -1, TRUE) < 0) {
             free(k);
-            pa_log_error("Failed to create secure directory: %s", pa_cstrerror(errno));
+            pa_log_error("Failed to create secure directory (%s): %s", k, pa_cstrerror(errno));
             goto fail;
         }
 
@@ -1677,7 +1677,7 @@ char *pa_get_runtime_dir(void) {
         goto fail;
 
     if (pa_make_secure_dir(d, m, (uid_t) -1, (gid_t) -1, TRUE) < 0) {
-        pa_log_error("Failed to create secure directory: %s", pa_cstrerror(errno));
+        pa_log_error("Failed to create secure directory (%s): %s", d, pa_cstrerror(errno));
         pa_xfree(d);
         goto fail;
     }