]> granicus.if.org Git - postgresql/commitdiff
Fix session-lifespan memory leak when a plperl function is redefined:
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 29 Nov 2009 21:02:28 +0000 (21:02 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 29 Nov 2009 21:02:28 +0000 (21:02 +0000)
we have to tell Perl it can release its compiled copy of the function
text.  Noted by Alexey Klyukin.

Back-patch to 8.2 --- the problem exists further back, but this patch
won't work without modification, and it's probably not worth the trouble.

src/pl/plperl/plperl.c

index 6caf66e0b5525641370dd0eb0e0ad8bda73eec9f..0a6f8f7861689561f32a039d00431756cceaa253 100644 (file)
@@ -1,7 +1,7 @@
 /**********************************************************************
  * plperl.c - perl as a procedural language for PostgreSQL
  *
- *       $PostgreSQL: pgsql/src/pl/plperl/plperl.c,v 1.136.2.4 2009/10/31 18:12:12 tgl Exp $
+ *       $PostgreSQL: pgsql/src/pl/plperl/plperl.c,v 1.136.2.5 2009/11/29 21:02:28 tgl Exp $
  *
  **********************************************************************/
 
@@ -1511,11 +1511,13 @@ compile_plperl_function(Oid fn_oid, bool is_trigger)
 
                if (!uptodate)
                {
+                       hash_search(plperl_proc_hash, internal_proname,
+                                               HASH_REMOVE, NULL);
+                       if (prodesc->reference)
+                               SvREFCNT_dec(prodesc->reference);
                        free(prodesc->proname);
                        free(prodesc);
                        prodesc = NULL;
-                       hash_search(plperl_proc_hash, internal_proname,
-                                               HASH_REMOVE, NULL);
                }
        }