Join our Newsletter — 33% off our NHI Course
Home FAQ Governance, Ownership & Risk Who is accountable when pentest findings are not…
Governance, Ownership & Risk

Who is accountable when pentest findings are not revalidated after code fixes are merged?

← Back to all FAQ
By NHI Mgmt Group Editorial Team Updated September 7, 2026 Domain: Governance, Ownership & Risk

Accountability should sit with the teams that own remediation and release governance. Developers need to verify the fix, security needs to confirm the issue is no longer exploitable, and release owners should require revalidation before closing the finding. Without that loop, organisations can mistake a code change for a resolved risk.

Accountability Breaks Down at the Hand-off Between Remediation and Closure

When pentest findings are fixed in code but never revalidated, the problem is rarely a lack of ownership in principle. The failure is usually in the hand-off: engineering treats the merge as evidence of resolution, security assumes someone else has checked the exploit path, and release governance closes the item without proof. That creates a governance gap, not just a technical one, because the original risk can survive even though the codebase has changed. NIST SP 800-53 Rev. 5 is relevant here because it ties remediation discipline to control evidence, verification, and accountable change handling rather than to intent alone.

In practice, many security teams encounter this only after a later test or incident shows that the original weakness was never actually eliminated.

How Revalidation Fits Into Fix, Test, and Release

Revalidation is the step that confirms the exploit condition is gone, not merely that a ticket moved state. A code fix can address the visible defect while leaving adjacent logic, configuration, or deployment behaviour unchanged, so the original finding should not be marked closed until someone has checked the live or staged path that pentest identified. That means the remediation owner must demonstrate the change, security must review the result against the original test objective, and the release process must treat closure as dependent on evidence rather than on merge completion.

A useful way to think about the process is:

  • Engineering owns the fix and the technical explanation of what changed.
  • Security owns the validation of whether the issue is still exploitable.
  • Release or product governance owns the rule that prevents closure without that confirmation.

This matters because pentest findings often describe an attack condition, not just a defect. If the fix changes one layer but leaves the same trust boundary, input path, or privilege assumption in place, the finding may still stand in a modified form. The most common failure is treating code review, unit tests, or merge approval as equivalent to security revalidation when they answer different questions. A strong control process records the original finding, the remediation change, the re-test outcome, and the closure decision as separate evidence. Where teams do not have the bandwidth to re-test every item immediately, they should at least distinguish between “fixed in code” and “verified resolved” so closure does not outrun assurance. This guidance breaks down when the original pentest finding was too vague to reproduce, because then revalidation cannot be anchored to a specific exploit path.

When “Fixed” and “Closed” Should Not Mean the Same Thing

Tighter closure discipline often increases delivery overhead, so organisations have to balance speed against assurance.

Some findings are easy to revalidate because the defect is narrow and the fix is obvious. Others are edge-case driven, depend on deployment settings, or only become visible in an integrated environment. In those cases, the question is not whether the code changed, but whether the conditions that made the issue exploitable still exist. If a team cannot reproduce the original test conditions, it should treat the item as open or partially verified, not silently close it on confidence alone. That is especially important where code fixes are merged quickly across multiple branches, because the implemented patch in one environment may not match what actually reached production.

There is also a governance distinction between remediation ownership and acceptance of residual risk. If a finding is intentionally deferred, the organisation should record that as an explicit exception with an accountable owner, not as an unverified closure. That distinction helps avoid a common reporting error: dashboards that show “resolved” items while the underlying exposure is merely unconfirmed. Where teams operate with strict release gates, revalidation should be a required condition for closure; where they operate with lighter governance, the minimum standard should still be a documented confirmation that the original exploit path no longer works.

Standards & Framework Alignment

This section maps relevant standards and security frameworks to the operational risks and controls described in this guidance.

NIST CSF 2.0 and CIS Controls v8 set the governance and control requirements practitioners need to meet.

FrameworkControl / ReferenceRelevance
NIST CSF 2.0GV.RM-01 — Risk Management StrategyRevalidation is a governance control on residual risk before closure.
PR.IP-12 — Vulnerability ManagementPentest findings are vulnerability items that need verification after remediation.
ID.RA-06 — Threat and Vulnerability AnalysisRevalidation confirms whether the original vulnerability still exists after change.
Recommendation — Require evidence-based closure for remediated findings before accepting residual risk. Verify that remediation eliminated the vulnerability before marking it closed. Reassess exploitability after fixes to confirm the exposure is removed.
CIS Controls v87.1 — Establish and Maintain a Vulnerability Management ProcessTracks remediation, validation, and closure of findings as an operational process.
17.1 — Establish and Maintain an Incident Response ProcessClosure discipline supports accountability when weaknesses remain unverified.
Recommendation — Document re-test evidence before closing vulnerability remediation records. Escalate unresolved findings through formal ownership and exception handling.

Practitioner Guidance

What to prioritise: Treat closure evidence as part of remediation, not as an optional postscript. If the team that merged the fix cannot show that the original exploit path was rechecked, the finding should remain open or move to a verified-pending state.

Decision rule: If the code change is not directly tied to a successful re-test of the original finding, do not let release completion override security validation. If the finding cannot be reproduced, escalate the case for triage rather than assuming the risk disappeared.

What to verify: Confirm that the fix addressed the actual attack condition, not just the visible symptom. Verify the result in the same environment class, with the same access path or trust boundary that made the finding material in the first place.

Practitioner takeaway: The accountable party for revalidation is the remediation-and-release chain as a whole, but the control only works when closure is evidence-led and security retains the right to say “not yet resolved.”

Deepen Your Knowledge

Sign up to our weekly newsletter — get 33% off our NHI Foundation Level Course

    NHIMG Editorial Note
    Reviewed and updated by the NHIMG editorial team on September 7, 2026.
    NHI Mgmt Group — the #1 independent authority on Non-Human Identity, IAM, and Agentic AI security. nhimg.org