Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [HTML][MySQL][PHP]Sprawdzanie hasła md5.
Gruchol
post
Post #1





Grupa: Zarejestrowani
Postów: 242
Pomógł: 0
Dołączył: 28.11.2014

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

Witam,

Napisałem takie coś :
[PHP] pobierz, plaintext 
  1. <!DOCTYPE HTML>
  2. <html lang="pl">
  3. <head>
  4. <title>Sklepik</title>
  5. </head>
  6. <body>
  7.  
  8.  
  9.  
  10. <?php
  11. $db = mysqli_connect('localhost', 'root', '', 'sf');
  12.  
  13. $id = $_GET['player'];
  14. $nick = mysqli_query($db, "SELECT user_name FROM user_data WHERE user_id='$id'");
  15. $user_name = $nick->fetch_assoc()['user_name'];
  16. $grzybki = mysqli_query($db, "SELECT mushroom FROM user_data WHERE user_id='$id'");
  17. $grzybkiwyswietl = $grzybki->fetch_assoc()['mushroom']; 
  18. $zloto = mysqli_query($db, "SELECT silver FROM user_data WHERE user_id='$id'");
  19. $zlotowyswietl = $zloto->fetch_assoc()['silver']; 
  20.  
  21. echo "<center><h1>Witaj w sklepiku $user_name!</h1></center>";
  22. echo "<center><h3>Aktalnie posiadasz $grzybkiwyswietl grzybkow oraz $zlotowyswietl zlota</h3></center>";
  23.  
  24. ?>
  25. <center>
  26. <form method="post" action="">
  27. <b>Zresetowanie lochów(bez kluczy)</b>
  28. <br/>
  29. <b>Aby potwierdzić wpisz hasło : </b>
  30. <input type="password" name="haslo"/>
  31. <br/>
  32. <br/>
  33. <input type="submit" value="30 grzybków"/></center>
  34. </form>
  35. <?php
  36. $db = mysqli_connect('localhost', 'root', '', 'sf');
  37. $haslozbazy = mysqli_query($db, "SELECT password FROM user_data WHERE user_name='$user_name'");
  38. @$haslozsklepu = $_POST["haslo"];
  39. if($haslozbazy["haslo"] == md5($haslozsklepu)) {
  40. 	mysqli_query($db, "UPDATE user_data SET dungeon_1 = '0', dungeon_2 = '0', dungeon_3 = '0', dungeon_4 = '0', dungeon_5 = '0', dungeon_6 = '0', dungeon_7 = '0', dungeon_8 = '0' WHERE user_name = '$user_name'");
  41. }
  42. else {
  43. 	echo "Haslo jest bledne";
  44. }
  45. if($haslozesklepu == '') {
  46.  
  47. 	echo "Pole jest puste";
  48. }
  49. ?>
  50. </body>
  51. </html>
[PHP] pobierz, plaintext

I mam problem z ifem który sprawdza hasło.
Gdy wywale tego ifa wszystko działa ok.
Hasło w bazie jest zahaszowane w md5.
Próbuje się przestawić z MySQL na MySQLi i nie wiem czy wszędzie zastosowałem właśnie MySQLi.
Proszę o sprawdzenie i pozdrawiam.
Go to the top of the page
+Quote Post
 
 Start new topic
Odpowiedzi
Rysh
post
Post #2





Grupa: Zarejestrowani
Postów: 821
Pomógł: 111
Dołączył: 11.09.2006
Skąd: Biała Podlaska

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

albo if empty albo if == zdecyduj się.

[PHP] pobierz, plaintext 
  1. $nick = mysqli_query($db, "SELECT user_name FROM user_data WHERE user_id='$id'");
  2. $user_name = $nick->fetch_assoc()['user_name'];
  3. $grzybki = mysqli_query($db, "SELECT mushroom FROM user_data WHERE user_id='$id'");
  4. $grzybkiwyswietl = $grzybki->fetch_assoc()['mushroom']; 
  5. $zloto = mysqli_query($db, "SELECT silver FROM user_data WHERE user_id='$id'");
  6. $zlotowyswietl = $zloto->fetch_assoc()['silver'];
[PHP] pobierz, plaintext

Po cholerę robisz tyle zapytań do bazy danych? Nie możesz dać:
[PHP] pobierz, plaintext 
  1. $zloto = mysqli_query($db, "SELECT user_name, mushroom, silver FROM user_data WHERE user_id='$id'");
[PHP] pobierz, plaintext

BTW, Twój kod jest strasznie napisany - pobierasz ID użytkownika z GET, czyli z linku - modyfikując link mogę zmienić każdemu co tylko chcę. nie wspominając już o sql injection.

Ten post edytował Rysh 31.12.2014, 19:19:45
Go to the top of the page
+Quote Post

Posty w temacie
- Gruchol   [HTML][MySQL][PHP]Sprawdzanie hasła md5.   31.12.2014, 14:58:31
- - Damonsson   A nie przyszło Ci do głowy, np. wyświetlić obydwie...   31.12.2014, 15:04:52
- - Gruchol   Przed ifem dopisałem echo "$haslozbazy |...   31.12.2014, 15:13:09
- - Damonsson   No widzisz, to już doszedłeś do wniosku, że źle po...   31.12.2014, 15:31:02
- - Gruchol   Kompletnie zapomniałem o tym. Na początku gdy pobi...   31.12.2014, 15:41:40
- - Damonsson   Przed wysłaniem robisz [PHP] pobierz, plaintext $h...   31.12.2014, 17:10:45
- - Gruchol   @Edit Wszystko już działa oprócz 1. [PHP] pobierz...   31.12.2014, 17:26:40
- - com   no bo nie masz warunku który to sprawdza, tłumaczy...   31.12.2014, 18:54:01
- - Gruchol   Zamieniłem tego ifa na if(!empty($haslozb...   31.12.2014, 19:09:19
- - Rysh   albo if empty albo if == zdecyduj się. [PHP] pobi...   31.12.2014, 19:14:50
- - Gruchol   Z tego get zrezygnuje i zrobię to inaczej postem. ...   31.12.2014, 19:34:24
- - untorched   [PHP] pobierz, plaintext if(isset($_POST...   31.12.2014, 19:55:44
- - Gruchol   Nie za bardzo wiem jak to zrobić. Próbowałem tak ...   31.12.2014, 20:12:10
- - com   tak jak masz jest ok jednakże zmień [HTML] pobierz...   1.01.2015, 18:26:40
- - Gruchol   Zrobiłem to tak : [PHP] pobierz, plaintext if...   1.01.2015, 21:50:38
- - com   no bo w każdym formularzu masz taki same imputy, w...   1.01.2015, 22:58:47

« 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: Przełącz na: Standardowy · Przełącz na: Linearny+ · Drzewo

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

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