From b58263c3df6dc2eb918bde92eb04f7fd5d2a061b Mon Sep 17 00:00:00 2001 From: Peter Eisentraut Date: Mon, 24 Nov 2025 11:02:35 +0100 Subject: [PATCH 2/6] Use better DatumGet* function Use DatumGetCString() instead of DatumGetPointer() for returning a C string. Right now, they are the same, but that doesn't always have to be so. --- src/test/modules/test_resowner/test_resowner_basic.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/test/modules/test_resowner/test_resowner_basic.c b/src/test/modules/test_resowner/test_resowner_basic.c index 8f794996371..b84ec2cb299 100644 --- a/src/test/modules/test_resowner/test_resowner_basic.c +++ b/src/test/modules/test_resowner/test_resowner_basic.c @@ -35,13 +35,13 @@ static const ResourceOwnerDesc string_desc = { static void ReleaseString(Datum res) { - elog(NOTICE, "releasing string: %s", DatumGetPointer(res)); + elog(NOTICE, "releasing string: %s", DatumGetCString(res)); } static char * PrintString(Datum res) { - return psprintf("test string \"%s\"", DatumGetPointer(res)); + return psprintf("test string \"%s\"", DatumGetCString(res)); } /* demonstrates phases and priorities between a parent and child context */ -- 2.52.0