2014-02-28 97 views
1

所以,我有以下代码:数据库密码检查

<?php 

    mysql_connect("HOSTADDRESS", "USERNAME", "PASS") or die(mysql_error()); 

    mysql_select_db("DATABASENAME") or die(mysql_error()); 

    //Checks if there is a login cookie; 

    if(isset($_COOKIE["ID_my_site"])) 

     //If there is, it logs you in and directs you to the member page 

    { 

     $username = $_COOKIE["ID_my_site"]; 

     $pass = $_COOKIE["ID_my_site"]; 

     $check = mysql_query("SELECT * FROM userdata WHERE emailaddress = '$emailaddress'") or die(mysql_error()); 

     while($info = mysql_fetch_array($check)) 

     { 

      if ($pass != $info["password"]) 

      { 

      } 

      else 

      { 

       header("Location: members.php"); 

      } 
     } 

    } 

    //If the login form is submitted; 

    if (isset($_POST["submit"])) { //If form has been submitted 

     //Makes sure they are filled in 

     if(!$_POST["emailaddress"] | !$_POST["pass"]) { 

      die("You did not fill in all required fields."); 

     } 

     //Checks it against the database; 

     if (!get_magic_quotes_gpc()) { 

      $_POST["email"] = addslashes($_POST["email"]); 

     } 

     $check = mysql_query("SELECT * FROM userdata WHERE emailaddress = '".$_POST["emailaddress"]."'") or die(mysql_error()); 

     //Gives error if user doesn't exist; 

     $check2 = mysql_num_rows($check); 

     if ($check2 == 0) { 

      die("That users does not exist in our database. <a href=register.php>Click here to register</a>"); 

     } 

     while($info = mysql_fetch_array($check)) 

     { 

      $_POST["pass"] = stripslashes($_POST["pass"]); 

      $info["password"] = stripslashes($info["password"]); 

      $_POST["pass"] = md5($_POST["pass"]); 

      //Gives error if the password is wrong 

      if ($_POST["pass"] != $info["password"]) { 

       die("Incorrect password, please try again."); 

      } 

      else 

      { 

       //If login is ok then we add a cookie 

       $_POST["emailaddress"] = stripslashes($_POST["emailaddress"]); 

       $hour = time() + 3600; 

       setcookie(ID_my_site, $_POST["emailaddress"], $hour); 

       setcookie(Key_my_site, $_POST["pass"], $hour); 

       //Then it redirects them to the members area 

       header("Location: members.php"); 

      } 

     } 

    } 

    else 

    { 

     //If they are not logged in 

     ?> 

     <form action="<?php echo $_SERVER['PHP_SELF']?>" method="post"> 

     <table border="0"> 

     <tr><td colspan=2><h1>Login</h1></td></tr> 

     <tr><td>Email Address:</td><td> 

     <input type="text" name="emailaddress" maxlength="60"> 

     </td></tr> 

     <tr><td>Password:</td><td> 

     <input type="password" name="pass" maxlength="12"> 

     </td></tr> 

     <tr><td colspan="2" align="right"> 

     <input type="submit" name="submit" value="Login"> 

     </td></tr> 

     </table> 

     </form> 

     <?php 

    } 

    ?> 

当我尝试通过网站登录,即使密码是相同的数据库的密码,它读取“密码不正确,请再试一次。'如果我使用数据库中找到的加密版本尝试密码,它也会显示此消息。请有人可以帮我解决这个问题吗?

+3

在你的问题粘贴代码,而不是一个可怕的广告出没的外部网站。 – geoffspear

+2

**危险**:您正在使用[一个**过时的**数据库API](http://stackoverflow.com/q/12859942/19068),并应使用[现代替换](http:// php。净/手动/ EN/mysqlinfo.api.choosing.php)。你也**易受[SQL注入攻击](http://bobby-tables.com/)**,现代的API会使[防御]更容易(http://stackoverflow.com/questions/60174/best-way-to-prevent-sql-injection-in-php)自己从。 – Quentin

+1

您正在使用[不适合的哈希算法](http://php.net/manual/en/faq.passwords.php),需要[更好地照顾](https://www.owasp.org/index。 php/Password_Storage_Cheat_Sheet)的用户密码。 – Quentin

回答

-1

if(!$_POST["emailaddress"] | !$_POST["pass"]) {

使用||等都将成为

if(!$_POST["emailaddress"] || !$_POST["pass"]) {