Xiuno BBS 审计之问题:缓存默认走 MySQL,性能风险
贰先生 6小时前

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

现象

Xiuno BBS 4.0.4 在 conf/conf.default.php 中将缓存类型默认配置为 'type' => 'mysql',即使用 cache_mysql 类把缓存数据写入 bbs_cache 表。虽然配置项同时支持 redis/memcached/apc/xcache/yac,但默认值未启用任何内存缓存。这意味着开箱即用的部署中,所有 cache_get/cache_set 操作都退化为对 MySQL 的 SELECT/INSERT/UPDATE,缓存层不仅没有减轻数据库压力,反而增加了一份额外的数据库读写开销,与"缓存加速"的设计初衷相悖。需求清单中"仅文件缓存无 Redis"的表述失实——实际是"默认 mysql 表缓存,可选 redis/memcached"。

源码证据

1. conf.default.php 默认 cache type 为 mysql

文件:xiunobbs_4.0.4/conf/conf.default.php 第 39-64 行

'cache' => array (
    'enable' => true,
    'type' => 'mysql',                          // 默认 mysql 表缓存
    'memcached' => array (
        'host' => 'localhost',
        'port' => '11211',
        'cachepre' => 'bbs_',
    ),
    'redis' => array (
        'host' => 'localhost',
        'port' => '6379',
        'cachepre' => 'bbs_',
    ),
    'xcache' => array (
        'cachepre' => 'bbs_',
    ),
    'yac' => array (
        'cachepre' => 'bbs_',
    ),
    'apc' => array (
        'cachepre' => 'bbs_',
    ),
    'mysql' => array (
        'cachepre' => 'bbs_',
    ),
),

2. cache.func.php 根据 type 实例化,mysql 时复用 $db 连接

文件:xiunobbs_4.0.4/xiunophp/cache.func.php 第 3-21 行

function cache_new($cacheconf) {
    if($cacheconf && !empty($cacheconf['enable'])) {
        switch ($cacheconf['type']) {
            case 'redis':     $cache = new cache_redis($cacheconf['redis']);       break;
            case 'memcached': $cache = new cache_memcached($cacheconf['memcached']); break;
            case 'pdo_mysql':
            case 'mysql':
                $cache = new cache_mysql($cacheconf['mysql']); break;
            case 'xcache':    $cache = new cache_xcache($cacheconf['xcache']);    break;
            case 'apc':       $cache = new cache_apc($cacheconf['apc']);          break;
            case 'yac':       $cache = new cache_yac($cacheconf['yac']);          break;
            default: return xn_error(-1, '不支持的 cache type:'.$cacheconf['type']);
        }
        return $cache;
    }
    return NULL;
}

3. xiunophp.php 把 $db 直接传给 cache_mysql,复用同一连接

文件:xiunobbs_4.0.4/xiunophp/xiunophp.php 第 118-124 行

$db = !empty($conf['db']) ? db_new($conf['db']) : NULL;

$conf['cache']['mysql']['db'] = $db; // 这里直接传 $db,复用 $db;如果传配置文件,会产生新链接。
$cache = !empty($conf['cache']) ? cache_new($conf['cache']) : NULL;
unset($conf['cache']['mysql']['db']); // 用完清除,防止保存到配置文件

4. user.func.php 业务代码对缓存类型做判断(业务层感知缓存实现)

文件:xiunobbs_4.0.4/model/user.func.php 第 54-63 行

function user_update($uid, $arr) {
    global $conf, $g_static_users;
    $r = user__update($uid, $arr);
    $conf['cache']['type'] != 'mysql' AND cache_delete("user-$uid");   // 业务层判断 cache 类型
    isset($g_static_users[$uid]) AND $g_static_users[$uid] = array_merge($g_static_users[$uid], $arr);
    return $r;
}

业务层出现"如果缓存是 mysql 就不删除"的逻辑,说明开发者承认 mysql 缓存与业务表共用连接、删除可能引发额外压力,缓存实现细节泄漏到业务代码。

5. cache_mysql 类直接对 bbs_cache 表 SQL 操作

文件:xiunobbs_4.0.4/xiunophp/cache_mysql.class.php(存在,第 1 行被 Grep 命中)

// 摘要:cache_mysql 把 cache_get/cache_set 实现为 SELECT/INSERT ... ON DUPLICATE KEY UPDATE

风险等级与结论

架构缺陷(中)

危害:

  • 默认 mysql 缓存使所有 cache_get/cache_set 退化为数据库 SQL,缓存层失去加速意义,反而增加数据库负载
  • 缓存表与业务表共用 MySQL 实例(默认配置),缓存击穿时直接拖垮业务库
  • 业务代码(如 user_update)被迫判断 $conf['cache']['type'] 决定是否删缓存,缓存实现细节泄漏到业务层
  • 高并发场景下 mysql 缓存表锁竞争成为瓶颈
  • 需求清单中"仅文件缓存无 Redis"表述失实:实际支持 redis/memcached/apc/xcache/yac,仅默认未启用

修复建议:

  • 默认改为 'type' => 'yac'  'apc'(无需额外服务、纯内存加速),生产环境再切换 redis
  • 业务层移除对 $conf['cache']['type'] 的判断,缓存抽象层应在内部处理"删除即失效"语义
  • 文档明确说明:mysql 缓存仅为兼容性默认值,不建议生产使用
最新回复 (0)
全部楼主
返回