From: Michael Paquier <michael@paquier.xyz>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: injection_points: Add tests for name limits
Date: Wed, 12 Nov 2025 01:34:09 +0000
Message-ID: <E1vIzkP-0069am-1G@gemulon.postgresql.org> (raw)
injection_points: Add tests for name limits
The maximum limits for point name, library name, function name and
private area size were not kept track of in the tests. The new function
introduced in 16a2f706951e gives a way to trigger them. This is not
critical but cheap to cover.
While on it, this commit cleans up some of the tests introduced by
16a2f706951e for NULL inputs by using more consistent argument values.
The coverage does not change, but it makes the whole less confusing with
argument values that are correct based their position in the SQL
function called.
Reviewed-by: Daniel Gustafsson <daniel@yesql.se>
Reviewed-by: Xuneng Zhou <xunengzhou@gmail.com>
Discussion: https://postgr.es/m/aRE7zhu6wOA29gFf@paquier.xyz
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/fb9bff04542138d7217e79fc5e1bb9d5fabd9026
Modified Files
--------------
.../injection_points/expected/injection_points.out | 21 +++++++++++++++++----
.../injection_points/sql/injection_points.sql | 17 +++++++++++++----
2 files changed, 30 insertions(+), 8 deletions(-)
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: michael@paquier.xyz, pgsql-committers@lists.postgresql.org
Subject: Re: pgsql: injection_points: Add tests for name limits
In-Reply-To: <E1vIzkP-0069am-1G@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