From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: Fix mislabeling of PROC_QUEUE->links as PGPROC, fixing UBSan on |
Date: | 2022-11-19 21:56:50 |
Message-ID: | E1owVpR-000vY4-Jg@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Fix mislabeling of PROC_QUEUE->links as PGPROC, fixing UBSan on 32bit
ProcSleep() used a PGPROC* variable to point to PROC_QUEUE->links.next,
because that does "the right thing" with SHMQueueInsertBefore(). While that
largely works, it's certainly not correct and unnecessary - we can just use
SHM_QUEUE* to point to the insertion point.
Noticed when testing a 32bit of postgres with undefined behavior
sanitizer. UBSan noticed that sometimes the supposed PGPROC wasn't
sufficiently aligned (required since 46d6e5f5679, ensured indirectly, via
ShmemAllocRaw() guaranteeing cacheline alignment).
For now fix this by using a SHM_QUEUE* for the insertion point. Subsequently
we should replace all the use of PROC_QUEUE and SHM_QUEUE with ilist.h, but
that's a larger change that we don't want to backpatch.
Backpatch to all supported versions - it's useful to be able to run postgres
under UBSan.
Reviewed-by: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Discussion: https://postgr.es/m/20221117014230.op5kmgypdv2dtqsf@awork3.anarazel.de
Backpatch: 11-
Branch
------
REL_12_STABLE
Details
-------
https://git.postgresql.org/pg/commitdiff/4cbcb7ed85b0e235300059dbbbdb83aec4ec8eaf
Modified Files
--------------
src/backend/storage/lmgr/proc.c | 24 ++++++++++++++----------
1 file changed, 14 insertions(+), 10 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Joe Conway | 2022-11-19 22:10:57 | Re: pgsql: Fix typos and bump catversion. |
Previous Message | Tom Lane | 2022-11-19 18:43:01 | pgsql: Disable debug_discard_caches in test_oat_hooks test. |