On Tue, 2 May 2000, Geoffrey Young wrote:

> ok, for anyone who is interested, I seem to have fixed the problem
> (maybe)...
> 
> here's a patch for Apache.xs from yesterday's cvs (and I didn't see any
> commits since then...)
> 
> --- Apache.xs.old       Tue May  2 14:25:09 2000
> +++ Apache.xs   Tue May  2 14:26:19 2000
> @@ -220,7 +220,7 @@
>  
>      perl_handler_merge_avs(hook, &avcopy);
>  
> -    return newRV_noinc((SV*)avcopy);
> +    return newRV_inc((SV*)avcopy);
>  }
> 
> 
> now, the implications of this are way over my head, but it seems to work
> with the test case below just fine (and my other code with which I initially
> caught the problem).  anyway...

hmm, that could result in a leak.  i'll look into this soon, thanks for
the details and patch!

Reply via email to