Improve error messages for base directory creation

diff --git a/unix/vncpasswd/vncpasswd.cxx b/unix/vncpasswd/vncpasswd.cxx
index 9f794e1299af90b5a9cbbfccc6fdb24abe45ac92..9833ec7bfc6ca9a80bc29a2466fb08417b6cd7a7 100644 (file)
--- a/unix/vncpasswd/vncpasswd.cxx
+++ b/unix/vncpasswd/vncpasswd.cxx
@@ -209,12 +209,13 @@ int main(int argc, char** argv)
   if (fname[0] == '\0') {
     const char *configDir = os::getvncconfigdir();
     if (configDir == nullptr) {
-      fprintf(stderr, "Can't obtain VNC config directory\n");
+      fprintf(stderr, "Could not determine VNC config directory path\n");
       exit(1);
     }
     if (os::mkdir_p(configDir, 0777) == -1) {
       if (errno != EEXIST) {
-        fprintf(stderr, "Could not create VNC config directory: %s\n", strerror(errno));
+        fprintf(stderr, "Could not create VNC config directory \"%s\": %s\n",
+                configDir, strerror(errno));
         exit(1);
       }
     }

diff --git a/unix/vncserver/vncsession.c b/unix/vncserver/vncsession.c
index a10e0789ebf8c66e3613521412e862dd561728fe..0830e81a5d63831b741e90812c9a3b1667ab7773 100644 (file)
--- a/unix/vncserver/vncsession.c
+++ b/unix/vncserver/vncsession.c
@@ -443,7 +443,7 @@ redir_stdio(const char *homedir, const char *display, char **envp)
 
     if (mkdir_p(logdir, 0755) == -1) {
         if (errno != EEXIST) {
-            syslog(LOG_CRIT, "Failure creating \"%s\": %s", logdir, strerror(errno));
+            syslog(LOG_CRIT, "Could not create VNC state directory \"%s\": %s", logdir, strerror(errno));
             _exit(EX_OSERR);
         }
     }

diff --git a/vncviewer/ServerDialog.cxx b/vncviewer/ServerDialog.cxx
index 3bce2b5100907a1d8e1d777eb6bf710fb351ad32..d51b871355a273f8016aa03a91ad47f338f052be 100644 (file)
--- a/vncviewer/ServerDialog.cxx
+++ b/vncviewer/ServerDialog.cxx
@@ -317,7 +317,7 @@ void ServerDialog::loadServerHistory()
 
   const char* stateDir = os::getvncstatedir();
   if (stateDir == nullptr)
-    throw Exception(_("Could not obtain the state directory path"));
+    throw Exception(_("Could not determine VNC state directory path"));
 
   char filepath[PATH_MAX];
   snprintf(filepath, sizeof(filepath), "%s/%s", stateDir, SERVER_HISTORY);
@@ -383,7 +383,7 @@ void ServerDialog::saveServerHistory()
 
   const char* stateDir = os::getvncstatedir();
   if (stateDir == nullptr)
-    throw Exception(_("Could not obtain the state directory path"));
+    throw Exception(_("Could not determine VNC state directory path"));
 
   char filepath[PATH_MAX];
   snprintf(filepath, sizeof(filepath), "%s/%s", stateDir, SERVER_HISTORY);

diff --git a/vncviewer/parameters.cxx b/vncviewer/parameters.cxx
index a03623db74edaebd2efd068bcede8f4c48793dc4..2e8ad7a13569067b5ca4abf0482c7d75294a0135 100644 (file)
--- a/vncviewer/parameters.cxx
+++ b/vncviewer/parameters.cxx
@@ -631,7 +631,7 @@ void saveViewerParameters(const char *filename, const char *servername) {
     
     const char* configDir = os::getvncconfigdir();
     if (configDir == nullptr)
-      throw Exception(_("Could not obtain the config directory path"));
+      throw Exception(_("Could not determine VNC config directory path"));
 
     snprintf(filepath, sizeof(filepath), "%s/default.tigervnc", configDir);
   } else {
@@ -735,7 +735,7 @@ char* loadViewerParameters(const char *filename) {
 
     const char* configDir = os::getvncconfigdir();
     if (configDir == nullptr)
-      throw Exception(_("Could not obtain the config directory path"));
+      throw Exception(_("Could not determine VNC config directory path"));
 
     snprintf(filepath, sizeof(filepath), "%s/default.tigervnc", configDir);
   } else {

diff --git a/vncviewer/vncviewer.cxx b/vncviewer/vncviewer.cxx
index c206427b22d3101086e6b1b3d77f8e4eb423ab1c..24099e12e18b1b861984f405adb2cc0a68381e17 100644 (file)
--- a/vncviewer/vncviewer.cxx
+++ b/vncviewer/vncviewer.cxx
@@ -560,7 +560,8 @@ create_base_dirs()
 
   if (os::mkdir_p(dir, 0755) == -1) {
     if (errno != EEXIST)
-      vlog.error(_("Could not create VNC config directory: %s"), strerror(errno));
+      vlog.error(_("Could not create VNC config directory \"%s\": %s"),
+                 dir, strerror(errno));
   }
 
   dir = os::getvncdatadir();
@@ -571,7 +572,8 @@ create_base_dirs()
 
   if (os::mkdir_p(dir, 0755) == -1) {
     if (errno != EEXIST)
-      vlog.error(_("Could not create VNC data directory: %s"), strerror(errno));
+      vlog.error(_("Could not create VNC data directory \"%s\": %s"),
+                 dir, strerror(errno));
   }
 
   dir = os::getvncstatedir();
@@ -582,7 +584,8 @@ create_base_dirs()
 
   if (os::mkdir_p(dir, 0755) == -1) {
     if (errno != EEXIST)
-      vlog.error(_("Could not create VNC state directory: %s"), strerror(errno));
+      vlog.error(_("Could not create VNC state directory \"%s\": %s"),
+                 dir, strerror(errno));
   }
 }