Logowanie nie działa.. (sesja ?) Opcje

[tara]

witam,

mam taki skrypt logowania:

[PHP] pobierz, plaintext 
<? 
// polaczenie z baza danych...
 
print &#092;"
<form method=&#092;"POST\" action=\"login_admin.php?a=1\">n 
   <h1> Zaloguj sie! </h1><br>n 
   User: <input type=&#092;"text\" name=\"login\"><br>n 
   Pass: <input type=&#092;"password\" name=\"haslo\"><br>n 
   <input type=&#092;"submit\" name=\"submit\" value=\"Zaloguj\"><br>n 
   <br>
  </form>&#092;";
 
 
if( isset( $_GET['a'] ) ) 
{ 
 
session_start(); 
session_name(admin_login); 
 
	   $z = md5($haslo);
	   $wynik = mysql_query (&#092;"SELECT ksywka,haslo FROM admin_login WHERE ksywka='$login' AND haslo='$z'\");
 
	  while ($rekord = mysql_fetch_array ($wynik)) {
	   
$haslob = stripslashes($rekord[1]);
$loginb = stripslashes($rekord[0]);
 
}
   if (($z == $haslob) || ($login == $loginb)) 
	   { 
 
		session_register(&#092;"login\");  
 
header(&#092;"Location: admin_zarzadzanie.php?\" . SID); 
}  else { 
		  echo(&#092;"Zly login/pass \"); 
 
			   } 
 
	}
 
 
?>
[PHP] pobierz, plaintext 

plik admin_zarzadzanie:

[PHP] pobierz, plaintext 
<? 
session_start(); 
session_name(admin_login); 
if (!isset($_SESSION[&#092;"login\"])) # sprawdza czy w sesji jest zaladowana zmienna login:) 
{ 
header(&#092;"Location: login_admin.php\"); # jezeli nie jest to wyswietla plik login.php 
exit(); 
if ($_SESSION[&#092;"ban\"] == 1) 
	{ 
	 echo(&#092;"Masz bana. <br> <pre> Error 403 <br> Brak dostępu do zasobów </pre>\"); 
	exit(); # jeżeli ma bana, to skrypt wyœwietli ten tekst i się zatrzyma, więc gosciu nie 
ostanie ani kawałka kodu więcej:) 
	} }
?>
Czesc <? print &#092;"$login\"; ?> co chcesz zrobić :
//......
[PHP] pobierz, plaintext 

i wraca mi do login_admin.php.. czyli chyba (IMG:http://forum.php.pl/style_emoticons/default/sad.gif) nie daje zmiennej login do sesji

[adax2000]

co prawda Mori zgłosił już tu jakieś uwagi ale może dodam jeszcze kilka swoich.
1. session_name(admin_login); czy admin_login to gdzieś ustawiony zapomocą define constans jeśli nie to lepiej by było: session_name('admin_login'); (z apostrofami)

2. session_start najlepiej dawać na początku skryptu czyli przed print to samo header, czyli całą funkcję daj przed print

3. oczywiście jeśli chcesz odczytywać zmieną za pomocą instrukcji $_SESSION['login'] to nie rejestrujesz jej session_register tylko poprzez przypisanie ale w twoim przypadku
chyba lepsze będzię $_SESSION['login']=$login a nie jak proponuje mori $_SESSION['login']=1, bo wtedy miałbyś czesc 1 co chcesz robić

4. jeśli natomiast rejestrujesz zmienne poprzez session_register to są one w nowym skrypcie po instrukcji session_start od razu dostępne jako globalne tu: $login

5. jeżeli w serwerz masz ustawione automatyczne zamienianie zmiennych post i innych na globalne (ze skryptu to wynika) to proponuję ze względów bezpieczeństwa wyłączyć tą opcję i odwoływać się do nich normalnie przez $HTTP_POST_VARS względnie $_POST

to tyle co mi się nasunęło pozdro i nara