int_array_enum function should be using fcinfo->flinfo->fn_extra for

working state, not fcinfo->context.  Silly oversight on my part in last
go-round of fixes.

diff --git a/contrib/intagg/int_aggregate.c b/contrib/intagg/int_aggregate.c
index afe55df881b07553d00eeb982dd652c0676bfbd8..bfbbe073f9860fa545b889952d0941cf4d0e39ee 100644 (file)
--- a/contrib/intagg/int_aggregate.c
+++ b/contrib/intagg/int_aggregate.c
@@ -198,9 +198,9 @@ int_enum(PG_FUNCTION_ARGS)
                PG_RETURN_NULL();
        }
 
-       if (!fcinfo->context)
+       if (!fcinfo->flinfo->fn_extra)
        {
-               /* Allocate a working context */
+               /* Allocate working state */
                MemoryContext   oldcontext;
 
                oldcontext = MemoryContextSwitchTo(fcinfo->flinfo->fn_mcxt);
@@ -224,19 +224,20 @@ int_enum(PG_FUNCTION_ARGS)
                if (pc->p->a.ndim != 1)
                        elog(ERROR, "int_enum only accepts 1-D arrays");
                pc->num = 0;
-               fcinfo->context = (Node *) pc;
+               fcinfo->flinfo->fn_extra = (void *) pc;
                MemoryContextSwitchTo(oldcontext);
        }
-       else    /* use an existing one */
-               pc = (CTX *) fcinfo->context;
+       else    /* use existing working state */
+               pc = (CTX *) fcinfo->flinfo->fn_extra;
+
        /* Are we done yet? */
        if (pc->num >= pc->p->items)
        {
                /* We are done */
                if (pc->flags & TOASTED)
                        pfree(pc->p);
-               pfree(fcinfo->context);
-               fcinfo->context = NULL;
+               pfree(pc);
+               fcinfo->flinfo->fn_extra = NULL;
                rsi->isDone = ExprEndResult;
        }
        else