代码审计里的报告写作:六个检查点
同一根因的多个入口合并成一条,报告长度能降一半,可读性反而更高
正着读代码容易漏,因为入口太多;反着找危险点,数据流只有那么几条。
先说结论
先列危险点,再回溯入口,最后确认校验与权限。
- 检查点一:所有外部入口是否都过统一校验
- 检查点二:危险函数是否集中封装
- 检查点三:校验是否只做一次、是否白名单
- 检查点四:权限判断是否在服务端
- 检查点五:错误信息是否泄露内部细节
- 检查点六:是否有兜底日志
检查点一:入口
一条发现的复现步骤超过六步,开发大概率会把工单退回来让你重写,前置条件比步骤更重要,这条经验来自三次返工。
把入口列全:HTTP 参数、请求头、上传、消息队列、定时任务的外部输入。漏一个就白审。
检查点二:危险点
写清前置条件,能避免开发用管理员账号测完之后说复现不了,也能挡掉一半无效工单,工单质量差别很大。
| 类型 | 关键词 | 关注点 |
|---|---|---|
| 拼接 | SQL/命令/模板拼接 | 是否参数化 |
| 解析 | XML/JSON/反序列化 | 是否禁用外部实体 |
| 执行 | eval/system/exec | 是否可达 |
| 路径 | 文件读写、包含 | 是否规范化后校验 |
检查点三:校验逻辑
修复建议要附验证方法,否则修没修好只能靠再测一遍来确认,来回一次就是两天,排期也要按这个算。
看到「先替换再校验」「只在一个接口校验」这两种写法,基本可以直接标红。

这个问题的几个审计细节
实际代码里最容易漏的几处:
- 同一根因的多个入口合并成一条,报告长度能降一半,可读性反而更高
检查点四到六
同一根因的多个入口合并成一条,报告长度能降一半,可读性反而更高。
越权是审计里性价比最高的方向:少一行判断,就是一个中高危。错误信息要统一,日志要记入口摘要。
再补充几点
这几条不改变结论,但会改变你评估风险时的顺序。
- 决策层通常只看前两页:有多少高危、影响什么业务、需要多久能修完
- 附录里的资产清单是下一次测试的起点,比正文更常被人翻出来用,按资产维度整理别按时间堆
- 只贴工具输出,没有一句业务影响判断
- 把扫描器的低危全列上,淹没了真正要修的问题
- 修复建议写成「加强校验」,开发不知道改哪一行
审计的产出不是「找到几个洞」,是「知道这类洞在哪个模式里反复出现」。
评论 (3)
防御那节写得克制,没有教人打站,这点好评。
补一句:修完要回归,很多人修一半。
日志留存时间那条被我们领导采纳了。
发表评论