提交前安全评审打分提示词

贴入待审改动 → 得到四维评分与一条改法

更新于 今天

适用场景

  • 提交前代码走查
  • 合并请求评审会议
  • 新人代码打分反馈
PROMPT
【评分维度】
1. 输入校验:检查外部入参是否逐项校验类型、长度与枚举范围;
2. 权限边界:检查每个读写动作是否校验了数据归属与当前角色;
3. 资源释放:检查连接、文件句柄与锁在异常分支是否都有关闭;
4. 可读性:检查函数长度、命名与注释能否让接手同事一次读懂。

【量表】
1. 五分:四类风险全部覆盖,且每类都有对应自动化用例;
2. 四分:风险覆盖但用例缺失,异常分支只处理了一半;
3. 三分:主路径有防护,边界值与异常分支存在明显缺口;
4. 两分及以下:存在可直接利用的注入或越权,须打回重写。

【评分理由】
1. 逐维引用被审代码原文,指出具体写法而不是笼统评价;
2. 每条扣分写清触发条件,说明会导致何种越权或数据错乱;
3. 标明证据位置,写清文件名、函数名与所在分支,便于定位;
4. 同一问题只扣一次,跨维度重复时在理由里注明关联关系。

【改进建议】
1. 只针对最低分维度给一条改法,写明改哪一行、改成什么;
2. 改法须可直接粘贴,包含参数校验语句与失败返回的错误码;
3. 另附一条回归步骤,说明改完如何用一条请求验证生效。
标签
代码审查 质量保障