Witam, napisałem skrypt logowania do strony i chciałbym żebyście wskazali mi ewentualne błędy w zabezpieczeniu tego skryptu.

[PHP] pobierz, plaintext 
<?php
    session_start();
    if (isset($_POST['login']) && (isset($_POST['password']))) {
        require_once("connect.php");
        mysqli_report(MYSQLI_REPORT_STRICT);
        try {
            $connect = new mysqli($host, $db_user, $db_password, $db_name);
            if ($connect->connect_errno != 0) {
                throw new Exception(mysqli_connect_errno());
            } else {
                $veryfication = true;
                $login = $_POST['login'];
                $var_login = filter_var($login, FILTER_SANITIZE_STRING);
                $password = $_POST['password'];
                $hash_pass = hash('sha512', $password);
                $result = $connect->query("SELECT * FROM users WHERE login LIKE '$var_login'");
                $how_much_user = $result->num_rows;
                if ($how_much_user == 1) {
                    $record = $result->fetch_assoc();
                    $user_pass = $record['password'];
                    if ($hash_pass !== $user_pass) {
                        $veryfication = false;
                    }
                } else {
                    $veryfication = false;
                }
                if ($veryfication == true) {
                    echo "Udalo Ci sie zalogowac!<br>Witaj ".$var_login."!";
                } else {
                    echo "bledne danie logowania;
                }
 
                $connect->close();
            }
        } catch(Exception $error) {
            echo '<span style="color:red;">Błąd serwera! Przepraszamy za niedogodności. Prosimy o rejestrację w innym terminie.</span>';
	    echo '<br>Informacja dla dewelopera: '.$error;
        }
    } else {
        header("Location: ../log-in");
    }
?>
[PHP] pobierz, plaintext 

Dwie rzeczy:

1. Zapoznaj się z prepared statements.
2. Hasło możesz sprawdzić na poziomie bazy danych, nie musisz pobierać danych użytkownika.

Nie mam pojęcia jak mam się za to zabrać czytam na php manual i nie wiem jak do tego podejść jak by ktoś mógł podać jakiś przykład albo w skrócie o tym opowiedzieć.



Poprawiłem by hasło sprawdzało na poziomie bazy danych

[PHP] pobierz, plaintext 
try {
    $connect = @new mysqli($host, $db_user, $db_password, $db_name);
    if ($connect->connect_errno) {
        throw @new Exception($connect->error);
    } else {
        $login = $_POST['login'];
        $login = filter_var($login, FILTER_SANITIZE_STRING);
        $password = $_POST['password'];
        $password = hash('sha512', $password);
        if (!$result = $connect->query(sprintf("SELECT * FROM users WHERE login LIKE '%s' AND password LIKE '%s'",
        mysqli_real_escape_string($connect, $login),
        mysqli_real_escape_string($connect, $password)))) {
            throw @new Exception($connect->error);
        } else {
            $how_much_users = $result->num_rows;
            if ($how_much_users == 1) {
                echo "udalo Ci sie zalogowac";
            } else {
                echo "bledne dane logowania";
            }
        }
    }
} catch(Exception $error) {
    echo $error;
}
[PHP] pobierz, plaintext 

http://php.net/manual/en/mysqli.prepare.php

ps:
$how_much_users
uzytkownicy sa policzalni wiec nie much a many.
A juz lepiej to w ogole: $usersCount

Czy to o to chodziło?? Wszystko działa tylko pytanie czy teraz jest bezpieczne ^^

[PHP] pobierz, plaintext 
try {
    $connect = @new mysqli($host, $db_user, $db_password, $db_name);
    if ($connect->connect_errno) {
        throw @new Exception($connect->error);
    } else {
        $login = $_POST['login'];
        $login = filter_var($login, FILTER_SANITIZE_STRING);
        $password = $_POST['password'];
        $password = hash('sha512', $password);
        if ($stmt = mysqli_prepare($connect, "SELECT password FROM users WHERE login=? ")) {
            mysqli_stmt_bind_param($stmt, "s", $login);
            mysqli_stmt_execute($stmt);
            mysqli_stmt_bind_result($stmt, $user_password);
            mysqli_stmt_fetch($stmt);
            mysqli_stmt_close($stmt);
            if ($password == $user_password) {
                echo "udalo Ci sie zalogowac";
            } else {
                echo "logowanie nie powiodlo sie";
            }
        }
        mysqli_close($connect);
    }
} catch(Exception $error) {
    echo $error;
}
[PHP] pobierz, plaintext 

No i 2 sprawa jak teraz zrobić porównanie hasła na poziomie bazy danych

Tak, o to chodzilo.
Co do hasla to bindujesz je tak samo jak login. Mozesz bindowac wiele wartosci a nie tylko jedna. W czym problem?

Ja bym jeszcze dodał poziom błędów na exception i pozbył się tego pierwszego ifa.