develooper Front page | perl.perl5.porters | Postings from September 2011

Re: [perl #98092] "Attempt to free unreferenced scalar" from dist/threads-shared/t/clone.t

Thread Previous | Thread Next
From:
Karl Williamson
Date:
September 1, 2011 06:55
Subject:
Re: [perl #98092] "Attempt to free unreferenced scalar" from dist/threads-shared/t/clone.t
Message ID:
4E5F8EA5.3060005@khwilliamson.com
On 08/31/2011 10:34 PM, Father Chrysostomos via RT wrote:
> On Wed Aug 31 20:41:39 2011, sprout wrote:
>> The other warnings are also the result of padlists in @DB::args.  If
>> they get cloned through @DB::args, their refcounts are not incremented
>> (except for that held by param->unreferenced), hence the double free.
>>
>> Is there any reason padlists can’t be marked AvREAL (so sv_dup_common
>> knows to use sv_dup_*inc* on the elems)? In cv_undef, we could turn off
>> the REAL flag just before:
>>
>> 	SvREFCNT_dec(MUTABLE_SV(CvPADLIST(cv)));
>> 	CvPADLIST(cv) = NULL;
>
> See the smoke-me/avreal-padlist branch.  All tests pass for me.  Is
> anyone reading my monologue?
>
>
I am
Thread Previous | Thread Next

nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About