One potential problem: you don't print the CGI header at any point.

Try putting:

print "Content-type: text/html\n\n";
[download]

or better yet:

use CGI qw(standard);

print header();
[download]

at the top of your code, before you print anything.

stephen