PHP CSRF 令牌代码。安全性足够吗？

Question

最近，我尽了最大努力保护我的网站免受 XSS 侵害，现在我正在防范 CSRF，在观看和阅读了一些有关此事的文章后，我创建了以下代码。

我想知道我的实现在数据库中使用字符串来帮助安全是否正确。我应该做些什么不同的事情吗？我应该检查双方的数据库吗？

代码：

if(!isset($_SESSION['register_token'])){

  $keytype = 'register';

  $getregisterkey = mysql_query("SELECT key FROM tokenkeys WHERE type='".$keytype."' ") or die(mysql_error()); 
  while ($row = mysql_fetch_array($getregisterkey))
  {
$registerkey = mysql_real_escape_string($row['key']);
  }; 


  $_SESSION['register_token']=sha1(uniqid(rand(), TRUE).$registerkey);
  $_SESSION['register_token_time']=time();
}

<input type="hidden" name="token" value="<?php echo $_SESSION['register_token'];?>" />



if($_POST['register_token']==$_SESSION['register_token']){
  $register_token_age=time()-$_SESSION['register_token_time'];
  if($register_token_age=>300){

    //process form
  } else{
    //valid token but expired
  }
} else{
  die('Access Forbidden')

}

Answer 1

XSRF 令牌的安全性取决于发送页面的通道。在此表单上使用 https 并且仅使用 https，并且仅将其提交到 https 端点。否则，MITM 可以在提供或提交表单时从表单中获取您的 XSRF 令牌。

while ($row = mysql_fetch_array($getregisterkey))
{
    $registerkey = mysql_real_escape_string($row['key']);
}; 

当有零行时永远不会执行，这种情况下，当您稍后这样做时，您不会从 $getregisterkey 获得太多熵，

$_SESSION['register_token']=sha1(uniqid(rand(), TRUE).$registerkey);

如果返回零行，我将确保您的实现快速失败。也许更改为 if ($row = mysql_fetch_array(...)) { ... } else { /* abort */ } 因为您无法从额外的行中获得任何好处。

rand() 必须是真正随机的或加密强度高 PRNG。

我不熟悉 PHP 的标准库，但 [wikipedia] 表明 rand() 的加密强度不高。 维基百科说

有人建议向 PHP 添加强随机数生成功能。

PHP 中的强加密 建议使用 openssl_random_pseudo_bytes()< /代码>

不要使用 rand() 或 mt_rand()

要在 PHP 中生成加密的强随机数，您必须使用 OpenSSL 库的函数 openssl_random_pseudo_bytes()。

如果您使用弱随机性，那么攻击者可以观察您生成的数字（通过请求多个版本的表单并解析隐藏的输入）并使用它来找出下一个数字可能是什么并伪造 CSRF 令牌。

如果攻击者修改 'register_token_time' 会话属性，那么他们就可以避开您的 XSRF 检查。

例如，如果您有一个页面这样做

$_SESSION[$_POST['x']] = $_POST['y'];

，那么攻击者可以通过 POST

x=register_token&y=pwnd

来替换会话中存储的 register_token，然后发送带有 XSRF 保护的帖子

token=pwnd

并绕过您的 XSRF 保护。