Make EXEC_BACKEND more convenient on macOS.
authorThomas Munro <tmunro@postgresql.org>
Thu, 12 Aug 2021 22:38:22 +0000 (10:38 +1200)
committerThomas Munro <tmunro@postgresql.org>
Thu, 12 Aug 2021 23:09:00 +0000 (11:09 +1200)
It's hard to disable ASLR on current macOS releases, for testing with
-DEXEC_BACKEND.  You could already set the environment variable
PG_SHMEM_ADDR to something not likely to collide with mappings created
earlier in process startup.  Let's also provide a default value that
works on current releases and architectures, for developer convenience.

As noted in the pre-existing comment, this is a horrible hack, but
-DEXEC_BACKEND is only used by Unix-based PostgreSQL developers for
testing some otherwise Windows-only code paths, so it seems excusable.

Back-patch to all supported branches.

Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://postgr.es/m/20210806032944.m4tz7j2w47mant26%40alap3.anarazel.de

src/backend/port/sysv_shmem.c

index 0cc83ffc16af587f236dd52c0f074fb46b0da1f3..9de96edf6ad43cfd057403e5b9720053b9960f62 100644 (file)
@@ -143,6 +143,16 @@ InternalIpcMemoryCreate(IpcMemoryKey memKey, Size size)
 
        if (pg_shmem_addr)
            requestedAddress = (void *) strtoul(pg_shmem_addr, NULL, 0);
+       else
+       {
+#if defined(__darwin__) && SIZEOF_VOID_P == 8
+           /*
+            * Provide a default value that is believed to avoid problems with
+            * ASLR on the current macOS release.
+            */
+           requestedAddress = (void *) 0x80000000000;
+#endif
+       }
    }
 #endif