即使应该返回false,SQL也返回true

时间:2014-01-26 22:33:24

标签: php sql

我有一个“设置密码”网站,只有当前没有密码才能设置密码。因此,我有一个SQL查询,只有当电子邮件与数据库中的电子邮件匹配且密码为NULL时,才会选择我们将使用的用户名和密码。如果这个返回false,那么除了echo“密码或用户名无效”之外,PHP脚本不应该执行任何操作。它还可以执行其他操作,例如检查密码的长度以及是否已确认密码,但这是一个搞乱的问题。有什么想法吗?

到目前为止,这是我的PHP代码:

<?php
session_start();
require 'connect.inc.php';
include 'core.php';
ob_start();
?>

<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />

<?php
echo "
            <form action='update.php' method='post'>
            Ange din E-post.<br>
            E-post: <input type='text' name='mail' /><br>
            Ange ditt lösenord, minst sex tecken.<br>
            Lösenord: <input type='password' name='pass1' /><br>
            Bekräfta: <input type='password' name='pass2' /><br>
            <input type='submit' value='Godkänn'/><br>";

if(isset($_POST['pass1'])&&isset($_POST['pass2']))
            {

                $email = $_POST['mail'];
                $query = "SELECT login_id, pass FROM users WHERE (email = '$email') AND (pass IS NULL)";
                $query_run = sqlsrv_query( $link, $query);



                if(($_POST['pass1'] == $_POST['pass2']) && !empty($_POST['pass2']) && $query_run && (strlen($_POST['pass2'])>5))
                {
                    $row = sqlsrv_fetch_array($query_run, SQLSRV_FETCH_ASSOC);

                    $id = $row['login_id'];
                    $_SESSION['id'] = $id;

                    $pass = $_POST['pass1'];

                    $query2 = "UPDATE users SET pass = '$pass' WHERE login_id = '$id'";
                    sqlsrv_query( $link, $query2);

                    $next = "https://localhost/title_choice.php";
                    header('Location: '.$next);
                }else
                {
                    echo "Lösenorden matchar ej, eller E-post ogiltig! Var god ange dem igen!";
                }
            }

ob_end_flush();
?>

1 个答案:

答案 0 :(得分:2)

看起来你正在做的是检查查询是否已经运行。此语句将检查查询是否已运行:

if(($_POST['pass1'] == $_POST['pass2']) && !empty($_POST['pass2']) && $query_run && (strlen($_POST['pass2'])>5))

相反,您应该检查查询是否返回任何内容。您可以像这样使用sqlsrv_has_rows

$rows = sqlsrv_has_rows($query_run);
if(($_POST['pass1'] == $_POST['pass2']) && !empty($_POST['pass2']) && $rows && (strlen($_POST['pass2'])>5))
{
    //rest of your code here
}
else{
    //no rows returned
}