Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> [php]Problem z wyświetlaniem komunikatu o złych danych
Flapper
post
Post #1





Grupa: Zarejestrowani
Postów: 56
Pomógł: 0
Dołączył: 10.04.2007
Skąd: znienacka.

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

Ten kod powinien w wypadku wpisania złego hasła wyświetlić nad formularzem "Podano złe dane!!!" Ale nie wyświetla absolutnie nic. Przy podaniu dobrego loginu i hasła jedynie loguje. Co jest nie tak? sad.gif
[PHP] pobierz, plaintext 
  1. <?php ob_start(); ?>
  2. <?php
  3. session_start();
  4. session_register("zalogowany");
  5.  
  6. if(empty($_SESSION["zalogowany"]))$_SESSION["zalogowany"]=0;
  7.  
  8. require "connection.php";
  9. connection();
  10.  
  11. function ShowLogin($komunikat=""){
  12. 	if($_SESSION["zalogowany"] == 0)
  13. 	{
  14. 	echo "$komunikat<br>";
  15. 	echo "<form action='index.php' method=post>";
  16. 	echo "Login: <input type=text name=login><br>";
  17. 	echo "Hasło: <input type=text name=haslo><br>";
  18. 	echo "<input type=submit value='Zaloguj!'>";
  19. 	echo "</form>";
  20. 	echo "Jeśli nie jesteś zarejestrowany, <a href='index.php?site=register'>zarejestruj się tutaj.</a>";
  21. 	}
  22. 	else
  23. 	{ 
  24. 		$id_gracza = $_SESSION["zalogowany"];
  25. 		$imie = mysql_query("SELECT name from `char` where ID LIKE '$id_gracza'");
  26. 		echo "Zalogowany jako $id_gracza";
  27. 		echo "<a href='index.php?wyloguj=tak'>wyloguj się</a>";
  28. 	}
  29. }
  30.  
  31. ?>
  32. <html>
  33. <head>
  34. <title>Olimpus MMORPG</title>
  35. <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-2">
  36. <link rel="Stylesheet" type="text/css" href="style.css" />
  37. </head>
  38. <body>
  39. <center>
  40. <div id="top">
  41.   <div id="NAGLOWEK">
  42. 	<img src='left.jpg'><img src='banner.jpg'><img src='right.jpg'>
  43.   </div>
  44.   <div id="top2">
  45.   <div id="LOGIN">
  46. 	<?
  47. if($_GET["wyloguj"]=="tak")
  48. {
  49. 	$_SESSION["zalogowany"]=0;echo "Zostałeś wylogowany z serwisu";
  50. }
  51. if(!empty($_POST["login"]) && !empty($_POST["haslo"]))
  52. {
  53. 	$loginn = $_POST['login'];
  54. 	$hasloo = $_POST['haslo'];
  55. 	$hasloo2 = md5($hasloo);
  56. 		if(mysql_query("SELECT * from `char` where name = '$loginn' AND pass = '$hasloo2'") == TRUE)
  57. 		{
  58. 			echo "Zalogowano poprawnie. <a href='index.php'>PrzejdĽ na stronę główną</a>";
  59.  
  60. $wynik = mysql_query("SELECT * FROM `char` WHERE name = '$loginn'");
  61.  
  62. while($wynik and $pola = mysql_fetch_array($wynik))
  63. {
  64.   $_SESSION["zalogowany"]=$pola["ID"];
  65. }		
  66. 		}
  67. 		else ShowLogin("Podano złe dane!!!");
  68. }
  69. 	else ShowLogin();
  70. ?>
  71.   </div>
  72.   <div id="MENU">
  73. 	<a href='index.php'>-Strona główna</a>
  74.   </div>
  75.   <div id="TRESC">
  76. 	<pre><?php
  77. if($_GET[site] != '')
  78. {
  79. 	include(''.$_GET[site].'.htm');
  80. }
  81. else
  82. {
  83. header("location: index.php?site=main");
  84. }
  85. ?>
  86.   </div>
  87.   </div>
  88.   <div id="STOPKA">
  89.   <?
  90. 	include('Stopka.txt');
  91.   ?>
  92.   </div>
  93. </div>
  94. </center>
  95. </body>
  96. </html>
[PHP] pobierz, plaintext
Go to the top of the page
+Quote Post
ocochodzi
post
Post #2





Grupa: Zarejestrowani
Postów: 18
Pomógł: 2
Dołączył: 27.02.2007

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

Stosuj wcięcia. Newralgiczny kawalek kodu masz nieczytelnie sformatowany.

Sprawdzanie poprawności sprawdziłeś do sprawdzenia czy zmienne w $_POST są ustawione czy nie. Zatem wpisanie nieistniejacego loginu i hasła nie pociągnie wykonania linii 67. Co gorsza, mozna jako dane wpisać typowe złosliwości typu SQL injection:

loginn: a' OR name <> ' aaevil.gif
hasloo2: a' OR pass <> ' aaevil.gif

wtedy PHP mając

"SELECT * from `char` where name = '$loginn' AND pass = '$hasloo2'"

złoży zapytanie:

SELECT * from `char` where name = 'a' OR name <> '' AND pass = 'a' OR pass <> ''

AND ma wyższy priorytet niz OR, wiec wyrazenie logiczne obliczy się jakby były nawiasy:
a OR b AND c OR d jest tożsame z a OR (b AND c) OR d
Zdanie d w naszym zapytaniu jest zawsze prawdziwe, wiec całe wyrazenie logiczne bedzie prawdzwe dla kazdego rekordu. Sprowadza się zatem do żadania pokazania uzytkownikow z loginem a lub niepustym haslem. Zasadniczo wszytkich, choć w zamysłe pewnie miało zwrócić tego jednego jedynego.

A co jeśli nie ma ataku SQL injection tylko nieistniejace login i hasło (czyli sytuacja o jaką pytałeś)? System zaloguje a funkcja ShowLogin w wierszu 25 nie znajdzie odpowiedniego rekordu. Poza tym mała uwaga wydajnościowa. Jesli już masz wiersz z danymi, to go nie puszczaj tak łatwo. Ten drugi SELECT jest niepotrzebny.

Ten post edytował ocochodzi 25.03.2008, 23:37:44
Go to the top of the page
+Quote Post
« Następny starszy · Przedszkole · Następny nowszy »
 

Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 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: 19.08.2025 - 07:31
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn