From 741b1aaf61a3da10250555f827c0a7a2f9bc2822 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Sun, 3 Nov 2019 10:57:49 -0500 Subject: [PATCH] Fix PG_GETARG_SEG_P() definition. MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit DatumGetPointer() takes a Datum argument, not a pointer. This is cosmetic given the current definitions of the underlying macros, but it's still formally a type violation. Bug was introduced in commit 389bb2818, but there seems no need to back-patch. Dagfinn Ilmari Mannsåker Discussion: https://postgr.es/m/d8jlfsxq3a0.fsf@dalvik.ping.uio.no --- contrib/seg/seg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/contrib/seg/seg.c b/contrib/seg/seg.c index 4e34fba7c7b..f87456405c3 100644 --- a/contrib/seg/seg.c +++ b/contrib/seg/seg.c @@ -19,7 +19,7 @@ #define DatumGetSegP(X) ((SEG *) DatumGetPointer(X)) -#define PG_GETARG_SEG_P(n) DatumGetSegP(PG_GETARG_POINTER(n)) +#define PG_GETARG_SEG_P(n) DatumGetSegP(PG_GETARG_DATUM(n)) /* -- 2.39.5