Yep, you didn't declare %in and since you are using strict, this causes a compilation error. I recommend you add this at the top:

use CGI qw(:standard);
use vars qw(%in);

CGI::ReadParse();
[download]

The use vars declares %in as a global variable (you could probably also use our). Personally, i prefer using CGI::param() instead of CGI::ReadParse() - you could construct $body like this instead:

$body .= param($_) . "\n" for param;
[download]

instead, or if you only want the form fields listed in the @param array:

$body .= param($_) . "\n" for @param;
[download]

Hope this helps more. :)

jeffa

L-LL-L--L-LL-L--L-LL-L--
-R--R-RR-R--R-RR-R--R-RR
B--B--B--B--B--B--B--B--
H---H---H---H---H---H---
(the triplet paradiddle with high-hat)