diff options
| author | Peter Eisentraut <peter@eisentraut.org> | 2022-08-28 10:47:10 +0200 |
|---|---|---|
| committer | Peter Eisentraut <peter@eisentraut.org> | 2022-09-05 13:30:44 +0200 |
| commit | 6bcda4a72123c3aa29fa3f03d952095675ad4468 (patch) | |
| tree | ccd9c5998367bb93509fce2a5c8c6bd49563f168 /contrib/hstore | |
| parent | 6dc0738812c3fdbd733d917267a0543ab5fdf997 (diff) | |
| download | postgresql-6bcda4a72123c3aa29fa3f03d952095675ad4468.tar.gz | |
Fix incorrect uses of Datum conversion macros
Since these macros just cast whatever you give them to the designated
output type, and many normal uses also cast the output type further, a
number of incorrect uses go undiscovered. The fixes in this patch
have been discovered by changing these macros to inline functions,
which is the subject of a future patch.
Reviewed-by: Aleksander Alekseev <aleksander@timescale.com>
Discussion: https://www.postgresql.org/message-id/flat/8528fb7e-0aa2-6b54-85fb-0c0886dbd6ed%40enterprisedb.com
Diffstat (limited to 'contrib/hstore')
| -rw-r--r-- | contrib/hstore/hstore_op.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/contrib/hstore/hstore_op.c b/contrib/hstore/hstore_op.c index 2f540d7ed6..0d4ec16d1e 100644 --- a/contrib/hstore/hstore_op.c +++ b/contrib/hstore/hstore_op.c @@ -1064,7 +1064,7 @@ hstore_each(PG_FUNCTION_ARGS) tuple = heap_form_tuple(funcctx->tuple_desc, dvalues, nulls); res = HeapTupleGetDatum(tuple); - SRF_RETURN_NEXT(funcctx, PointerGetDatum(res)); + SRF_RETURN_NEXT(funcctx, res); } SRF_RETURN_DONE(funcctx); |