* posix_ipc.cc (check_path): Fix typo in comment. Align naming

	convention rules to Linux.  Add comment.

2 files changed, 17 insertions, 4 deletions

diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog
index 68c222a78..85604eaca 100644
--- a/winsup/cygwin/ChangeLog
+++ b/winsup/cygwin/ChangeLog
@@ -1,3 +1,8 @@
+2009-06-12  Corinna Vinschen  <corinna@vinschen.de>
+
+	* posix_ipc.cc (check_path): Fix typo in comment.  Align naming
+	convention rules to Linux.  Add comment.
+
 2009-06-09  Corinna Vinschen  <corinna@vinschen.de>
 
 	* autoload.cc (GetSystemTimes): Remove.
diff --git a/winsup/cygwin/posix_ipc.cc b/winsup/cygwin/posix_ipc.cc
index 693b65498..2578f545b 100644
--- a/winsup/cygwin/posix_ipc.cc
+++ b/winsup/cygwin/posix_ipc.cc
@@ -47,7 +47,7 @@ enum ipc_type_t
 static bool
 check_path (char *res_name, ipc_type_t type, const char *name, size_t len)
 {
-  /* Note that we require the existance of the apprpriate /dev subdirectories
+  /* Note that we require the existance of the appropriate /dev subdirectories
      for POSIX IPC object support, similar to Linux (which supports the
      directories, but doesn't require to mount them).  We don't create
      these directory here, that's the task of the installer.  But we check
@@ -65,13 +65,21 @@ check_path (char *res_name, ipc_type_t type, const char *name, size_t len)
       set_errno (EINVAL);
       return false;
     }
-  /* Name must start with a single slash. */
-  if (!name || name[0] != '/' || name[1] == '/' || !name[1])
+  /* Name must not be empty, or just be a single slash, or start with more
+     than one slash.  Same for backslash.
+     Apart from handling backslash like slash, the naming rules are identical
+     to Linux, including the names and requirements for subdirectories, if
+     the name contains further slashes. */
+  if (!name || (strchr ("/\\", name[0])
+		&& (!name[1] || strchr ("/\\", name[1]))))
     {
       debug_printf ("Invalid %s name '%s'", ipc_names[type].description, name);
       set_errno (EINVAL);
       return false;
     }
+  /* Skip leading (back-)slash. */
+  if (strchr ("/\\", name[0]))
+    ++name;
   if (len > PATH_MAX - ipc_names[type].prefix_len)
     {
       debug_printf ("%s name '%s' too long", ipc_names[type].description, name);
@@ -80,7 +88,7 @@ check_path (char *res_name, ipc_type_t type, const char *name, size_t len)
     }
   __small_sprintf (res_name, "%s/%s%s", ipc_names[type].prefix,
 					type == semaphore ? "sem." : "",
-					name + 1);
+					name);
   return true;
 }