Простая форма смены пароля, как исправить ошибку?

Question

[DMax921]

Здравствуйте друзья, я на просторах интернета нашел простую форму смены пароля, но ошибка, "Старый пароль не верен" var_dump говорит что пароль верен в базе он как md5 и Select выводит его так же, но он не проверяется.. Я так понимаю, проблема в запросе?
Сума можно сойти, посмотрите пожалуйста опытным глазом..

<?php session_start(); ?>
<?php require_once("includes/db_connection.php"); ?>
<div class="lk">
            <h1>Добро пожаловать, <span><?php echo $_SESSION['login'];?>!</span></h1>
                    <form action="" method="post">
                    <div class="lk2">
                    <p class="submit"> <input class="button" id="newpass" name="newpass" type= "submit" value="Сменить пароль"></p>
                    </div>
                    </form>
                    <?
                if (isset($_POST["newpass"]))
                {?>
                            <form action="" name="changepass" method="post">
                            <div class="lk3">
                                <div class="lk4">
                                <p><label >Старый пароль<br>
                                    <input class="input" id="lastpass" name="lastpass" size="20" type="password" value=""></label></p>
 
                                <p><label>Новый пароль<br>
                                    <input class="input" id="newpass" name="newpass"size="20" type="password" value=""></label></p>
                                    
                                <p><label >Подтвердите новый пароль<br>
                                    <input class="input" id="comfirmpass" name="comfirmpass" size="20" type="password" value=""></label></p>
                                    
                                <p class="submit"><input class="button" name="change" type= "submit" value="Сменить"></p>
                                    </div>
                            </div>  
                            </form>
                        <?}?>
                
<? 
if(isset ($_POST["change"]))
  {
    if(!empty($_POST["lastpass"]))
    {
      if(!empty($_POST["newpass"]))
      {
        if($_POST["comfirmpass"] == $_POST["newpass"])
        {
          $lastpass = mysql_real_escape_string($_POST["lastpass"]);
          $newpass = mysql_real_escape_string($_POST["newpass"]);
          $comfirmpass = md5(htmlspecialchars($_POST["comfirmpass"]));
          
          $searchuser = "SELECT * FROM users WHERE login='".$_SESSION['login']."' AND password='".md5($lastpass)." ' LIMIT 1";
		  var_dump($searchuser);
            $searchresult = mysql_query($searchresult);
            if($searchresult)
            {
              $updatedata = "UPDATE users SET password='".md5($newpass)."' WHERE login='".$_SESSION['login']."'";
                $updateresult = mysql_query($updatedata);
                if($updateresult)
                {
                  echo 'Пароль успешно изменен!';
                }
            }
              else
              {
                echo 'Старый пароль введен неверно!';
              }
        }
          else
          {
            echo 'Подтверждение пароля не введено!';
          }
      }
        else
        {
          echo 'Новый пароль не указан!';
        }
    }
      else
      {
        echo 'Старый пароль не указан!';
              }
  }
?>

Comments

[alex-1917]

ты нашел что-то где-то на каких-то просторах, и просишь найти ошибку?? сначала покажи свои попытки или выглядит как сделай за меня, а это моментом в баньку
а код этот еще и гавняный, куча дырок в БД гарантирована... я такой код видел лет 12 назад, поищи тщательнее...

[skymike]

Кстати да, уже давно пора вместо mysql модуля юзать PDO или хотябы mysqli ..

Answer 1

[skymike]

$searchresult = mysql_query($searchresult);
Попробовать сменить на:
$searchresult = mysql_query($searchuser);

Comments

[DMax921]

Я тоже так делал, но проверки по старому паролю не было, это поле просто игнорировалось. Хотя новые пароль записывается...

Answer 2

[DMax921]

Сделал вот так, по запросу нахожу пользователя с ИД1
в Update если совпадает ИД и старый пароль то записываем Новый пароль.
Но есть проблема, при верном и неверном старом пароле, выводится сообщение Что пароль успешно изменен. Поправьте пожалуйста, я не понимаю...
Спасибо

<? 
if(isset ($_POST["change"]))
  {
    if(!empty($_POST["lastpass"]))
    {
      if(!empty($_POST["newpass"]))
      {
        if($_POST["comfirmpass"] == $_POST["newpass"])
        {
          $lastpass = mysql_real_escape_string($_POST["lastpass"]);
          $newpass = mysql_real_escape_string($_POST["newpass"]);
          $comfirmpass = md5(htmlspecialchars($_POST["comfirmpass"]));
          
          $searchuser = "SELECT * FROM users WHERE id='".$_SESSION['id']."' LIMIT 1";
		  var_dump($searchuser);
            $searchresult = mysql_query($searchuser);
            if($searchresult)
            {
              $updatedata = "UPDATE users SET password='".md5($newpass)."' WHERE id='".$_SESSION['id']."' AND password='".md5($lastpass)."'";
                $updateresult = mysql_query($updatedata);
                if($updateresult)
                {
                  echo 'Пароль успешно изменен!';
                }
            }
              else
              {
                echo 'Старый пароль введен неверно!';
              }
        }
          else
          {
            echo 'Подтверждение пароля не введено!';
          }
      }
        else
        {
          echo 'Новый пароль не указан!';
        }
    }
      else
      {
        echo 'Старый пароль не указан!';
              }
  }
?>

Comments

[skymike]

Верни старый запрос на место, проверки старого пароля там как таковой нету, нужно дописать, вытащить мд5 пароля с первого запроса и сравнить введённый старый пароль предварительно загнав его в мд5 с хешем вытащеным из базы. както вот так:

if($searchresult) {
          if($hash_from_base == md5($lastpass)){    
               $updatedata = "UPDATE users SET password='".md5($newpass)."' WHERE login='".$_SESSION['login']."'";
                $updateresult = mysql_query($updatedata);
                if($updateresult){
                  echo 'Пароль успешно изменен!';
                }
          } else {
                echo 'Старый пароль введен неверно!';
            }
     }

[DMax921]

skymike, ваш пример почему-то не сработал, пароль не меняется, сверки нет...
Я сделал вот так и заработало, пароль проверяется, меняется, если не верный, уведомляет, если верный - тоже., может кому-то пригодится.
Но вам отдельное спасибо, хорошего вам настроения :)

$searchuser = "SELECT * FROM users WHERE id='".$_SESSION['id']."' AND password='".md5($lastpass)." ' LIMIT 1";
            $searchresult = mysql_query($searchuser);
            if(mysql_num_rows($searchresult) > 0)
            {
              $updatedata = "UPDATE users SET password='".md5($newpass)."' WHERE id='".$_SESSION['id']."'";
                $updateresult = mysql_query($updatedata);
                if($updateresult)
                {
                  echo 'Пароль успешно изменен!';
                }
            }
              else
              {
                echo 'Старый пароль введен неверно!';
              }