problem z autentycznoscia sesji Opcje

[piotrekk]

Witam,
mam następujący problem.
Tworze witrynę do której przypisałem sesje. Funkcja zapisująca login do sesji wygląda następująco:

[PHP] pobierz, plaintext 
<?php
function registerLogin($login)
{
   // Rejestracja loginUsername w celu pokazania, ze uzytkownik jest zalogowany
   $_SESSION["login"] = $login;
 
   // Rejestracja adresu IP dla uruchomionej sesji
	$_SESSION["loginIP"] = $_SERVER["REMOTE_ADDR"];
}
?>
[PHP] pobierz, plaintext 

Wykorzystuje ją przy logowaniu. Następnie po zalogowaniu przenoszony jestem do głównego miejsca, gdzie moge dokonywać zakupów. Klikając na koszyk wykonowany jest skryp szybkiego zamowienia, gdzie sesja sprawdzana jest w następujący sposób.

[PHP] pobierz, plaintext 
<?php
function autentycznoscSesji()
{
	if(isset($_SESSION["login"]))
	{	
		unregisterLogin();
		header("Location : ../panelLogowanie.php");
		exit;
	}
// sprawdza czy wywolanie pochodzi z innego adresu ip niz na poczatku
	elseif($_SESSION["loginIP"] != $_SERVER["REMOTE_ADDR"])
	{
// WYWOLANIE POCHODZI Z INNEGO KOMPUTERA NIZ KOMPUTER UZYTY WCZESNIEJ, MOZLIWE ZE 
KTOS CHCE PRZECHWYCIC SESJE
		//isset($_SESSION["loginIP"]) || 
 
		unregisterLogin();
		header("Location: ../panelLogowanie.php");
		exit;
	}
 
 
}
?>
[PHP] pobierz, plaintext 

Gdy nie jestem zalogowany wszystko działa poprawnie i przenoszony jestem do panelu logowania, jednak gdy sie zaloguje funkcja nie działa poprawnie i nie wykonywana jest dalsza cześć skryptu.
Proszę o pomoc, jeżeli potrzebne są jakieś dodatkowe informacje to oczywiście je napisze.

dzięki !

[Cezar708]

na poczatek to zmieniłbym:

[PHP] pobierz, plaintext 
<?php
function unregisterLogin()
{
  unset($_SESSION["login"]);
  unset($_SESSION["loginIP"]);
}
?>
[PHP] pobierz, plaintext 

w końcu jak kolesia wylogowywujesz to nie muszisz sprawdzać czy istnieje ( w najgorszym razie nie istnieje i tak usuwasz )

po drugie zamień:

[PHP] pobierz, plaintext 
<?php
function autentycznoscSesji()
{
  if (   !isset($_SESSION["login"]) 
		 || !isset($_SESSION["loginIP"]) 
		 || ($_SESSION["loginIP"] != $_SERVER["REMOTE_ADDR"])
  ) {
	return false;
  }
  return true;
}
?>
[PHP] pobierz, plaintext 

i dopiero obsługę zrób w kodzie (ewentualnie dopisz jeszcze jakąś funkcję):

[PHP] pobierz, plaintext 
<?php
//1. sprawdzanie czy uzytkownik jest zareejstrowany ( spr. sesji)
require_once "include/function.php";
require_once "include/mysql.inc";
 
session_start();
 
if ( !autentycznoscSesji() ){
  unregisterLogin();
  header("Location : ../panelLogowanie.php");
  die;
}
 
// dalsza część
?>
[PHP] pobierz, plaintext 

być może błąd wynika z tego że w niektórych przypadkach Twoja funkcja nie podejmuje żadnej akcji a ma to miejsce w momencie gdy usr jest niezalogowany. Sprawdź jak to działa.