Fp Check
Trail of BitsOfficialPublished by Trail of Bits
- Skill
- Review and audit
- Security
- CC-BY-SA-4.0
- No issues found
AutoGineer summary
Written by AutoGineer from the skill’s own files, not by the author
fp-check provides a structured methodology for validating suspected security bugs. It guides the user through a rigorous verification process to determine if a finding is a true positive or a false positive, supported by documented evidence. The skill distinguishes between standard and deep verification paths based on the complexity of the bug, ensuring that each claim is thoroughly analyzed against its specific context, data flow, and threat model.
What it does
- Restates and clarifies vulnerability claims to identify potential misunderstandings.
- Routes verification tasks between standard and deep paths based on bug complexity.
- Performs data flow analysis to trace inputs from source to sink.
- Applies class-specific verification requirements for various bug types.
- Produces a final verdict of TRUE POSITIVE or FALSE POSITIVE with supporting evidence.
- Evaluates findings against mandatory gate reviews.
When to use it
- When you need to verify if a specific security finding is real or exploitable.
- When you need to confirm if a reported bug is a false positive.
- When you have a clear, specific vulnerability claim that requires validation.
- When standard verification steps are insufficient for complex or cross-component bugs.
Advantages
- Eliminates guesswork by requiring evidence-based verification.
- Prevents common analytical errors through a structured devil's advocate review.
- Provides a clear, repeatable process for triaging multiple suspected bugs.
- Ensures context-aware analysis by considering architectural and historical factors.
Usage examples
- “Verify if the reported heap buffer overflow in parse_header is a true positive.”
- The agent will restate the claim, perform data flow analysis, check the specific memory corruption conditions, and provide a verdict with evidence.
- “Is this SQL injection finding a false positive?”
- The agent will trace the data flow from the input source to the database sink and verify if the input is properly sanitized or parameterized.
Install
Download the skill as a ZIP:
https://autogineer.com/agent-skills/trailofbits/fp-check/downloadUnzip it into the cross-client skills folder of your project:
mkdir -p .agents/skills && unzip fp-check.zip -d .agents/skills
Documentation by Trail of Bits
False Positive Check
When to Use
"Is this bug real?" or "is this a true positive?"
"Is this a false positive?" or "verify this finding"
"Check if this vulnerability is exploitable"
Any request to verify or validate a specific suspected bug
When NOT to Use
Finding or hunting for bugs ("find bugs", "security analysis", "audit code")
General code review for style, performance, or maintainability
Feature development, refactoring, or non-security tasks
When the user explicitly asks for a quick scan without verification
Rationalizations to Reject
If you catch yourself thinking any of these, STOP.
| Rationalization | Why It's Wrong | Required Action |
|---|---|---|
| "Rapid analysis of remaining bugs" | Every bug gets full verification | Return to task list, verify next bug through all phases |
| "This pattern looks dangerous, so it's a vulnerability" | Pattern recognition is not analysis | Complete data flow tracing before any conclusion |
| "Skipping full verification for efficiency" | No partial analysis allowed | Execute all steps per the chosen verification path |
| "The code looks unsafe, reporting without tracing data flow" | Unsafe-looking code may have upstream validation | Trace the complete path from source to sink |
| "Similar code was vulnerable elsewhere" | Each context has different validation, callers, and protections | Verify this specific instance independently |
| "This is clearly critical" | LLMs are biased toward seeing bugs and overrating severity | Complete devil's advocate review; prove it with evidence |
Step 0: Understand the Claim and Context
Before any analysis, restate the bug in your own words. If you cannot do this clearly, ask the user for clarification. Half of false positives collapse at this step — the claim doesn't make coherent sense when restated precisely.
Document:
What is the exact vulnerability claim? (e.g., "heap buffer overflow in
parse_header()whencontent_lengthexceeds 4096")What is the alleged root cause? (e.g., "missing bounds check before
memcpyat line 142")What is the supposed trigger? (e.g., "attacker sends HTTP request with oversized Content-Length header")
What is the claimed impact? (e.g., "remote code execution via controlled heap corruption")
What is the threat model? What privilege level does this code run at? Is it sandboxed? What can the attacker already do before triggering this bug? (e.g., "unauthenticated remote attacker vs privileged local user"; "runs inside Chrome renderer sandbox" vs "runs as root with no sandbox")
What is the bug class? Classify the bug and consult bug-class-verification.md for class-specific verification requirements that supplement the generic phases below.
Execution context: When and how is this code path reached during normal execution?
Caller analysis: What functions call this code and what input constraints do they impose?
Architectural context: Is this part of a larger security system with multiple protection layers?
Historical context: Any recent changes, known issues, or previous security reviews of this code area?
Route: Standard vs Deep Verification
After Step 0, choose a verification path.
Standard Verification
Use when ALL of these hold:
Clear, specific vulnerability claim (not vague or ambiguous)
Single component — no cross-component interaction in the bug path
Well-understood bug class (buffer overflow, SQL injection, XSS, integer overflow, etc.)
No concurrency or async involved in the trigger
Straightforward data flow from source to sink
Follow standard-verification.md. No task tracking — work through the linear checklist sequentially, documenting findings inline.
Deep Verification
Use when ANY of these hold:
Ambiguous claim that could be interpreted multiple ways
Cross-component bug path (data flows through 3+ modules or services)
Race conditions, TOCTOU, or concurrency in the trigger mechanism
Logic bugs without a clear spec to verify against
Standard verification was inconclusive or escalated
User explicitly requests full verification
Follow deep-verification.md. Track each phase as a task with explicit dependencies, and execute the phases using the plugin's analysis agents.
Default
Start with standard. Standard verification has two built-in escalation checkpoints that route to deep when complexity exceeds the linear checklist.
Batch Triage
When verifying multiple bugs at once:
Run Step 0 for all bugs first — restating each claim often collapses obvious false positives immediately
Route each bug independently (some may be standard, others deep)
Process all standard-routed bugs first, then deep-routed bugs
After all bugs are verified, check for exploit chains — findings that individually failed gate review may combine to form a viable attack
Final Summary
After processing ALL suspected bugs, provide:
Counts: X TRUE POSITIVES, Y FALSE POSITIVES
TRUE POSITIVE list: Each with brief vulnerability description
FALSE POSITIVE list: Each with brief reason for rejection
References
Standard Verification — Linear single-pass checklist for straightforward bugs
Deep Verification — Full task-based orchestration for complex bugs
Gate Reviews — Six mandatory gates and verdict format
Bug-Class Verification — Class-specific verification requirements for memory corruption, logic bugs, race conditions, integer issues, crypto, injection, info disclosure, DoS, and deserialization
False Positive Patterns — 13-item checklist and red flags for common false positive patterns
Evidence Templates — Documentation templates for data flow, mathematical proofs, attacker control, and devil's advocate reviews
Files
- ATTRIBUTION.mdAdded by AutoGineer918 B
- LICENSELicence file added by AutoGineer19.7 KiB
- SKILL.md6.6 KiB
- agents/openai.yaml245 B
- assets/trail-of-bits-mark.svg3.0 KiB
- references/bug-class-verification.md7.9 KiB
- references/deep-verification.md8.6 KiB
- references/evidence-templates.md2.9 KiB
- references/false-positive-patterns.md6.9 KiB
- references/gate-reviews.md1.9 KiB
- references/standard-verification.md4.4 KiB
references/deep-verification.md
Deep Verification
Full task-based verification for complex bugs. Use when routing from SKILL.md selects the deep path, or when standard verification escalates.
If Escalated from Standard
When a bug escalates from standard verification:
Review all evidence gathered during the standard pass — do not repeat completed work
Identify which phases below are already satisfied by existing evidence
Create tasks only for remaining phases, starting from where standard left off
Preserve and reference all prior findings in new task descriptions
Verification Task List
For each bug (Bug #N), create one task per phase with the dependency structure below. Wire the dependencies so each phase stays blocked until every task it depends on is complete.
── Phase 1: Data Flow Analysis ──────────────────────────────────
"BUG #N - Phase 1.1: Map trust boundaries and trace data flow"
Then in parallel (each blocked by 1.1):
"BUG #N - Phase 1.2: Research API contracts and safety guarantees"
"BUG #N - Phase 1.3: Environment protection analysis"
"BUG #N - Phase 1.4: Cross-reference analysis"
── Phase 2: Exploitability Verification (blocked by Phase 1) ───
In parallel:
"BUG #N - Phase 2.1: Confirm attacker controls input data"
"BUG #N - Phase 2.2: Mathematical bounds verification"
"BUG #N - Phase 2.3: Race condition feasibility proof"
Then (blocked by 2.1, 2.2, 2.3):
"BUG #N - Phase 2.4: Adversarial analysis"
── Phase 3: Impact Assessment (blocked by Phase 2) ─────────────
In parallel:
"BUG #N - Phase 3.1: Demonstrate real security impact"
"BUG #N - Phase 3.2: Primary control vs defense-in-depth"
── Phase 4: PoC Creation (blocked by Phase 3) ──────────────────
"BUG #N - Phase 4.1: Create pseudocode PoC with data flow diagrams"
Then in parallel (each blocked by 4.1):
"BUG #N - Phase 4.2: Create executable PoC if feasible"
"BUG #N - Phase 4.3: Create unit test PoC if feasible"
"BUG #N - Phase 4.4: Negative PoC — show exploit preconditions"
Then (blocked by 4.2, 4.3, 4.4):
"BUG #N - Phase 4.5: Verify PoC demonstrates the vulnerability"
── Phase 5: Devil's Advocate (blocked by Phase 4) ──────────────
"BUG #N - Phase 5.1: Devil's advocate review"
── Gate Review (blocked by Phase 5) ────────────────────────────
"BUG #N - GATE REVIEW: Evaluate all six gates before verdict"Execution Rules
Mark each task as in-progress when starting, completed only with concrete evidence
Parallel sub-phases: Launch independent sub-phases concurrently using the plugin's agents. Collect all results before proceeding to the next dependency gate.
Dependency gates: Never start a phase until all tasks it depends on are completed.
Apply all 13 checklist items from false-positive-patterns.md to each bug
Agents
Delegate these agents to their respective phases. Pass the bug description and any prior phase results as context.
| Agent | Phases | Purpose |
|---|---|---|
data-flow-analyzer | 1.1–1.4 | Trace data flow, map trust boundaries, check API contracts and environment protections |
exploitability-verifier | 2.1–2.4 | Prove attacker control, mathematical bounds, race condition feasibility |
poc-builder | 4.1–4.5 | Create pseudocode, executable, unit test, and negative PoCs |
Phases 3 (Impact Assessment), 5 (Devil's Advocate), and the Gate Review are handled directly — they require synthesizing results across phases and should not be delegated.
Phase Requirements
The task list above names every phase. Below are the key pitfalls and decision criteria for each — focus on what you might get wrong.
Phase 1: Data Flow Analysis
1.1: Map trust boundaries (internal/trusted vs external/untrusted) and trace data from source to alleged vulnerability. Apply class-specific verification from bug-class-verification.md. Key pitfall: Analyzing code in isolation without tracing the full validation chain. Conditional logic upstream may make the vulnerable code mathematically unreachable (see false-positive-patterns.md items 1 and 1a).
1.2: Check API contracts before claiming overflows — many APIs have built-in bounds protection that prevents the alleged issue regardless of inputs.
1.3: Before concluding vulnerability, verify that no compiler, runtime, OS, or framework protections prevent exploitation. Note: mitigations like ASLR and stack canaries raise the exploitation bar but do not eliminate the vulnerability itself. Distinguish "prevents exploitation entirely" (e.g., Rust's safe type system) from "makes exploitation harder" (e.g., ASLR).
1.4: Check if similar code patterns exist elsewhere and are handled safely. Review test coverage, code review history, and design documentation for this area.
Phase 2: Exploitability Verification
2.1: Prove attacker controls the data reaching the vulnerability. Key pitfall: Assuming network/external data reaches the operation without tracing the actual path — internal storage set by trusted components is not attacker-controlled.
2.2: Create explicit algebraic proofs for bounds-related issues. Use the template in evidence-templates.md. Verify: IF validation_check_passes THEN bounds_guarantee_holds.
2.3: For race conditions, prove concurrent access is actually possible. Key pitfall: Assuming race conditions in single-threaded initialization or synchronized contexts.
2.4: Assess full attack surface: input control, validation bypass paths, timing dependencies, and state manipulation.
Phase 3: Impact Assessment
3.1: Distinguish real security impact (RCE, privesc, info disclosure) from operational robustness issues.
3.2: Distinguish primary security controls from defense-in-depth. Failure of a defense-in-depth measure is not a vulnerability if primary protections remain intact.
Phase 4: PoC Creation
Always create a pseudocode PoC. Additionally, create executable and/or unit test PoCs when feasible:
Pseudocode with data flow diagrams showing the attack path (always)
Executable PoC in the target language demonstrating the vulnerability (if feasible)
Unit test PoC exercising the vulnerable code path with crafted inputs (if feasible)
See evidence-templates.md for PoC templates.
Negative PoC (Phase 4.4): Demonstrate the gap between normal operation and the exploit path — what preconditions must hold for the vulnerability to trigger, and why they don't hold under normal conditions.
Phase 5: Devil's Advocate Review
Before final verdict, systematically challenge the vulnerability claim. Assume you are biased toward finding bugs and rating them as critical — actively work against that bias.
Challenges arguing AGAINST the vulnerability:
What non-vulnerability explanations exist for this code pattern?
How would the original developers justify this implementation?
What crucial system architecture context might be missing?
Am I seeing a vulnerability because the pattern "looks dangerous" rather than because it actually is?
Even if validation looks insufficient, does it actually prevent the claimed condition?
Am I incorrectly assuming attacker control over trusted data?
Have I rigorously proven the mathematical condition for vulnerability can occur?
Beyond theoretical possibility, is this practically exploitable?
Am I confusing defense-in-depth failure with a primary security vulnerability?
What compiler/runtime/OS protections might prevent exploitation?
Am I hallucinating this vulnerability? LLMs are biased toward seeing bugs everywhere and rating every finding as critical — is this actually a real, exploitable issue or am I pattern-matching on scary-looking code?
Challenges arguing FOR the vulnerability (false-negative protection):
Am I dismissing a real vulnerability because the exploit seems complex or unlikely?
Am I inventing mitigations or validation logic that I haven't verified in the actual source code? Re-read the code after reaching a conclusion.
See evidence-templates.md for the devil's advocate documentation template.
Gate Review
Apply the six gates from gate-reviews.md to reach a verdict.
Requirements and compatibility
Works with: Claude Code, Claude apps, OpenAI Codex, GitHub Copilot, Cursor, Gemini CLI, Kiro, OpenCode, Goose, Amp, Roo Code, JetBrains Junie, Factory Droid, Devin (ex-Windsurf), Mistral Vibe, OpenHands, Letta
“Works with” means the tool reads the open Agent Skills format; a skill written for one tool may use features others lack.
Needs: Declares allowed tools
Security
No issues found by our static scan· No issues found
A static scan is not a guarantee. Skills are third-party content: review a skill before you let an agent use it.
Attribution and licence
- Published by
- Trail of Bits
- Publisher
- Trail of Bits (Official)
- Source
- trailofbits/skills · plugins/fp-check/skills/fp-check · commit e4dd0bb
- Licence
- Creative Commons Attribution-ShareAlike 4.0 International (redistribution allowed)
- LICENSE (added by AutoGineer from LICENSE)
- ATTRIBUTION.md (written by AutoGineer)
- Synced
AutoGineer is not affiliated with or endorsed by Trail of Bits or Trail of Bits.
Download ZIP
The ZIP holds the skill’s files, its licence files and an ATTRIBUTION.md. Download fp-check.zip