Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> [MySQL][PHP]Problem ze skryptem logowania
krzesik
post
Post #1





Grupa: Zarejestrowani
Postów: 516
Pomógł: 1
Dołączył: 25.08.2012

Ostrzeżenie: (0%)
-----

Witam, mam dziwny problem ze skryptem do logowania. Polega on naty, że jeśli user wpisze złe dane do otrzymuje komunikat o tym fakcie, jeśli jednak znow źle wpisze to zostaje przeniesiony na stronę której nie powinien widzieć.
Generalnie wygląda to tak, jakby skrypt sprawdzał pierwszym razem w MySQL login i hasło, a drugim całkowicie to olewał..... Prosze o poradę.

kod z logowanie.php
[PHP] pobierz, plaintext 
  1. ...
  2. <?php
  3. session_start();
  4. session_register("login");
  5.  
  6. if(empty($_SESSION["zalogowany"]))$_SESSION["zalogowany"]=0;
  7.  
  8. if (!isset($_SESSION['login'])) 
  9. {
  10. if ($_POST['wyslane'] and !empty($_POST['login']) and !empty($_POST['haslo']) ) 
  11. {
  12.         include 'config.php'; 
  13.         $login = htmlspecialchars(trim($_POST["login"]), ENT_QUOTES);
  14.         $haslo = htmlspecialchars(trim($_POST["haslo"]), ENT_QUOTES);
  15.  
  16. {														
  17. $_SESSION["zalogowany"]=1;
  18. $_SESSION['status'] = $_POST['status'] +0;
  19. }													
  20.  
  21. $wynik=mysql_query("SELECT * FROM uzytkownik WHERE (login='$login' and haslo='$haslo' and status<>2) LIMIT 1");
  22.  
  23. if (mysql_num_rows($wynik) == 1)
  24. {
  25.             $informacja = mysql_fetch_array($wynik);
  26.             $_SESSION["login"] = $informacja["login"];
  27.             $_SESSION["uprawnienia"] = $informacja["uprawnienia"] +0;
  28.             $_SESSION["status"] = $informacja["status"] +0;
  29.             header('Location: logowanie.php');
  30. }
  31. else {
  32.             echo '<center><b><p>
  33.             <img src=img/ikony/user_delete.png style=width:16px;height:16px;border:0px; /><br>
  34. 				Nieprawidłowy Login / Hasło </p></b>';
  35. }
  36.         mysql_close($baza);
  37.  
  38. }
  39. echo <<< KONIEC
  40.  
  41. <html>
  42. <body>
  43. <center>
  44. <body onload="document.logowanie.login.focus()">
  45. <form name="logowanie" form action="logowanie.php" method="post">
  46. <input type="hidden" name="wyslane" value="TRUE" />
  47. <input type="hidden" name="uprawnienia" value="$uprawnienia" />
  48. <p><img src="img/ikony/user.png" title="Podaj swój LOGIN" style="width:16px;height:16px;border:0px;" />&nbsp;&nbsp;Login: <input type="text" name="login" value="" title="Podaj Nazwisko i Imię"/></p>
  49. <p><img src="img/ikony/key.png" title="Podaj swoje HASŁO" style="width:16px;height:16px;border:0px;" />&nbsp;&nbsp;Hasło: <input type="password" name="haslo" title="Podaj swoje HASŁO"/></p>
  50. <p><input type="submit" value="Zaloguj" /></p>
  51. </form>
  52. </center>
  53. </body>
  54. </html>
  55.  
  56. KONIEC;
  57.  
  58. }
  59. else
  60. {
  61. 	    header('Location: tabela.php');
  62. }
  63. if ($_GET["wyloguj"] == "tak") {
  64.     //niszczenie sesji użytkownika
  65.     session_unset();
  66.     session_destroy();
  67.     header('Location: index.php'); // przekierwanie na stronę główną
  68. }
  69.  
  70. ?>
  71. ...
[PHP] pobierz, plaintext


a to mam w skrypcie po autoryzacji tabela.php:

[PHP] pobierz, plaintext 
  1. ...
  2. {
  3. session_start();
  4. header('Location: logowanie.php');
  5. }
  6. ...
[PHP] pobierz, plaintext


Ten post edytował krzesik 6.12.2012, 22:38:41
Go to the top of the page
+Quote Post
Wazniak96
post
Post #2





Grupa: Zarejestrowani
Postów: 550
Pomógł: 75
Dołączył: 5.06.2012
Skąd: Lębork

Ostrzeżenie: (0%)
-----

Myślę, że błąd leży w drugim if, według mnie powinno to wyglądać tak:
[PHP] pobierz, plaintext 
  1. if(empty($_SESSION["zalogowany"]))$_SESSION["zalogowany"]=0;
  2.  
  3. if (!isset($_SESSION['login']) || empty($_SESSION['login'])) 
  4. {
[PHP] pobierz, plaintext


Rejestrujesz sesję, a if sprawdzał czy sesja istnieje. A skoro została zarejestrowana to istnieje.
Możesz zrobić to poprawiając if lub też usuwając register. Będzie tak nawet lepiej ponieważ nowa wersja PHP nie obsługuje już tej funkcji i sypią się błędy. (IMG:style_emoticons/default/smile.gif)

Tak PS: Po co sesja "zalogowany" skoro z góry ustawiasz ją w dwóch miejscach na 0, i po co te tokeny z 17 i 19 lini?
Go to the top of the page
+Quote Post
krzesik
post
Post #3





Grupa: Zarejestrowani
Postów: 516
Pomógł: 1
Dołączył: 25.08.2012

Ostrzeżenie: (0%)
-----

niestety nie pomogło, nadal objaw jest taki sam, skrypt po drugim błędym logowaniu przechodzi na stronę po haśle...
Go to the top of the page
+Quote Post
nospor
post
Post #4





Grupa: Moderatorzy
Postów: 36 557
Pomógł: 6315
Dołączył: 27.12.2004
$_SESSION["zalogowany"]=1;
Ustawiasz to niezależnie od tego czy koleś wpisał dane poprawne czy nie. Nie jest to przypadkiem błąd?

No i to:
session_register("login");
wywal to jeśli nie zrobiłeś tego do tej pory.
Go to the top of the page
+Quote Post
krzesik
post
Post #5





Grupa: Zarejestrowani
Postów: 516
Pomógł: 1
Dołączył: 25.08.2012

Ostrzeżenie: (0%)
-----

Bardzo dziękuję o to właśnie chodziło
Go to the top of the page
+Quote Post
« Następny starszy · Przedszkole · Następny nowszy »
 

Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Standardowy · Przełącz na: Linearny+ · Przełącz na: Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Aktualny czas: 23.08.2025 - 22:48
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn