Generated banner for Fp Check
Generated logo for Fp Check

Fp Check

Trail of BitsOfficialPublished by Trail of Bits

  • Skill
  • Review and audit
  • Security
  • CC-BY-SA-4.0
  • No issues found

From the author

Systematically verifies suspected security bugs to eliminate false positives, producing a TRUE POSITIVE or FALSE POSITIVE verdict with documented evidence for each. Use when asked whether a specific finding is real, exploitable, or a false positive, or to verify or validate a suspected vulnerability — not for hunting or discovering new bugs.
— the description by Trail of Bits

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

  1. Download the skill as a ZIP:

    https://autogineer.com/agent-skills/trailofbits/fp-check/download
  2. Unzip 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.

RationalizationWhy It's WrongRequired Action
"Rapid analysis of remaining bugs"Every bug gets full verificationReturn to task list, verify next bug through all phases
"This pattern looks dangerous, so it's a vulnerability"Pattern recognition is not analysisComplete data flow tracing before any conclusion
"Skipping full verification for efficiency"No partial analysis allowedExecute all steps per the chosen verification path
"The code looks unsafe, reporting without tracing data flow"Unsafe-looking code may have upstream validationTrace the complete path from source to sink
"Similar code was vulnerable elsewhere"Each context has different validation, callers, and protectionsVerify this specific instance independently
"This is clearly critical"LLMs are biased toward seeing bugs and overrating severityComplete 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() when content_length exceeds 4096")

  • What is the alleged root cause? (e.g., "missing bounds check before memcpy at 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:

  1. Run Step 0 for all bugs first — restating each claim often collapses obvious false positives immediately

  2. Route each bug independently (some may be standard, others deep)

  3. Process all standard-routed bugs first, then deep-routed bugs

  4. 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:

  1. Counts: X TRUE POSITIVES, Y FALSE POSITIVES

  2. TRUE POSITIVE list: Each with brief vulnerability description

  3. 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

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, Python pickle, PHP unserialize, .NET BinaryFormatter each 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