pgsql: Avoid harmless Valgrind no-buffer-pin errors.

From: Peter Geoghegan <pg(at)bowt(dot)ie>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Avoid harmless Valgrind no-buffer-pin errors.
Date: 2020-07-19 23:13:19
Message-ID: E1jxIUh-0000cO-5F@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Avoid harmless Valgrind no-buffer-pin errors.

Valgrind builds with assertions enabled sometimes perform a
theoretically unsafe page access inside an assertion in
heapam_tuple_lock(). This happened when the eval-plan-qual isolation
test ran one of the permutations added by commit a2418f9e238.

Avoid complaints from Valgrind by moving the assertion ever so slightly.
This is minor cleanup for commit 1e0dfd16, which added Valgrind buffer
access instrumentation.

No backpatch, since this only happens within an assertion, and seems
very unlikely to cause any real problems even with assert-enabled
builds.

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/a766d6ca22ac7c233e69c896ae0c5f19de916db4

Modified Files
--------------
src/backend/access/heap/heapam_handler.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Amit Kapila 2020-07-20 02:29:14 pgsql: Fix minor typo in nodeIncrementalSort.c.
Previous Message Peter Geoghegan 2020-07-19 16:47:43 pgsql: Mark buffers as defined to Valgrind consistently.