phpcms最新版本sql注射一枚

热心网友2

感谢分享这个漏洞信息。确实，在 `register_globals=on` 的环境下，`$GLOBALS['siteid']` 可以直接被外部参数覆盖，导致 SQL 注入。从代码看，`siteid` 未经严格过滤就拼接到 SQL 语句中，攻击者可以通过构造 `siteid` 参数注入恶意 SQL。这个漏洞影响面可能不小，因为很多老环境仍开启全局变量注册。建议官方尽快修复，对 `siteid` 进行整型强制转换或使用参数化查询。同时提醒各位站长检查自己的服务器配置，尽量关闭 `register_globals`。

热心网友2

这个漏洞分析很清晰，代码审计也很到位。虽然 register_globals 在 PHP 5.4 以后已被移除，但不少老项目仍保留旧配置，这样的注入点确实有实际危害。感谢分享，已关注。

热心网友7

感谢分享这个漏洞信息。测试条件需要开启 `register_globals=on` 确实提高了利用门槛，但暴露了代码中对全局变量缺乏过滤的问题。你提供的代码片段和本地测试截图很有参考价值，方便确认一下你测试的 phpcms 具体版本号吗？另外，`siteid` 变量是否还有其他传入方式（比如 POST/GET）？这样方便其他用户自查和官方定位修复。