存储模块HAVING子句注入及表名解析缺陷修复分析

漏洞概述 该漏洞涉及两个安全问题： 1. HAVING子句注入：在 函数中， 子句被直接拼接到SQL查询中，导致潜在的SQL注入风险。 2. 逗号连接表名间隙：在 函数中， 子句后的表名列表被直接拼接到SQL查询中，可能导致表名注入。 影响范围 存储模块：漏洞存在于 文件中的 和 函数。 SQL查询：任何使用这些函数的SQL查询都可能受到攻击。 修复方案 1. HAVING子句注入修复： - 在 函数中，对 子句进行严格的验证和过滤。 - 检查 子句是否为字符串，长度是否超过限制，是否包含非法字符或关键字。 - 如果检测到非法内容，抛出 异常。 2. 逗号连接表名间隙修复： - 在 函数中，对 子句后的表名列表进行严格的验证和过滤。 - 使用正则表达式匹配合法的表名，并验证每个表名的所有权。 - 如果检测到非法内容，抛出 异常。 POC代码 ```typescript // 修复前的代码 const extractTableRefs = (sql: string): string[] => { const refs = new Set(); const refPattern = /(?:FROM for (const match of sql.matchAll(refPattern)) { refs.add(match[1]); } return [...refs]; }; // 修复后的代码 const extractTableRefs = (sql: string): string[] => { const refs = new Set(); const refPattern = /(?:FROM for (const match of sql.matchAll(refPattern)) { const tableName = match[1]; if (tableName && verifySqlAccess(tableName)) { refs.add(tableName); } } return [...refs]; }; // 修复前的代码 function sanitizeHavingClause(having: string): string { if (typeof having !== 'string') { throw new StorageRouteError('having must be a string'); } if (having.length > 200) { throw new StorageRouteError('having clause is too long (max 200 chars)'); } if (/[;\/\-\\\/\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-\-\/\.\-

目标达成感谢每一位支持者 — 我们达成了 100% 目标！

存储模块HAVING子句注入及表名解析缺陷修复分析

同源更多文章

目标达成 感谢每一位支持者 — 我们达成了 100% 目标！

存储模块HAVING子句注入及表名解析缺陷修复分析

同源更多文章

目标达成感谢每一位支持者 — 我们达成了 100% 目标！