From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: Avoid copying undefined data in _readA_Const(). |
Date: | 2023-03-19 19:36:24 |
Message-ID: | E1pdypM-004MdY-Ai@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Avoid copying undefined data in _readA_Const().
nodeRead() will have created a Node struct that's only allocated big
enough for the specific node type, so copying sizeof(union ValUnion)
can be copying too much. This provokes valgrind complaints, and with
very bad luck could perhaps result in SIGSEGV.
While at it, tidy up _equalA_Const to avoid duplicate checks of isnull.
Per report from Alexander Lakhin. This code is new as of a6bc33019,
so no need to back-patch.
Discussion: https://postgr.es/m/4995256b-cc65-170e-0b22-60ad2cd535f1@gmail.com
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/e060cd59fabda2a0b2a7e119a58887791d030942
Modified Files
--------------
src/backend/nodes/equalfuncs.c | 8 +++-----
src/backend/nodes/readfuncs.c | 25 ++++++++++++++++++++++++-
2 files changed, 27 insertions(+), 6 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | David Rowley | 2023-03-20 00:30:00 | pgsql: Fix memory leak in Memoize cache key evaluation |
Previous Message | Tom Lane | 2023-03-18 20:12:08 | pgsql: Doc: fix documentation example for bytea hex output format. |