会话检查绕过漏洞

Question

我们有一个 php 页面，它是网站的管理部分。它用于对数据库执行一些更新操作。代码如下所示

session_start(); 
     if (!isset($_SESSION['somevariable']) ) 
        {
        header("Location:loginpage.php");
        }



    $id=$_GET['somevariable];
$sql = "UPDATE sometable SET somecolumn='' where someothercolumn=?";
$stmt = $con->prepare($sql);
$stmt->bind_param('s',$id);
$stmt->execute();

我们注意到存在一些漏洞，这段代码似乎是从未知来源以周期性间隔（5 秒）运行的，这看起来不像有人拥有管理部分的密码并且正在运行手动操作。

我们想知道黑客能否绕过此登录检查并在没有密码的情况下执行其余代码？对上述代码中的漏洞的任何见解都会有所帮助。提前致谢

Answer 1

我看到了两个漏洞：

1) CSRF（直接从get方法中使用变量）

2) 调用头函数后退出不使用

正确的代码应该是这样的：

<?php
header("Location: http://www.example.com/"); /* Redirect browser */

/* Make sure that code below does not get executed when we redirect. */
exit;
?>

参见 php.net 文档 Link

Answer 2

如果管理员有一个活动会话，并且黑客以某种方式发现了他们的 PHPSESSID cookie 的值，他们可以发送 cookie 并劫持会话。

确保您使用 SSL 加密，以防止有人窃听连接并获取 cookie。