[PATCH] fix segfault with DO and plperl/plperlu - Mailing list pgsql-hackers

From Alex Hunsaker
Subject [PATCH] fix segfault with DO and plperl/plperlu
Date
Msg-id x2q34d269d41004172348hbc78d168y5e913a951be91ca2@mail.gmail.com
Whole thread Raw
Responses Re: [PATCH] fix segfault with DO and plperl/plperlu  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
If you do:

# DO $do$ 1; $do$ LANGUAGE plperlu;
# DO $do$ 1; $do$ LANGUAGE plperl;

You get a segfault as we try to SvREFCNT_dec(...); for the wrong
interpreter.  To fix push down the restore_context() so that we do the
above on the correct perl interpreter.
--
 *** a/src/pl/plperl/plperl.c
--- b/src/pl/plperl/plperl.c
***************
*** 1154,1170 **** plperl_inline_handler(PG_FUNCTION_ARGS)
      PG_CATCH();
      {
          current_call_data = save_call_data;
-         restore_context(oldcontext);
          if (desc.reference)
              SvREFCNT_dec(desc.reference);
          PG_RE_THROW();
      }
      PG_END_TRY();

      current_call_data = save_call_data;
-     restore_context(oldcontext);
      if (desc.reference)
          SvREFCNT_dec(desc.reference);

      error_context_stack = pl_error_context.previous;

--- 1154,1170 ----
      PG_CATCH();
      {
          current_call_data = save_call_data;
          if (desc.reference)
              SvREFCNT_dec(desc.reference);
+         restore_context(oldcontext);
          PG_RE_THROW();
      }
      PG_END_TRY();

      current_call_data = save_call_data;
      if (desc.reference)
          SvREFCNT_dec(desc.reference);
+     restore_context(oldcontext);

      error_context_stack = pl_error_context.previous;

Attachment

pgsql-hackers by date:

Previous
From: Josh Kupershmidt
Date:
Subject: patch: Distinguish between unique indexes and unique constraints
Next
From: Simon Riggs
Date:
Subject: Re: testing HS/SR - 1 vs 2 performance