"or return;" doesn't tell you much about what went wrong, if Net::FTP has anything to tell you. On the call to new, use "or die $@;", and on all other calls, use "or die $ftp->message();".