]> git.ipfire.org Git - thirdparty/postgresql.git/commit
refint: Fix segfault in check_foreign_key().
authorNathan Bossart <nathan@postgresql.org>
Thu, 14 May 2026 18:11:49 +0000 (13:11 -0500)
committerNathan Bossart <nathan@postgresql.org>
Thu, 14 May 2026 18:11:49 +0000 (13:11 -0500)
commited0c4d5af2ef4b3d0880aba768fe7948f3e3e1e6
tree14ed484a989ae8eabdc9fe931eca55f8a35095cd
parent1b0dd08157bf945909849c5e73d9e3f5b057c63b
refint: Fix segfault in check_foreign_key().

When an UPDATE statement triggers check_foreign_key() with the
action set to "cascade", it generates more UPDATE statements to
modify the key values in referencing relations.  If a new key value
is NULL, SPI_getvalue() returns a NULL pointer, which is
subsequently passed to quote_literal_cstr(), causing a segfault.
To fix, skip quoting when a new key value is NULL and insert an
unquoted NULL keyword instead.

Oversight in commit 260e97733b.  While the refint documentation
recommends marking primary key columns NOT NULL, the aforementioned
scenario accidentally worked on platforms where snprintf()
substitutes "(null)" for NULL pointers.  Note that for
character-type columns, the old code quoted "(null)" as a string
literal, so this didn't always produce correct results.  But it
still seems better to fix this than to reject cases that previously
worked.

Reported-by: Nikita Kalinin <n.kalinin@postgrespro.ru>
Author: Ayush Tiwari <ayushtiwari.slg01@gmail.com>
Reviewed-by: Pierre Forstmann <pierre.forstmann@gmail.com>
Discussion: https://postgr.es/m/19476-bd04ea6241345303%40postgresql.org
Backpatch-through: 14
contrib/spi/refint.c