mysqli, sha1/salt password valid check

Question

Таким образом, я создаю форму для входа. Предоставление пользователю входных данных, чтобы проверить правильность формата электронной почты и пароль длиннее 6 символов, если они действительны, тогда проверьте, соответствуют ли учетные данные входа действительны, и они совпадают с электронной почтой/паролем, хранящимся в таблице пользователей. В регистрационной форме я шифрую пароль, используя sha1 и данную соль. Когда я проверю правильность пароля , я пишу одну и ту же проверку, но хотя я ввожу правильные логин учетные данные, кажется, невозможно проверить логин. Так вот мой код: Пользователь функция Логин:

function user_login($email, $password){ 
    global $db; 
    $salt='%#@#@**#_-!'; 
    $userid = id_from_email($email); 
    $email = sanitize($email); 
    $password = sha1($password.$salt); 
    $query="SELECT UserId FROM users WHERE Email= '".$email."' and UserPass= '".$password."' "; 
    $result=$db->query($query); 
    $valid = $result->num_rows; 
    return ($valid >1) ? $userid : false; 
} 

А вот часть login.php, которая проверяет, является ли Войти правомерен:

else if(empty($errors) == true) { 
     $userlogin=user_login($email, $password); 
     if ($userlogin==false){ 
     $errors[]='Wrong email/password combination.'; 
     } else { 
     //set the user session 
     session_start(); 
     $_SESSION['UserId']=$userlogin; 
     header('Location:success.php'); 
     } 

Независимо от того, если я ставлю правильный пароль и Эл. адрес. Она всегда выдает «комбинацию Неправильный адрес электронной почты/пароль» А вот как я храню пропуск в базе данных на register.php

$salt='%#@#@**#_-!'; 
$password = sha1($password.$salt); 

Я был бы очень признателен, если вы могли бы помочь мне.

Thankyou.

Answer 1

То, что вы действительно хотите это:

return ($valid >= 1) ? $userid : false; 

вместо

return ($valid > 1) ? $userid : false; 

, если вы не ожидали, что по крайней мере 2 строки должна соответствовать данному адресу электронной почты и хэш пароля.

И я предпочел бы это:

return ($valid == 1) ? $userid : false; 

Answer 2

Изменить вызов к базе данных с соответствующей переменной электронной почты:

$query="SELECT UserId FROM users WHERE Email= '".$useremail."' and UserPass= '".$password."' "; 

Answer 3

Вы выбираете UserId из базы данных, но никогда не использовать этот результат; ваша функция user_login() возвращает результат id_from_email($email), а не идентификатор, полученный от базы данных. Проверьте возвращаемое значение id_from_email(), или еще лучше - верните идентификатор из запроса базы данных.