postgres.git / summary / log / commit / refs

commit    b0313f9cc8f54d6a5c12f8987c9b6afa0a5bbced
Author:   Robert Haas <rhaas@postgresql.org>
Date:     Fri Jan 26 14:51:15 2018 +0000
First released in: 11.0, 10.2

    pageinspect: Fix use of wrong memory context by hash_page_items.
    
    This can cause it to produce incorrect output.
    
    Report and patch by Masahiko Sawada.
    
    Discussion: http://postgr.es/m/CAD21AoBc5Asx7pXdUWu6NqU_g=Ysn95EGL9SMeYhLLduYoO_OA@mail.gmail.com


 contrib/pageinspect/hashfuncs.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/contrib/pageinspect/hashfuncs.c b/contrib/pageinspect/hashfuncs.c
index 3d0e3f9757e..99b61b8669f 100644
--- a/contrib/pageinspect/hashfuncs.c
+++ b/contrib/pageinspect/hashfuncs.c
@@ -313,10 +313,10 @@ hash_page_items(PG_FUNCTION_ARGS)
 
 		fctx = SRF_FIRSTCALL_INIT();
 
-		page = verify_hash_page(raw_page, LH_BUCKET_PAGE | LH_OVERFLOW_PAGE);
-
 		mctx = MemoryContextSwitchTo(fctx->multi_call_memory_ctx);
 
+		page = verify_hash_page(raw_page, LH_BUCKET_PAGE | LH_OVERFLOW_PAGE);
+
 		uargs = palloc(sizeof(struct user_args));
 
 		uargs->page = page;


[parent: c1869542b3a4]