diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2016-11-02 00:09:28 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2016-11-02 00:09:28 -0400 |
| commit | af636d7b535cf1bcee59a60d22b712ecba5a400f (patch) | |
| tree | 77035f9e6d2b655b501b176ddd8f1348e6abc5c1 | |
| parent | e0491c19d5cf7b398b45b620d238339737078109 (diff) | |
| download | postgresql-af636d7b535cf1bcee59a60d22b712ecba5a400f.tar.gz postgresql-af636d7b535cf1bcee59a60d22b712ecba5a400f.zip | |
Fix portability bug in gin_page_opaque_info().
Somebody apparently thought that "if Int32GetDatum is good,
Int64GetDatum must be better". Per buildfarm failures now
that Peter has added some regression tests here.
| -rw-r--r-- | contrib/pageinspect/ginfuncs.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/contrib/pageinspect/ginfuncs.c b/contrib/pageinspect/ginfuncs.c index c0de3be8df8..5489206ff53 100644 --- a/contrib/pageinspect/ginfuncs.c +++ b/contrib/pageinspect/ginfuncs.c @@ -100,7 +100,7 @@ gin_page_opaque_info(PG_FUNCTION_ARGS) GinPageOpaque opaq; HeapTuple resultTuple; Datum values[3]; - bool nulls[10]; + bool nulls[3]; Datum flags[16]; int nflags = 0; uint16 flagbits; @@ -152,9 +152,9 @@ gin_page_opaque_info(PG_FUNCTION_ARGS) memset(nulls, 0, sizeof(nulls)); values[0] = Int64GetDatum(opaq->rightlink); - values[1] = Int64GetDatum(opaq->maxoff); - values[2] = PointerGetDatum( - construct_array(flags, nflags, TEXTOID, -1, false, 'i')); + values[1] = Int32GetDatum(opaq->maxoff); + values[2] = PointerGetDatum(construct_array(flags, nflags, + TEXTOID, -1, false, 'i')); /* Build and return the result tuple. */ resultTuple = heap_form_tuple(tupdesc, values, nulls); |