[PHP/MySQL] Prosty panel logowania Opcje

[sleedz]

Witam.
Stworzyłem prosty panel logowania. Może komuś się przyda. Piszcie co w nim można zmienić...

index.php

[PHP] pobierz, plaintext 
<?php
 
    session_start();
    
    include "config.php";
    
    if (isset($_SESSION['user_id'])) {
        echo "zalogowałeś się jako <b>".$user_name."</b><br />";
        echo "E-mail: ".$user_email."<br>
                Data dołączenia: ".$user_date."<br>";
        echo "<a href=\"logout.php\">[LOGOUT]</a>";
    }
    else {
        
        echo "<form method=\"GET\" action=\"login.php\">
                <input type=\"text\" name=\"login\"><br>
                <input type=\"password\" name=\"pass\"><br>
                <input type=\"submit\" name=\"log\" value=\"Login\">
              </form>";
            
    }
            
?>
[PHP] pobierz, plaintext 

config.php

[PHP] pobierz, plaintext 
<?php
 
    session_start();
 
    $db_host = "localhost";
    $db_user = "root";
    $db_pass = "root";
    $db_base = "baza";
    
    $db = mysql_connect($db_host, $db_user, $db_pass);
    mysql_select_db($db_base, $db);
    
    if (isset($_SESSION['user_id'])) {
        $user_id     = $_SESSION['user_id'];
        $user_name   = $_SESSION['user_name'];
        $user_email  = $_SESSION['user_email'];
        $user_rights = $_SESSION['user_rights'];
        $user_date   = $_SESSION['user_date'];
    }
 
?>
[PHP] pobierz, plaintext 

login.php

[PHP] pobierz, plaintext 
<?php
 
    session_start();
    
    include "config.php";
 
    $user_login = $_GET['login'];
    $user_pass  = $_GET['pass'];
    
    $sql = "SELECT * FROM user WHERE user_name = '".$user_login."' AND user_pass = '".$user_pass."';";
    $result = mysql_query($sql)
        or die("Podałeś błędny login lub hasło");
        
    $rows = mysql_num_rows($result);
    
    if ($rows == 1) {
        
        $r = mysql_fetch_assoc($result);
        session_register("user_id");
        session_register("user_name");
        session_register("user_email");
        session_register("user_rights");
        session_register("user_date");
        
        $_SESSION['user_id']     = $r['user_id'];
        $_SESSION['user_name']   = $r['user_name'];
        $_SESSION['user_email']  = $r['user_email'];
        $_SESSION['user_rights'] = $r['user_rights'];
        $_SESSION['user_date']   = $r['user_date'];
        
        header("Location: index.php");
    }
    else {
        echo "Podałeś błędny login lub hasło... <br> <a href=\"index.php\">Powrót</a>";
    }
 
?>
[PHP] pobierz, plaintext 

logout.php

[PHP] pobierz, plaintext 
<?php
 
    session_start();
    session_destroy();
    
    header("Location: index.php");
 
?>
[PHP] pobierz, plaintext 

No i oczywiście trzeba stworzyć odpowiednią tabelę w bazie:

[SQL] pobierz, plaintext 
CREATE TABLE `user` (
  `user_id` int(30) NOT NULL AUTO_INCREMENT,
  `user_name` varchar(50) collate latin2_bin DEFAULT NULL,
  `user_pass` varchar(50) collate latin2_bin DEFAULT NULL,
  `user_email` varchar(50) collate latin2_bin DEFAULT NULL,
  `user_date` date DEFAULT NULL,
  PRIMARY KEY  (`user_id`)
);
[SQL] pobierz, plaintext 

[erix]

echo "<form method=\"GET\" action=\"login.php\">

Via GET? :| A jak ktoś Ci się gapi przez ramię na pasek adresu przeglądarki, to co wtedy?

[PHP] pobierz, plaintext 
<?php
$sql = "SELECT * FROM user WHERE user_name = '".$user_login."' AND user_pass = '".
?>
[PHP] pobierz, plaintext 

Podatne na SQL Injection.

Taka dygresja:

[PHP] pobierz, plaintext 
<?php
if (isset($_SESSION['user_id'])) {
       $user_id     = $_SESSION['user_id'];
       $user_name   = $_SESSION['user_name'];
       $user_email  = $_SESSION['user_email'];
       $user_rights = $_SESSION['user_rights'];
       $user_date   = $_SESSION['user_date'];
   }
?>
[PHP] pobierz, plaintext 

Po co powielać parę razy to samo?

[Norbit_24]

[PHP] pobierz, plaintext 
<?php
$sql = "SELECT * FROM user WHERE user_name = '".$user_login."' AND user_pass = '".
?>
[PHP] pobierz, plaintext 

Podatne na SQL Injection.

Co zrobic aby nie było poddatne na sql injection...?