System sesji - czy mój kod jest optymalny? Opcje

[klapaucius]

Witam. Oto mój kod związany z systemem sesji ma na mojej stronie. Chodzi tu o przechowywanie zmiennej z identyfikatorem użytkownika oraz czasem, po którym sesja wygasa:

[PHP] pobierz, plaintext 
session_start();
 
if (!isset($_SESSION['inicjuj'])){ // 1 zabezpieczenia sesji
    session_regenerate_id();
    $_SESSION['inicjuj'] = true;
    $_SESSION['ip'] = $_SERVER['REMOTE_ADDR'];
}
if($_SESSION['ip'] != $_SERVER['REMOTE_ADDR']){ // 2 zabezpieczenie sesji
    header("Location: ../index.php?page=glowna");     
}
 
$intTimeoutSeconds = 600; // ustawienie czasu sesji
 
if (!isset($_SESSION['zalogowany'])) { // jeśli zmienna nie jest zarejestrowana
    $_SESSION['zalogowany'] = 0;       // przypisz jej początkową wartość
}elseif(($_SESSION['intLastRefreshTime']+$intTimeoutSeconds)<time()){
        session_destroy();
        if($_SERVER['PHP_SELF'] != '/index.php'){
          header("Location: ../index.php?page=glowna&sesja_wygasla=tak");
        }else{
          header("Location: ../index.php?page=glowna");
        }
}
 
$_SESSION['intLastRefreshTime'] = time();
 
if($_SESSION['zalogowany'] == 0 && $_SERVER['PHP_SELF'] == "/game/index.php"){ // jesli uzytkownik nie jest zalogowany
  header("Location: ../index.php?page=glowna");
}elseif($_SESSION['zalogowany'] != 0 && $_SERVER['PHP_SELF'] == "/index.php"){ // podobnie jak wyzej
  header("Location: game/index.php?page=main");
}
[PHP] pobierz, plaintext 

Czy kod jest w miarę optymalny? Czy coś w nim trzeba zmienic? Jak narazie działa ale wydaje mi się ze jest trochę za długi i że ma jakieś błędy... Czy mam rację? Czy można go jakoś bardziej zoptymalizować czy jest ok?

[darko]

Kod, który podałeś jest podatny na atak Sql Injection. //edit: m.in. na tego typu ataki, poczytaj o sql injection przynajmniej na wikipedii (nie mam czasu szukać za ciebie, jes bardzo dużo bardzo dobrych artykułów na ten temat, wystarczy poszukać)., zapytaniach preintepretowanych i ogólnie korzystaj z PDO i nigdy, przenigdy nie ufaj danym pochodzącym bezpośrednio od użytkownika (wstawiasz dane z post bezpośrednio do zapytania sql, które wykonujesz).

Ten post edytował darko 7.10.2011, 23:00:01