此文章为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 缓存仅为兼容性默认值,不建议生产使用