V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
gouchaoer
V2EX  ›  PHP

把 PDO::quote 的结果去掉两边的单引号,其结果和 mysqli_real_escape_string 等效么,安全么?

  •  
  •   gouchaoer · 2017-10-12 18:39:08 +08:00 · 2967 次点击
    这是一个创建于 2583 天前的主题,其中的信息可能已经有所发展或是发生改变。
    PDO::quote 的字符串有个麻烦就是两边带上了单引号,现在我在处理一些历史遗留代码,所以直接写了个函数类似这样:
    function escape($str)
    {
    global $pdo;
    return substr($pdo->quote($str), 1, -1);
    }

    然后实际这样用:
    $id=$_GET['id'];
    $id=escape($id);
    $sql="SELECT * FROM t WHERE id={$id}";
    ...

    这样安全么?
    第 1 条附言  ·  2017-10-13 10:55:00 +08:00
    写错了,是这样的:$sql="SELECT * FROM t WHERE id='{$id}'";
    7 条回复    2017-10-13 14:35:20 +08:00
    gouchaoer
        1
    gouchaoer  
    OP
       2017-10-12 18:40:17 +08:00
    mysqli_real_escape_string 就不带两边的单引号,不过这里用的是 PDO
    WuwuGin
        2
    WuwuGin  
       2017-10-12 19:28:24 +08:00 via Android
    pdo queto 就是添加单引号和处理转义特殊字符的,单引号里面的内容不会被 PHP 执行。你这样去掉等于没有使用此函数。
    如果想避免注入,使用 prepare 函数,在需要执行用户输入的参数的时间,使用 bindParam 或者 bindValue。
    2ME
        3
    2ME  
       2017-10-13 09:44:05 +08:00
    prepare 预处理 bind 绑定设置参数 PDO::PARAM_INT
    gouchaoer
        4
    gouchaoer  
    OP
       2017-10-13 10:55:11 +08:00
    @WuwuGin 写错了,是这样的:$sql="SELECT * FROM t WHERE id='{$id}'";
    gouchaoer
        5
    gouchaoer  
    OP
       2017-10-13 10:56:02 +08:00
    我知道 bind 好,但是历史代码是那样的,你 bind 要改很多逻辑,我没办法保证不出问题,还是对输入消毒改动小
    gouchaoer
        6
    gouchaoer  
    OP
       2017-10-13 11:01:02 +08:00
    WuwuGin
        7
    WuwuGin  
       2017-10-13 14:35:20 +08:00 via Android
    @gouchaoer 其实我觉得你这样是符合 queto 函数的最终目的的。但是防注入不光靠 queto 啊。
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   3130 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 23ms · UTC 12:46 · PVG 20:46 · LAX 04:46 · JFK 07:46
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.