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/bug-class-verification.md
Bug-Class-Specific Verification
Different bug classes require different verification approaches. After classifying the bug in Step 0, apply the class-specific requirements below in addition to the generic verification phases.
Jump to the relevant class:
Memory Corruption — overflows, UAF, double-free, type confusion
Logic Bugs — auth bypass, access control, state transitions, privilege escalation
Race Conditions — TOCTOU, data races, signal handling
Integer Issues — overflow, underflow, truncation, signedness
Crypto Weaknesses — weak algorithms, nonce reuse, timing channels
Injection — SQL, XSS, command, template, path traversal
Information Disclosure — uninitialized memory, leaks, side channels
Denial of Service — algorithmic complexity, resource exhaustion, crashes
Deserialization — object injection, gadget chains
Memory Corruption
Buffer overflow, heap overflow, stack overflow, out-of-bounds read/write, use-after-free, double-free, type confusion.
Language safety check first: Memory corruption in safe Rust, Go (without unsafe.Pointer/cgo), or managed languages (Java, C#, Python) is almost always a false positive — the type system or runtime prevents it. Verify whether the code is in an unsafe block (Rust), uses cgo/unsafe.Pointer (Go), or calls native code via JNI/P/Invoke. If the code is entirely in the safe subset, reject the memory corruption claim unless it involves a compiler bug or soundness hole.
Verify:
What exactly gets corrupted? (which object, field, or memory region)
What is the corruption size and offset? Can the attacker control them?
Is the corruption a useful exploitation primitive (arbitrary read/write, vtable overwrite, function pointer overwrite) or just a crash?
What allocator is in use (glibc, tcmalloc, jemalloc, Windows heap)? Does it have hardening that blocks exploitation?
For UAF: trace the object lifetime — what frees it, what reuses the memory, can the attacker control the replacement object?
For type confusion: prove the type mismatch exists and that misinterpretation of the data leads to a useful primitive.
Logic Bugs
Authentication bypass, access control errors, incorrect state transitions, confused deputy, privilege escalation through API misuse.
Verify:
Check against the specification, RFC, or design docs — not just the code. Does the implementation match the intended behavior?
Map all state transitions. Can the system reach a state the developer didn't anticipate?
Identify implicit assumptions that are never enforced in code.
For auth bugs: verify ALL authentication/authorization paths, not just the one that appears broken. Is there a secondary check that catches it?
Logic bugs pass every bounds check and mathematical proof — don't let clean static analysis convince you it's a false positive.
Race Conditions
TOCTOU, data races, signal handling races, concurrent state modification.
Verify:
What is the actual race window? Is it nanoseconds or seconds?
Can the attacker widen the window (e.g., by stalling a thread with a slow NFS mount, large allocation, or CPU contention)?
Verify the threading model: what threads/processes can actually access this data concurrently?
Check all synchronization primitives in use — mutexes, atomics, RCU, lock-free structures.
For TOCTOU on filesystem: can the attacker control the path between check and use (symlink races)?
Integer Issues
Overflow, underflow, truncation, signedness errors, wraparound.
Verify:
What are the exact integer types and their ranges at every point in the computation?
Is the overflow signed (undefined behavior in C/C++ — compiler may exploit this) or unsigned (defined wraparound)?
Trace the integer through all casts, conversions, and promotions. Where does truncation or sign extension occur?
After the integer issue occurs, is the resulting value actually used in a dangerous way (allocation size, array index, loop bound)?
Check if compiler warnings (
-Wconversion,-Wsign-compare) flag this.
Crypto Weaknesses
Weak algorithms, bad parameters, nonce reuse, padding oracle, insufficient randomness, timing side channels.
Verify:
Check parameter choices against current standards (NIST, IETF) and known attacks. "AES-128" is fine; "DES" is not.
Verify randomness sources. Is the PRNG cryptographically secure? Is it properly seeded?
For nonce reuse: prove the same nonce can actually be used twice in practice, not just theoretically.
For timing side channels: is the code actually reachable by an attacker who can measure timing? Network jitter may make remote timing attacks impractical.
Compare the implementation against a reference implementation or test vectors from the spec.
Injection
SQL injection, XSS, command injection, server-side template injection, path traversal, LDAP injection.
Verify:
Trace attacker input from entry point to the sink (query, command, template, filesystem path). Is there any sanitization or escaping along the way?
Check if the framework provides automatic escaping (e.g., parameterized queries, template auto-escaping). If so, is it actually enabled and not bypassed?
For XSS: what context does the input land in (HTML body, attribute, JavaScript, URL)? Each requires different escaping.
For path traversal: is the path canonicalized before the access check? Can
../or null bytes bypass validation?Test actual payload delivery through all intermediate processing — encoding, decoding, and transformation steps may neutralize or enable the payload.
Information Disclosure
Uninitialized memory reads, error message leaks, timing side channels, padding oracles.
Verify:
What specific data leaks? Not all leaks are equal — a stack leak revealing ASLR base or canary is critical; one revealing a static string is worthless.
Is the leaked data actually useful to an attacker for further exploitation (ASLR bypass, session tokens, crypto keys)?
For uninitialized memory: prove the memory is actually uninitialized at the point of read, not just potentially uninitialized on some code path.
For timing side channels: can the attacker make enough measurements with sufficient precision? What's the noise level?
For error messages: does the error path actually reach the attacker, or is it logged server-side only?
Denial of Service
Algorithmic complexity, resource exhaustion, crash bugs, infinite loops, memory bombs.
Verify:
What is the resource consumption ratio? Attacker sends X bytes, server consumes Y resources. Is the amplification meaningful?
Can the resource be reclaimed (connection closes, memory freed) or is it permanent exhaustion?
For algorithmic complexity: what is the actual worst-case input? Prove it triggers worst-case behavior, don't just claim O(n²).
For crash bugs: is the crash reliably triggerable, or does it depend on specific heap/stack layout?
Does the service restart automatically? A crash that causes a 100ms restart is different from one that requires manual intervention.
Deserialization
Unsafe deserialization, object injection, gadget chain exploitation.
Verify:
Does the attacker actually control the serialized data that reaches the deserialization call?
Does a usable gadget chain exist in the classpath/import graph? Without a gadget chain, unsafe deserialization is a design smell, not an exploitable bug.
What deserialization library and version is in use? Are there known gadget chains for it?
Are there type restrictions, allowlists, or look-ahead deserialization filters that block dangerous classes?
For language-specific: Java
ObjectInputStream, Pythonpickle, PHPunserialize, .NETBinaryFormattereach have different exploitation characteristics.
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