Forum PHP.pl

WItam,

Napisałem prosty skrypt logowania.
Proszę o opinie co można zmienić żeby podnieść bezpieczeństwo

[PHP] pobierz, plaintext 
<?php
session_start();
 
if (!isset($_SESSION['inicjuj']))
   {
      session_regenerate_id();
      $_SESSION['inicjuj'] = true;
      $_SESSION['ip'] = $_SERVER['REMOTE_ADDR'];
 
   }
if($_SESSION['ip'] != $_SERVER['REMOTE_ADDR'])
   {
      session_unset();
      session_destroy();
      die();
   }
 
 
require('langs/lang.class.php');
$txt='';
 if ($_SESSION['login'] == "zalogowany")
   {
      include("include/header.php");
   }
 else
   {
      $txt.="<FORM action=\"index.php\" method=\"POST\">";
	$txt.="<TABLE align=\"center\" bgcolor=\"#fffaaa\" border=\"0\" width=\"350px\" CELLPADDING=\"2\" CELLSPACING=\"2\">";
        $txt.="<TR>
          <TD>
            <h2>". __('login_nazwa')."</h2>
            ".__('login_login')."<br />
            <input type=\"textbox\" name=\"u\" size=\"15\">
          </TD>
        </TR>
        <TR>
	     <TD>".__('login_pass')."<br />
	       <input type=\"password\" name=\"p\" size=\"15\">
	       <input type=\"hidden\" name=\"log\">
	     </TD>
        </TR>
        <TR>
           <TD>
             <br />
             <input class=\"wyslij_formularz\" align=\"center\" type=\"submit\" name=\"submit\" value=\"".__('login_zaloguj')."\">
	     </TD>
	</form>";
 
        if (isset($_POST['log']))
          {
            #--pobranie danych dotyczących logowania
            $login = $_POST['u'];
            $pass =  $_POST['p'];
            #--sprawdzenie czy wsyzstkie dane są wpisane
            if(empty($login) || empty($pass))
                {
                           $txt.="<tr><td>";
                           $txt.=__('login_blad');
                           $txt.="</td></tr>";
                }
              else
                {
                  #--kodowanie hasła
                  $login = trim(strip_tags(mysql_real_escape_string(HTMLSpecialChars($login))));
                  $pass = trim(strip_tags(mysql_real_escape_string(HTMLSpecialChars($pass))));
                  $pass = sha1($pass);
 
                  require("!konfiguracja/bd.php");
                  $logowanie=mysql_query("SELECT u_id FROM users WHERE u_login='$login' AND u_haslo='$pass'");
                      if(mysql_num_rows($logowanie)==0)
                        {
                           $txt.="<tr><td>";
                           $txt.=__('login_blad2');
                           $txt.="</td></tr>";
                        }
                      else
                        {
                           $row = mysql_fetch_array($logowanie);
                           $_SESSION['login'] = "zalogowany";
                           $_SESSION['id'] = $row['u_id'];
                           $_SESSION['ip'] = $_SERVER['REMOTE_ADDR'];
                           header("Location:index.php");
                        }
 
                 }
             }
     $txt.="</table>";
     echo $txt;
   }
 
?>
[PHP] pobierz, plaintext