Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> Wszystkie hasła md5 tak samo się kodują
Pudi
post 7.11.2010, 11:46:50
Post #1





Grupa: Zarejestrowani
Postów: 76
Pomógł: 0
Dołączył: 3.03.2009
Skąd: WLKP

Ostrzeżenie: (10%)
X----

Witam posiadam system rejestracji użytkowników ale niestety coś się zepsuło założyłem 3 konta 1
Pudi hasło sdasdad(przykładowo)
Test1 hasło testy1
Test2 hasło test2
I Tylko hasło w koncie Pudi działa można się logować itp, a w innych kontach jest napisane hasło nieprawidłowe po obejrzeniu phpmyadmin zauważyłem ze wszystkie konta mają tak samo zakodowane hasło jak konto PUDI, Sprawdziłem Konto Test1 i Test2 logowało się po podaniu hasła do konta Pudi i tutaj moje pytanie co robię źle kod pliku register.php
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. $login = $_POST['login'];
  4. $haslo = $_POST['haslo'];
  5. $haslo2 = $_POST['haslo2'];
  6. $email = $_POST['email'];
  7. $losowe = $_POST['l'];
  8. $data=date('d.m.Y, H:i');
  9. $poziom = 0;
  10.  
  11. if ($_POST){
  12.  
  13. if(!empty($login) AND !empty($haslo) AND !empty($haslo2) AND !empty($email)){
  14.  
  15. if($haslo != $haslo2){
  16.  
  17. $info .=  'podane hasłą są różne<br />';
  18.  
  19. }else{
  20.  
  21. if(!ereg("^.+@.+\..+$", $email)){
  22.  
  23. $info .=  'niepoprawny adres e-mail<br />';
  24.  
  25. }else{
  26.  
  27. require_once('config.php');
  28.  
  29. $login = czysc($login);
  30. $haslo = czysc($haslo);
  31. $haslo2 = czysc($haslo);
  32. $email = czysc($email);
  33.  
  34.  
  35. $query = "SELECT login FROM users WHERE login='$login'";
  36. $result = mysql_query($query);
  37.  
  38. if(@mysql_num_rows($result) > 0){
  39.  
  40. $info .=  "wybrana nazwa użytkownika($login) jest już zajęta<br />";
  41.  
  42. }else{
  43.  
  44. $query = "INSERT INTO users (login,haslo,email,data,poziom) VALUES ('$login',md5('$haslo1'),'$email','$data','$poziom')";
  45. mysql_query($query);
  46.  
  47. $info .=  'rejestracja zakończyła się sukcesem<br />';
  48.  
  49. }
  50.  
  51. }
  52.  
  53. }
  54.  
  55. }else{
  56.  
  57. $info .= 'uzupełnij wszystkie pola<br />';
  58.  
  59. }
  60.  
  61. }
  62.  
  63. ?>
  64. <html>
  65. <head>
  66. <meta name="generator" content="WebSite PRO 4.3" />
  67. <meta http-equiv="content-type" content="text/html; charset=iso-8859-2" />
  68. <title>Rejestracja</title>
  69. </head>
  70. <body>
  71.  
  72. <form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post">
  73.  
  74. <table>
  75.  
  76. <tr>
  77. <td>login:</td>
  78. <td><input type='text' name='login'></td>
  79. </tr>
  80.  
  81. <tr>
  82. <td>hasło:</td>
  83. <td><input type='password' name='haslo'></td>
  84. </tr>
  85.  
  86. <tr>
  87. <td>powtórz hasło:</td>
  88. <td><input type='password' name='haslo2'></td>
  89. </tr>
  90.  
  91. <tr>
  92. <td>e-mail:</td>
  93. <td><input type='text' name='email'><input type='hidden' name='control' value='1'></td>
  94. </tr>
  95.  
  96. <tr>
  97. <td><input type='submit' value='zarejestruj'></td>
  98. </tr>
  99.  
  100. <tr>
  101. <td></td>
  102. <td colspan='2'><?php if(!empty($info)){ echo $info; } ?></td>
  103. </tr>
  104.  
  105. </table>
  106.  
  107. </form>
  108.  
  109. </body>
  110. </html>
[PHP] pobierz, plaintext

A tak wyglada baza danych:



Proszę o pomoc bo kompletnie nie wiem gdzie jest błąd
Go to the top of the page
+Quote Post
 
 Start new topic
Odpowiedzi
greycoffey
post 7.11.2010, 11:59:37
Post #2





Grupa: Zarejestrowani
Postów: 320
Pomógł: 29
Dołączył: 3.04.2010

Ostrzeżenie: (20%)
X----

Zmienna $haslo1 nie istnieje, za to istnieją $haslo i $haslo2. Jesli funkcja czysc, to funkcja wyciagnieta z jednego tutka na webhelpie, to lepiej ją zmień, jest niebezpieczna, co tam opisałem.
Go to the top of the page
+Quote Post

Posty w temacie
- Pudi   Wszystkie hasła md5 tak samo się kodują   7.11.2010, 11:46:50
- - lwc   Mniemam, że skrypt traktuje tą zmienną haslo1 jako...   7.11.2010, 11:48:06
|- - Pudi   Cytat(lwc @ 7.11.2010, 11:48:06 ) Mni...   7.11.2010, 12:02:37
- - greycoffey   Zmienna $haslo1 nie istnieje, za to istnieją ...   7.11.2010, 11:59:37
- - greycoffey   Cytat(lwc @ 7.11.2010, 11:48:06 ) Mni...   7.11.2010, 12:03:13
- - markonix   [PHP] pobierz, plaintext '".md5($haslo1...   7.11.2010, 12:03:44
|- - Pudi   Cytat(markonix @ 7.11.2010, 12:03:44 ...   7.11.2010, 12:06:31
- - tehaha   ponadto, żeby funkcja md5 działała to trzeba zrobi...   7.11.2010, 12:06:12
- - greycoffey   Cytat(markonix @ 7.11.2010, 12:03:44 ...   7.11.2010, 12:07:38
- - tehaha   a skąd w ogóle bierzesz zmienną $hasło1, skor...   7.11.2010, 12:07:56
- - wdev   Zmienna $haslo1 nie pojawia się wcześniej (ni...   7.11.2010, 12:08:44
- - greycoffey   Cytat(tehaha @ 7.11.2010, 12:06:12 ) ...   7.11.2010, 12:09:16
- - markonix   Zakomentuj na razie: Kod// mysql_query($q...   7.11.2010, 12:09:42
- - greycoffey   Cytat(markonix @ 7.11.2010, 12:09:42 ...   7.11.2010, 12:11:53
- - Pudi   sorrka zły kod wkleiłem co zmiena jest haslo1 tak ...   7.11.2010, 12:13:13
- - greycoffey   Dodaj na początku: [PHP] pobierz, plaintext <?p...   7.11.2010, 12:19:19
|- - Pudi   Cytat(greycoffey @ 7.11.2010, 12:19:1...   7.11.2010, 12:21:45
- - greycoffey   Krytycznych błędów tu nie ma, po prostu sprawdzaj ...   7.11.2010, 12:28:15
|- - Pudi   Cytat(greycoffey @ 7.11.2010, 12:28:1...   7.11.2010, 13:00:38
- - ciekawskiii   dziwny problem, w kodzie wszystko wyglada ok, wywa...   7.11.2010, 13:17:53
- - greycoffey   Cytat(Pudi @ 7.11.2010, 13:00:38 ) To...   7.11.2010, 13:39:18

« Następny starszy · PHP · 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: 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 Wersja Lo-Fi Aktualny czas: 14.08.2025 - 17:26
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn