帳戶激活PHP

Question

我創建了自己的這個賬號註冊激活腳本，我都檢查一遍並再次發現錯誤，我沒有看到一個具體的錯誤...

您會是這樣：

http://domain.com/include/register.php?key=true&p=AfRWDCOWF0BO6KSb6UmNMf7d333gaBOB

它來自電子郵件，當用戶點擊它，他們重定向到該腳本：

if($_GET['key'] == true) 
{ 
    $key = $_GET['p']; 

    $sql = "SELECT * FROM users 
      WHERE user_key = '" . $key . "'"; 

    $result = mysql_query($sql) or die(mysql_error()); 

    if(mysql_affected_rows($result) > 0) 
    { 
     $sql = "UPDATE users 
       SET user_key = '', user_active = '1' 
       WHERE user_key = '" . $key . "'"; 

     $result = mysql_query(sql) or die(mysql_error()); 

     if($result) 
     { 
      $_SESSION['PROCESS'] = $lang['Account_activated']; 
      header("Location: ../index.php"); 
     } 
     else 
     { 
      $_SESSION['ERROR'] = $lang['Key_error']; 
      header("Location: ../index.php"); 
     } 
    } 
    else 
    { 
     $_SESSION['ERROR'] = $lang['Invalid_key']; 
     header("Location: ../index.php"); 
    } 
} 

它甚至沒有工作，我查看數據庫中的用戶與該密鑰，它匹配，但它繼續作爲一個錯誤，這是非常討厭我。數據庫是正確的，表和列是正確的，沒有錯誤的數據庫，這是腳本不起作用。

幫幫我，夥計。

謝謝:)

Answer 1

1. 變化$_GET['key'] == true到$_GET['key'] == "true"
2. 你這樣做if之前，一個成功的mysql_connect(...)或mysql_pconnect(...)？
3. 更改mysql_affected_rows($result);至mysql_num_rows($result);。受影響，您可以使用DELETE或UPDATESQL語句。
4. 在第二個如果被打開之前，在第二個之前添加mysql_result(...),mysql_free_result($result);以釋放分配給先前結果的內存。
5. if($result)更改爲if(mysql_affected_rows($result));。你可以在這裏做到這一點。
6. 在header(...);函數調用的後面添加一個return 0;或exit(0);取決於您的完整代碼邏輯。
7. 您正在使用$key變量SQL語句，讓你的代碼上SQL注入攻擊拿到找零$key = $_GET['p'];到$key = mysql_real_escape_string($_GET['p']);
8. 更安全，我認爲你在header()功能定位失敗。在header()的url地址應該是全部像：http://www.example.com/somewhere/index.php
9. 並檢查你的$_GET['p']變量是否存在！如果不存在，並且存在$_GET['key']，則可以找到所有激活的用戶。然後我認爲如果你有user_activated標記，那麼將user_key設置爲「'是不合適的。

Answer 2

你不應該使用：

if(mysql_affected_rows($result) > 0) 

您應該使用mysql_num_rows（）

Answer 3

您的問題是：

$result = mysql_query($sql) or die(mysql_error()); 

「或」 讓您的發言boolean所以$結果得到一個True而不是由mysql_query返回的值（）

echo 'Hello' or die('bye'); // outputs nothing, because result is True not 'Hello' 

3 or die() == True; // true 
3 or die() != 3; // true 

或相同||並且它是邏輯語句的操作符。

這將工作：

$result = mysql_query($sql); 
if(!$result) die(mysql_error());  

同樣的錯誤是在幾個小時前提出：link

---

的包裝箱，或者可以使用：

defined('FOO') or 
    define('FOO', 'BAR'); 

mysql_connect(...) or die(...); 

mysql_select_db(....) or die(...); 

mysql_query('UPDATE ...') or die(...); 

if(FOO or BAR) { ... }