Xiuno BBS 审计之问题：无读写分离，读压主库

贰先生 一级用户组发布于 5小时前

此文章为XIUNOX版本重构审计时发现问题，XIUNOX版本已优化修复此问题。分享出来方便后续想基于xiuno bbs4.0.4版本制作维护版本或插件模板等需求的开发者和站长参考。

现象

Xiuno BBS 4.0.4 在 db_mysql 类中提供了主从连接能力（master + slaves），但默认配置 'slaves' => array() 为空，从库连接回退到主库连接（$this->rlink = $this->wlink）。业务层调用 db_find_one/db_find/db_count/db_maxid 等读操作时，虽然 db_mysql::find_one 内部会调用 connect_slave() 获取读连接，但由于 slaves 为空，所有读请求最终都打到主库。系统未强制要求生产环境配置 slaves，也未提供读请求路由到从库的明确策略，导致"支持主从"形同虚设。

源码证据

1. conf.default.php slaves 默认为空数组

文件：xiunobbs_4.0.4/conf/conf.default.php 第 12-37 行

'db' => array (
    'type' => 'mysql',
    'mysql' => array (
        'master' => array (
            'host' => 'localhost',
            'user' => 'root',
            'password' => 'root',
            'name' => 'test',
            'tablepre' => 'bbs_',
            'charset' => 'utf8',
            'engine' => 'innodb',
        ),
        'slaves' => array (),                          // 默认空，无从库
    ),
    'pdo_mysql' => array (
        'master' => array (
            // ...
        ),
        'slaves' => array (),                          // 同样为空
    ),
),

2. db_mysql::connect_slave 在 slaves 为空时回退到 master 连接

文件：xiunobbs_4.0.4/xiunophp/db_mysql.class.php 第 36-50 行

// 连接从服务器，如果有多台，则随机挑选一台，如果为空，则与主服务器一致。
public function connect_slave() {
    if($this->rlink) return $this->rlink;
    if(empty($this->conf['slaves'])) {
        if($this->wlink === NULL) $this->wlink = $this->connect_master();
        $this->rlink = $this->wlink;                  // 回退到主连接
        $this->rconf = $this->conf['master'];
    } else {
        $n = array_rand($this->conf['slaves']);
        $conf = $this->conf['slaves'][$n];
        $this->rconf = $conf;
        $this->rlink = $this->real_connect($conf['host'], $conf['user'], $conf['password'], $conf['name'], $conf['charset'], $conf['engine']);
    }
    return $this->rlink;
}

3. db.func.php 读函数虽传 $d 参数，但默认 N ULL 回退到$ _SERVER['db'] 主连接

文件：xiunobbs_4.0.4/xiunophp/db.func.php 第 60-99 行

function db_sql_find($sql, $key = NULL, $d = NULL) {
    $db = $_SERVER['db'];                              // 主连接
    $d = $d ? $d : $db;                                // 默认用主
    if(!$d) return FALSE;
    $arr = $d->sql_find($sql, $key);
    db_errno_errstr($arr, $d, $sql);
    return $arr;
}

function db_count($table, $cond = array(), $d = NULL) {
    $db = $_SERVER['db'];
    $d = $d ? $d : $db;
    if(!$d) return FALSE;
    $r = $d->count($d->tablepre.$table, $cond);
    db_errno_errstr($r, $d);
    return $r;
}

业务调用几乎从不传 $d 参数：

文件：xiunobbs_4.0.4/model/user.func.php 第 27 行

function user__read($uid) {
    $user = db_find_one('user', array('uid'=>$uid));   // 不传 $d，用主连接
    return $user;
}

全站 db_find_one、db_find、db_count、db_maxid 调用均不显式传从库连接，依赖 db_mysql::find_one 内部调用 connect_slave()。但 connect_slave() 在 slaves 为空时返回主连接，读请求实际打到主库。