Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [php+mysql] sonda do sprawdzenia
Forum PHP.pl > Inne > Oceny
rooki
14.09.2007, 09:33:42
Chciałbym prosić o sprawdzenie sondy, którą napisałem pod względem poprawności gdyż jeśli idzie o PHP to jestem początkujący. Moim zamiarem było, żeby przy zmianie opcji / dodaniu lub usunięciu opcji tylko minimalnie zmodyfikować kod - czyli u mnie przy zmianie opcji nalezy zmodyfikować tabelę w bazie danych, kod html odpowiedzialny za wyświetlenie formularza oraz kod PHP ten: "$tekst_odp[0]="tak"; $tekst_odp[1]="nie"; " - to wszystko.

[PHP] pobierz, plaintext 
  1. <?php
  2. ob_start(); 
  3. //MAMY TABELĘ "GLOSOWANIE1" Z POLAMI: ID(KLUCZ PODSTAWOWY, AUTOINKREMENTACJA); TAK; NIE.
  4.  
  5. if(isset($_POST['button'])) //jeśli naciśnięto na "button"
  6. {
  7.  
  8. setcookie("sonda", "2", time()+20); //posyłamy ciastko ważne 20 sekund
  9.  
  10. $opcja=$_POST["opcja"]; 
  11.  
  12.  $uchwyt_polaczenia = mysql_connect("localhost", "xxxx", "xxxxxxx") or die ("Nie można połączyć się z MySQL");
  13.  mysql_select_db ("xxxxxx") or die ("Nie można połączyć się z bazą.");
  14.  $wynik=mysql_query("SELECT * FROM glosowanie1 WHERE id=1;");
  15.  
  16.  
  17. 	  $rekord = mysql_fetch_assoc ($wynik);
  18. 	  $liczba=$rekord["$opcja"];
  19.  
  20. 	  $liczba++;
  21. 	 
  22.  
  23. mysql_query("UPDATE glosowanie1 SET $opcja=$liczba WHERE id=1;");
  24. mysql_close($uchwyt_polaczenia);
  25.  
  26.  
  27.  
  28. header("Location: sonda.php"); 
  29. exit; 
  30. }
  31.  
  32.  
  33.  
  34. elseif(isset($_POST['button2'])||$_COOKIE['sonda']=="2")
  35. {
  36. $tekst_odp[0]="tak";
  37. $tekst_odp[1]="nie";
  38.  
  39.  
  40.  
  41. $rozmiar=sizeof($tekst_odp); 
  42.  
  43.  
  44.  $uchwyt_polaczenia = mysql_connect("localhost", "xxxx", "xxxxx") or die ("Nie można połączyć się z MySQL");
  45.  mysql_select_db ("xxxxxxx") or die ("Nie można połączyć się z bazą.");
  46.  $wynik=mysql_query("SELECT * FROM glosowanie1 WHERE id=1;");
  47.  $rekord = mysql_fetch_assoc ($wynik);
  48.  
  49.  $suma_glosow=0;
  50.  
  51.  for ($odp=0;$odp<$rozmiar;$odp++)
  52.  {
  53.  $suma_glosow=$suma_glosow+$rekord["$tekst_odp[$odp]"];
  54.  }
  55.  
  56.  if($suma_glosow==0) //aby uniknąć dzielenia przez zero
  57.  {
  58. 	for ($odp=0;$odp<$rozmiar;$odp++)
  59. 	{
  60. 	$procent[$odp]=0; 
  61. 	}
  62.  }
  63.  
  64. else
  65. {
  66. 	for ($odp=0;$odp<$rozmiar;$odp++)
  67. 	{
  68. 	$procent[$odp]=round(($rekord["$tekst_odp[$odp]"]/$suma_glosow)*100); //"round"-zaokrąglenie
  69. 	}
  70. }
  71.  
  72.  
  73.  
  74.  
  75.  mysql_close($uchwyt_polaczenia);
  76.  
  77.  for ($odp=0;$odp<$rozmiar;$odp++)
  78. 	{
  79. 	 print "<IMG SRC='1.gif' WIDTH=$procent[$odp] HEIGHT=10/>";
  80. 	print " $tekst_odp[$odp] $procent[$odp] %<br/>";
  81. 	 }
  82. }
  83.  
  84.  
  85. ob_end_flush(); 
  86. ?>
  87.  
  88.  
  89.  
  90.  
  91.  
  92. <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
  93. <html xmlns="http://www.w3.org/1999/xhtml">
  94. <head>
  95. <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
  96. <title>Untitled Document</title>
  97. </head>
  98.  
  99. <body>
  100.  
  101. <?php
  102.  
  103. if($_COOKIE['sonda']!="2") //jeśli wartość ciastka jest różna od "2" (nie głosowano) to wyświetl formularz
  104. {
  105. ?>
  106. 	<form name="form1" method="post">
  107. 	<p>Czy lubisz koty?</p>
  108. 	<p>
  109. 	<input type="radio" name="opcja" value="tak" checked/>tak<br />
  110. 	<input type="radio" name="opcja" value="nie"/>nie<br />
  111. 	<input type="submit" name="button" value="Wyślij" />
  112. 	<input type="submit" name="button2" value="Wynik" />
  113. 	</p>
  114. 	</form>
  115.  
  116. <?php
  117. }
  118. ?>
  119.  
  120.  
  121.  
  122.  
  123.  
  124. </body>
  125. </html>
[PHP] pobierz, plaintext



Powinienem coś w tym kodzie zmienić czy jest ok ? Bo cały wczorajszy dzień nad tym siedziałem, żeby przy dodaniu/usuwaniu/zmienianiu opcji tylko minimalnie wszystko modyfikować smile.gif.

Tu można przetestować: http://misiekmisiek.lua.pl/sonda.php
qqrq
14.09.2007, 09:48:14
Powinieneś ten post dać do "Oceny".
nospor
14.09.2007, 10:01:03
ten kod:
[PHP] pobierz, plaintext 
  1. <?php
  2. $wynik=mysql_query("SELECT * FROM glosowanie1 WHERE id=1;");
  3.  
  4.  
  5. 	  $rekord = mysql_fetch_assoc ($wynik);
  6. 	  $liczba=$rekord["$opcja"];
  7.  
  8. 	  $liczba++;
  9. 	 
  10.  
  11. mysql_query("UPDATE glosowanie1 SET $opcja=$liczba WHERE id=1;");
  12. ?>
[PHP] pobierz, plaintext

zastepuje sie tym:
[PHP] pobierz, plaintext 
  1. <?php
  2. mysql_query("UPDATE glosowanie1 SET $opcja=$opcja+1 WHERE id=1;");
  3. ?>
[PHP] pobierz, plaintext


Ty to testowales? Bo ta twoja blokada z ciachem cos nie dziala.

pozatym przenosze


edit:
Twoj kod wogole nie jest zabezpieczony przed atakami Sql Injection
http://forum.php.pl/index.php?showtopic=23...t=0&start=0

edit: na dowod ze taki atak moge przeprowadzic, to dla opcji tak wpisalem 20000 smile.gif
rooki
14.09.2007, 11:00:48
Nospor wielkie dzięki, ustawiłem ważność ciastka na 20 sekund dlatego pewnie można oddawać głos po głosowaniu prawie natychmiast.

Ale kurcze jak żeś to zrobił, że mam w bazie w tabeli na tak ponad 20 000 głosów ? party.gif blinksmiley.gif
Ja myślałęm, że ataki SQL Injection to można przeprowadzić wtedy gdy jest pole w które coś trzeba coś wpisać, a u mnie przecież tylko się zaznacza opcję smile.gif.
Czytam to z tego linku co podałeś, ale narazie jeszcze nie doszedłęm jak Ty to mogłeś zrobić smile.gif.
nospor
14.09.2007, 11:10:54
Cytat
Nospor wielkie dzięki, ustawiłem ważność ciastka na 20 sekund dlatego pewnie można oddawać głos po głosowaniu prawie natychmiast.
Slepy nie jestem i widze ze jest 20 sekund. Glosowalem szybciej niz 20 sekund. Ale pod inna przegladarka dzialalo zabezpieczenie. Dodam ze obydwie przegladarki obsluguja ciacha.

Cytat
Ja myślałęm, że ataki SQL Injection to można przeprowadzić wtedy gdy jest pole w które coś trzeba coś wpisać, a u mnie przecież tylko się zaznacza opcję
No nie dokonca. Ty wysylasz formularzem pole o nazwie "opcja" i wartosci tak lub nie. To, ze ja tam nie moge wstawic value z poziomu ogolnego, to nie znaczy ze ja nie moge grzebac w twoim htmlu winksmiley.jpg
Pozatym ządania do Twojej strony nie muszą isc z przeglądarki.

Cytat
Czytam to z tego linku co podałeś, ale narazie jeszcze nie doszedłęm jak Ty to mogłeś zrobić
Value zamiast "tak" dalem :
"tak=20000 /*"
rooki
14.09.2007, 11:58:12
Coś chyba jest nie tak z tym serwerem, bo u mnie na postawionym serwerze lokalnym zarówno w FF jak i IE7 nie można głosować po oddaniu głosu przez 20 sekund.
A co do tych 20000 Twoich głosów to je usunąłem i wprowadziłęm w kodzie PHP:
[PHP] pobierz, plaintext 
  1. <?php
  2. $opcja=addslashes($_POST["opcja"]);
  3. ?>
[PHP] pobierz, plaintext


Teraz już chyba 20000 razy nie zagłosujesz smile.gif.
nospor
14.09.2007, 12:01:34
Cytat
Teraz już chyba 20000 razy nie zagłosujesz
ech.... kiepsko czytales tamten topic. A ja ci nawet podalem co wpisalem....
No to sobie teraz zajrzyj do bazy... 20 000 jak marzenie tongue.gif
rooki
14.09.2007, 12:21:28
To ja już nie wiem - mysql_real_escape_string()-nie; addslashes()-nie; to jak można się przed tym atakiem zabezpieczyć jeśli mógłbyś napisać smile.gif.
nospor
14.09.2007, 12:25:17
Cytat
o jak można się przed tym atakiem zabezpieczyć jeśli mógłbyś napisać
Spojrz w moj podpis:
Myśl, myśl..... winksmiley.jpg
Skoro option ma zawierac "tak" lub "nie" to sprawdzaj czy to zawiera a jak nie to odeslij hakera z kwitkiem
rooki
14.09.2007, 12:56:08
ok zrobione wstydnis.gif smile.gif Dzięki ! smile.gif
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.