Question

Can someone run their eyes over this statement? I keep getting a syntax error and I’m stumped as to what is wrong.

mysql_query("INSERT INTO emails (to, from, subject, content, ip) VALUES('$email_to', '$email_from', '$subject', '$content', '$ip' ) ") 

Thanks!

EDIT: Error

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ''to', 'from', subject, content, ip) VALUES('[MY EMAIL ADDRESS]', 'l', 'hi', ' at line 1

EDIT 2:

I have sanitized.

$email_to = mysql_real_escape_string($_POST['email_to']);
$email_from = mysql_real_escape_string($_POST['email_from']);
$subject = mysql_real_escape_string($_POST['subject']);
$content = mysql_real_escape_string($_POST['content']);

Answer 1

Try this:

mysql_query(
    "INSERT INTO emails (`to`, `from`, subject, content, ip) 
    VALUES('$email_to', '$email_from', '$subject', '$content', '$ip' )") 

I think the error raises because from is a reserved word… backticks should solve this problem.
Remember you MUST always sanitize user input to avoid SQL injection!!