Revert previous patch to put the shared memory segment on win32

in the Global\ namespace, because it caused permission errors on
a lot of platforms.

We need to come up with something better for 8.4, but for now
revert to the pre-8.3.4 behaviour.

diff --git a/src/backend/port/win32_shmem.c b/src/backend/port/win32_shmem.c
index 066e0283a11bebd670dda9ac3a116d192fe72966..930f602b18306f13108f49edbd1d000f8a998021 100644 (file)
--- a/src/backend/port/win32_shmem.c
+++ b/src/backend/port/win32_shmem.c
@@ -6,7 +6,7 @@
  * Portions Copyright (c) 1996-2008, PostgreSQL Global Development Group
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/port/win32_shmem.c,v 1.5 2008/07/04 10:50:18 mha Exp $
+ *       $PostgreSQL: pgsql/src/backend/port/win32_shmem.c,v 1.6 2008/10/30 17:04:09 mha Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -57,7 +57,13 @@ GetSharedMemName(void)
                elog(FATAL, "could not generate full pathname for datadir %s: %lu",
                         DataDir, GetLastError());
 
-       for (cp = retptr + 18; *cp; cp++)
+       /* 
+        * XXX: Intentionally overwriting the Global\ part here. This was not the
+        * original approach, but putting it in the actual Global\ namespace
+        * causes permission errors in a lot of cases, so we leave it in
+        * the default namespace for now.
+        */
+       for (cp = retptr; *cp; cp++)
                if (*cp == '\\')
                        *cp = '/';