From 9b80d1d14ad03bc45e7a18d0ee137001ded1e4b3 Mon Sep 17 00:00:00 2001 From: Deng Zhengrong Date: Tue, 10 Jul 2012 08:32:54 +0800 Subject: [PATCH] core: add more verbose error info 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 | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/pulsecore/core-util.c b/src/pulsecore/core-util.c index e435cc408..1e40ba0a6 100644 --- a/src/pulsecore/core-util.c +++ b/src/pulsecore/core-util.c @@ -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; }