NanoMQ ACL订阅权限控制缺陷分析与修复方案

背景概述

NanoMQ作为一款轻量级MQTT消息中间件，其访问控制列表(ACL)功能是保障系统安全性的核心机制。在0.20.8版本中，发现当配置deny_action参数为ignore时，订阅(subscribe)操作的ACL控制存在逻辑问题，导致未授权用户可能绕过权限限制。

问题现象

在典型的三层安全配置场景下：

1. 强制身份验证(allow_anonymous=false)
2. 默认拒绝策略(no_match=deny)
3. 拒绝行为设置为静默模式(deny_action=ignore)

即使ACL规则明确配置了订阅拒绝策略，客户端仍能成功订阅未授权主题。例如：

• 用户"CanPublish"被允许发布(publish)到test/#主题
• 用户"NoRights"被显式禁止订阅(subscribe)所有主题
• 实际测试中"NoRights"用户仍能建立订阅

技术分析

通过源码追踪发现，订阅处理逻辑(sub_handler.c)存在以下关键逻辑问题：

1. 权限检查流程中，当ACL验证结果为拒绝时：

if (permit != PERMIT_ALLOW) {
    if (work->config->acl_deny_action == ACL_DISCONNECT) {
        goto next; // 仅disconnect模式会中断流程
    }
    // 此处缺少跳转语句
}

2. 由于缺失跳转指令，控制流会继续执行后续的订阅处理逻辑，而非终止当前非法订阅请求。

解决方案

修复方案明确且简洁：在deny_action判断后无条件跳转。核心修改为：

if (permit != PERMIT_ALLOW) {
    if (work->config->acl_deny_action == ACL_DISCONNECT) {
        goto next;
    }
    goto next; // 新增关键跳转
}

安全影响评估

该问题属于中等风险：

• 攻击面：需要已认证的恶意用户
• 影响范围：违反最小权限原则，可能导致信息泄露
• 缓解措施：临时方案可将deny_action设为disconnect

最佳实践建议

1. 生产环境应及时升级到包含修复的版本
2. 复杂ACL规则配置后必须进行双向测试（允许/拒绝场景）
3. 建议采用默认拒绝策略(no_match=deny)配合明确allow规则
4. 重要业务系统应考虑启用disconnect模式增强安全性

深度思考

MQTT协议设计中，订阅权限与发布权限的分离控制是安全模型的重要特征。本次问题修复提醒开发者：

• 权限验证必须实现"fail-secure"原则
• 控制流跳转是安全逻辑的关键保障点
• 静默处理(ignore)模式需要特别谨慎对待