代码审计里的报告写作:六个检查点

渗透测试 · 3 分钟阅读· 310 次阅读

同一根因的多个入口合并成一条,报告长度能降一半,可读性反而更高

正着读代码容易漏,因为入口太多;反着找危险点,数据流只有那么几条。

先说结论

先列危险点,再回溯入口,最后确认校验与权限。

  • 检查点一:所有外部入口是否都过统一校验
  • 检查点二:危险函数是否集中封装
  • 检查点三:校验是否只做一次、是否白名单
  • 检查点四:权限判断是否在服务端
  • 检查点五:错误信息是否泄露内部细节
  • 检查点六:是否有兜底日志

检查点一:入口

一条发现的复现步骤超过六步,开发大概率会把工单退回来让你重写,前置条件比步骤更重要,这条经验来自三次返工。

把入口列全:HTTP 参数、请求头、上传、消息队列、定时任务的外部输入。漏一个就白审。

检查点二:危险点

写清前置条件,能避免开发用管理员账号测完之后说复现不了,也能挡掉一半无效工单,工单质量差别很大。

类型 关键词 关注点
拼接 SQL/命令/模板拼接 是否参数化
解析 XML/JSON/反序列化 是否禁用外部实体
执行 eval/system/exec 是否可达
路径 文件读写、包含 是否规范化后校验

检查点三:校验逻辑

修复建议要附验证方法,否则修没修好只能靠再测一遍来确认,来回一次就是两天,排期也要按这个算。

看到「先替换再校验」「只在一个接口校验」这两种写法,基本可以直接标红。

代码审计里的报告写作

这个问题的几个审计细节

实际代码里最容易漏的几处:

  • 同一根因的多个入口合并成一条,报告长度能降一半,可读性反而更高

检查点四到六

同一根因的多个入口合并成一条,报告长度能降一半,可读性反而更高。

越权是审计里性价比最高的方向:少一行判断,就是一个中高危。错误信息要统一,日志要记入口摘要。

再补充几点

这几条不改变结论,但会改变你评估风险时的顺序。

  • 决策层通常只看前两页:有多少高危、影响什么业务、需要多久能修完
  • 附录里的资产清单是下一次测试的起点,比正文更常被人翻出来用,按资产维度整理别按时间堆
  • 只贴工具输出,没有一句业务影响判断
  • 把扫描器的低危全列上,淹没了真正要修的问题
  • 修复建议写成「加强校验」,开发不知道改哪一行

审计的产出不是「找到几个洞」,是「知道这类洞在哪个模式里反复出现」。

评论 (3)

白
白帽小陈

防御那节写得克制,没有教人打站,这点好评。

南
南风

补一句:修完要回归,很多人修一半。

K
Kai

日志留存时间那条被我们领导采纳了。

发表评论

验证码

提交后需站长审核,通过后才会显示。