mirrors/postgres
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Revert previous patch to put the shared memory segment on win32

Browse Source 
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.
This commit is contained in:
Magnus Hagander 2008-10-30 17:04:09 +00:00
parent c8a18a282b
commit 9cc0222680
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/backend/port/win32_shmem.c
View File

@ -6,7 +6,7 @@
* Portions Copyright (c) 1996-2008, PostgreSQL Global Development Group * Portions Copyright (c) 1996-2008, PostgreSQL Global Development Group
* *
* IDENTIFICATION * 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", elog(FATAL, "could not generate full pathname for datadir %s: %lu",
DataDir, GetLastError()); 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 == '\\') if (*cp == '\\')
*cp = '/'; *cp = '/';