Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> [php+mysql] sonda do sprawdzenia
rooki
post 14.09.2007, 09:33:42
Post #1





Grupa: Zarejestrowani
Postów: 10
Pomógł: 0
Dołączył: 6.07.2007

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

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
Go to the top of the page
+Quote Post
qqrq
post 14.09.2007, 09:48:14
Post #2





Grupa: Zarejestrowani
Postów: 418
Pomógł: 8
Dołączył: 16.11.2006

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

Powinieneś ten post dać do "Oceny".


--------------------
Poezja śpiewana ;) | Niunia | Nic ciekawego...
Go to the top of the page
+Quote Post
nospor
post 14.09.2007, 10:01:03
Post #3





Grupa: Moderatorzy
Postów: 36 557
Pomógł: 6315
Dołączył: 27.12.2004
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


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer
Go to the top of the page
+Quote Post
rooki
post 14.09.2007, 11:00:48
Post #4





Grupa: Zarejestrowani
Postów: 10
Pomógł: 0
Dołączył: 6.07.2007

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

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.
Go to the top of the page
+Quote Post
nospor
post 14.09.2007, 11:10:54
Post #5





Grupa: Moderatorzy
Postów: 36 557
Pomógł: 6315
Dołączył: 27.12.2004
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 /*"


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer
Go to the top of the page
+Quote Post
rooki
post 14.09.2007, 11:58:12
Post #6





Grupa: Zarejestrowani
Postów: 10
Pomógł: 0
Dołączył: 6.07.2007

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

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.
Go to the top of the page
+Quote Post
nospor
post 14.09.2007, 12:01:34
Post #7





Grupa: Moderatorzy
Postów: 36 557
Pomógł: 6315
Dołączył: 27.12.2004
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


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer
Go to the top of the page
+Quote Post
rooki
post 14.09.2007, 12:21:28
Post #8





Grupa: Zarejestrowani
Postów: 10
Pomógł: 0
Dołączył: 6.07.2007

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

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.
Go to the top of the page
+Quote Post
nospor
post 14.09.2007, 12:25:17
Post #9





Grupa: Moderatorzy
Postów: 36 557
Pomógł: 6315
Dołączył: 27.12.2004
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


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer
Go to the top of the page
+Quote Post
rooki
post 14.09.2007, 12:56:08
Post #10





Grupa: Zarejestrowani
Postów: 10
Pomógł: 0
Dołączył: 6.07.2007

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

ok zrobione wstydnis.gif smile.gif Dzięki ! smile.gif
Go to the top of the page
+Quote Post
« Następny starszy · Oceny · 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 Wersja Lo-Fi Aktualny czas: 13.06.2025 - 06:31
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn