First off, there's some issues with syntax and deprecated feature use:
If you have
<form action='process.php' method='post'>
<input type='text' name='cat'>
<input type='submit' value='Submit!'>
</form>
then this should be process.php:
<?php
//print $cat; // BAD!
print $_POST['cat']; // good
print $_REQUEST['cat']; // good
if(eregi("/cat/",$_POST['cat'])) // BAD!
{}
if(preg_match("/cat/",$_POST['cat'])) // good
{}
if($_POST['cat'] != "dog" && $_POST['cat'] != "rabbit") // BAD!
{}
if($_POST['cat'] != "dog" and $_POST['cat'] != "rabbit") // good
{}
if($_POST['cat'] == "cat" || $_POST['cat'] == "kitty") // BAD!
{}
if($_POST['cat'] == "cat" or $_POST['cat'] == "kitty") // good
{}
?>
These aren't requirements, but recommendations. For example, $cat is created from the form using register_globals. Register_globals can be very dangerous, as users can change and create variables when you don't want them to: http://us2.php.net/manual/en/security.registerglobals.php. && and || have precedence issues; you'd probably rather use 'and' and 'or': . http://us2.php.net/manual/en/language.operators.php#language.operators.precedence.ereg is deprected; it's slow, and less feature-filled than preg: http://www.php.net/manual/en/ref.pcre.php.
Also, on this line:
else { mail("$email","$subject","$message","From: $name <$eemail>") or die("email error");
You have one var $eemail, and you say mail("$email"...
For mail() to work, the server needs to have the smtp server property in php.ini set: http://us2.php.net/manual/en/ref.mail.php
Hope this helps.
