Forum PHP.pl

Witam, napisałem taki o to kod i nie wiem czy jest bezpieczny. do pliku login.php leci postem login i haslo z formularza, następnie sprawdza czy istnieje takie w bazie jeżeli tak to tworzy sesje o nazwie admin $_SESSION['admin'] = true; następnie by coś robić w pliku admin.php trzeba mieć tą sesje jeżeli ona nie istnieje to przerzuca na login.html

[PHP] pobierz, plaintext 
// Plik konfiguracyjny
<?php
    class base {
        protected $host="localhost";
        protected $db="base";
        protected $user="root";
        protected $pass="root";
        protected $connect;
 
        public function __construct() {
            $this->connect = new PDO("mysql:host=".$this->host.";dbname=".$this->db,$this->user,$this->pass, array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES 'utf8'"));
        }
    }
 
    $base = new base;
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
// Plik logowania
<?php
   class login extends base{
      function access(){
 
              $this->login = htmlentities($_POST['login']);
              $this->password = htmlentities($_POST['password']);
 
                $login = $this->connect->prepare('SELECT * FROM user WHERE login=:login AND password=:password');
                   $login->bindParam(':login', $this->login);
                   $login->bindParam(':password', MD5($this->password));
                   $login->execute();
                   $rows = $login->rowCount();
                   $dane = $login->fetch();
 
                     if($rows == 1){
                         $_SESSION['admin'] = true;
                         session_regenerate_id();
                         header('Location: admin.php');
                     }else{  $_SESSION['wrong'] = true; exit(); login(); }
 
      }
   }
      $login = new login;  
      $login->access();
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
//Plik dostępu
<?php
   session_start();
   if(!isset($_SESSION['admin'])){ header('Location login.html'); exit(); }
?>
[PHP] pobierz, plaintext 

to niepotrzebne:

[PHP] pobierz, plaintext 
$base = new base;
[PHP] pobierz, plaintext 

a to nielogiczne:

[PHP] pobierz, plaintext 
class login extends base
[PHP] pobierz, plaintext 

właśnie tego tyczyła się rada Liona