Que esta mal en el UPDATE

[granasturcom]

require_once "bd.php";//conecta mi base de datos
if(empty($_POST['submit']) ) //si envia formulario
{
$id=$_POST['id'];
$nombre = $_POST['nombre'];
$antigua = $_POST['antigua'];
$nueva = $_POST['nueva'];
$email = $_POST['email'];
$sql = "UPDATE login SET nombre='$nombre', direccion='$direccion',".
telefono='$telefono', email='$email' WHERE loginid=$id";
$result = mysql_query($sql);
header("Location:Usuarios.php");