http://www.perlmonks.org?node_id=564768


in reply to Re^2: Mad, bad and dangerous to know
in thread Mad, bad and dangerous to know

Do you really think that Matt's code is easy to read? I think it looks horrible. It's massively overcomplicated, it's full of outdated Perl 4 idioms and it doesn't make any use of modules which would reduce it in length by about a half.

But my biggest problem with it is the number of people who have written other bad code following Matt's example. I'd guess that at least half of the broken, hand-coded CGI parsers that I see are based on Matt's code.

Update: It's also worth noting that whilst the latest version of Matt's formmail is (as far as I can see) free from security holes, older versions that are famously insecure are still in use on hundreds of web servers.

I know that the nms versions of the program are more secure and better written. I hope that they are also easier to read.

--
<http://dave.org.uk>

"The first rule of Perl club is you do not talk about Perl club."
-- Chip Salzenberg

Replies are listed 'Best First'.
Re^4: Mad, bad and dangerous to know
by gellyfish (Monsignor) on Jul 31, 2006 at 15:33 UTC

    I'm pretty certain that the "un-webify" code that is everywhere came from some early Matt program.

    ?J\