diff options
author | David Mitchell <davem@iabyn.com> | 2010-09-19 12:49:07 +0100 |
---|---|---|
committer | David Mitchell <davem@iabyn.com> | 2010-09-20 08:16:12 +0100 |
commit | aac9d523d6a5c05209364195355a241de1641c93 (patch) | |
tree | 5911b74c03741361f1aa57fa4034305b42c20360 | |
parent | ca556bcdca736b2f85c11650c70b2371169c0225 (diff) | |
download | perl-aac9d523d6a5c05209364195355a241de1641c93.tar.gz |
[perl #40388] perl_destruct() leaks PL_main_cv
Well yes, it does, kinda; but it's harmless. Add a comment to that effect.
-rw-r--r-- | perl.c | 4 |
1 files changed, 4 insertions, 0 deletions
@@ -751,6 +751,10 @@ perl_destruct(pTHXx) PL_main_root = NULL; } PL_main_start = NULL; + /* note that PL_main_cv isn't usually actually freed at this point, + * due to the CvOUTSIDE refs from subs compiled within it. It will + * get freed once all the subs are freed in sv_clean_all(), for + * destruct_level > 0 */ SvREFCNT_dec(PL_main_cv); PL_main_cv = NULL; PL_dirty = TRUE; |