On Fri, Apr 24, 2009 at 11:17:55AM +0200, Raphael Hertzog wrote:
> On Thu, 23 Apr 2009, Richard Mawson wrote:
> > 1530       IS->invoice_details(\%myconfig, \%$form);
> 
> This looks wrong, what happens if you use:
> S->invoice_details(\%myconfig, $form);
> 
> $form is a ref
> %$form is the hash behind the ref
> \%$form is a new ref of the hash behind the $form ref

I agree - this does look wrong, but whether invoice_details is operating on
the original ref to form ($form) or a new ref to the same form (\%$form), it 
should be the same form being operated on. 

I did in fact change it to exactly* what you suggest and it made no difference 
to the errors I saw. (* typo s/S/IS/ accepted)

I'm not sure that the errors seen should ever be possible to provoke - even 
with maliciously crafted perl.. I'm getting the feeling this is a bug in the 
perl binary, but I don't really know.


-- 
Richard Mawson
Mawson IT Services

Attachment: signature.asc
Description: Digital signature

Reply via email to