阻止用户编辑其他条目

时间:2015-07-03 15:35:56

标签: php mysql authentication

我有一个PHP脚本,允许用户将条目注册到数据库。条目是自动增量的。我发现,用户#A可以通过将edit.php?id = 2更改为id = 1来获取用户#B的条目。

当然我想阻止它。所以我的想法是:如果mysql条目中的用户ID字段与我的php脚本中的$ _SESSION ['user_id']匹配,则允许编辑。

用户应该只能编辑他们自己发布的条目。

实现这一目标的最佳和最有效的方法是什么?

<?php $bruker = $_SESSION['user_id']; ?>

<?php }
/*
   EDIT RECORD
*/
// if the 'id' variable is set in the URL, we know that we need to edit a record
if (isset($_GET['id']))
{
    // if the form's submit button is clicked, we need to process the form
    if (isset($_POST['submit']))
    {
        // make sure the 'id' in the URL is valid
        if (is_numeric($_POST['id']))
        {
            // get variables from the URL/form
            $id = $_POST['id'];
            $elv = htmlentities($_POST['elv'], ENT_QUOTES);
            $vald = htmlentities($_POST['vald'], ENT_QUOTES);
            $art = htmlentities($_POST['art'], ENT_QUOTES);
            $dato = htmlentities($_POST['dato'], ENT_QUOTES);
            $vekt = (int)$_POST['vekt'];
            $lengde = (int)$_POST['lengde'];
            $flue = htmlentities($_POST['flue'], ENT_QUOTES);
            $gjenutsatt = (int)$_POST['gjenutsatt'];
            $kjonn = (int)$_POST['kjonn'];
            $bilde = htmlentities($_POST['bilde'], ENT_QUOTES);
            $user = $_SESSION['user_id'];

            // check that required fields are not empty
            if ($elv == '' || $vald == '' || $art == '' || $dato == '' || $vekt == '' || $kjonn == '')
            {
                // if they are empty, show an error message and display the form
                $error = 'Du må fylle ut de påkrevde feltene!';
                renderForm($elv, $vald, $art, $dato, $vekt, $lengde, $flue, $gjenutsatt, $kjonn, $bilde, $user, $error, $id);
            }
            else
            {
                // if everything is fine, update the record in the database
                if ($stmt = $mysqli->prepare("UPDATE fisk SET elv = ?, vald = ?, art = ?, dato = ?, vekt = ?, lengde = ?, flue = ?, gjenutsatt = ?, kjonn= ?, bilde = ?, user = ?
                    WHERE id=?"))
                {
                    $stmt->bind_param("ssssiisiisii", $elv, $vald, $art, $dato, $vekt, $lengde, $flue, $gjenutsatt, $kjonn, $bilde, $user, $id);
                    $stmt->execute();
                    $stmt->close();
                }
                // show an error message if the query has an error
                else
                {
                    echo "ERROR: could not prepare SQL statement.";
                }

                // redirect the user once the form is updated
                header("Location: /");
            }
        }
        // if the 'id' variable is not valid, show an error message
        else
        {
            echo "Error!";
        }
    }
    // if the form hasn't been submitted yet, get the info from the database and show the form
    else

1 个答案:

答案 0 :(得分:0)

假设您的用户拥有唯一ID,您只需在SQL中添加其他WHERE子句:

  

if($ stmt = $ mysqli-&gt; prepare(“UPDATE fisk SET elv = ?, vald = ?, art =   ?,dato = ?, vekt = ?, lengde = ?, flue = ?, gjenutsatt = ?, kjonn = ?,,   bilde = ?, user =?                       WHERE id =? AND created_user =?“))

显然,将created_user替换为用于存储创建条目的用户ID的列。

这样它只会更新用户尝试编辑它时创建的行。

更安全的是,您可以通过首先查询所讨论行的已创建用户ID然后根据您的建议检查用户ID $ _SESSION来阻止他们看到该页面 - 然后杀死脚本或重定向它们之前进入查询。