Subject: | Please use safe open |
Date: | Fri, 30 Oct 2015 20:05:41 +0100 |
To: | bug-Proc-Simple [...] rt.cpan.org |
From: | Christoph Biedl <debian.axhn [...] manchmal.in-ulm.de> |
Hello,
browsing the source in order to understand the redirect_output
semantics, I came across these two gems:
(...)
| open(STDERR, ">$self->{'redirect_stderr'}") ;
(...)
| open(STDOUT, ">$self->{'redirect_stdout'}") ;
(...)
Please change this to the save variant ASAP:
| open(STDERR, '>', $self->{'redirect_stderr'}) ;
And I hope I don't have to explain why this is a serious issue.
Christoph
Message body not shown because it is not plain text.