pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Tom Lane <tgl@sss.pgh.pa.us>
To: pgsql-committers@postgresql.org
Subject: pgsql: Fix another instance of unsafe coding for shm_toc_lookup failure
Date: Fri, 02 Feb 2018 23:32:15 +0000
Message-ID: <E1ehkoZ-0005uW-43@gemulon.postgresql.org> (raw)

Fix another instance of unsafe coding for shm_toc_lookup failure.

One or another author of commit 5bcf389ec seems to have thought that
computing an offset from a NULL pointer would yield another NULL pointer.
There may possibly be architectures where that works, but common machines
don't work like that.  Per a quick code review of places calling
shm_toc_lookup and not using noError = false.

Branch
------
master

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

Modified Files
--------------
src/backend/executor/nodeHash.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)



view thread (5+ messages)  latest in thread

Message-ID: <E1ehkoZ-0005uW-43@gemulon.postgresql.org>
Permalink:  ../E1ehkoZ-0005uW-43@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1ehkoZ-0005uW-43@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: tgl@sss.pgh.pa.us
  Subject: Re: pgsql: Fix another instance of unsafe coding for shm_toc_lookup failure
  In-Reply-To: <E1ehkoZ-0005uW-43@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox