# Use Context Reviewer Security > OWASP Top 10 のセキュリティ レビュー。可読性 (use-context-reviewer-readability)、テスト設計 (use-context-reviewer-testability) には使わない。 - Skill: `thkt/use-context-reviewer-security-2` (Agent Skill, multi-file: 9 files) - Install (CLI): `npx skillmds@latest add thkt/use-context-reviewer-security-2` - Raw SKILL.md: https://api.skillmd.com/api/skills/thkt/use-context-reviewer-security-2/raw - Safety review: pending - Works with: Claude Code, Claude.ai, OpenAI Codex - Category: Security - Author: thkt (https://skillmd.com/u/thkt) - Updated: 2026-09-22 - Page: https://skillmd.com/skills/thkt/use-context-reviewer-security-2 --- # use-context-reviewer-security ## 検出 (OWASP Top 10) LLM01 は信頼できないコンテンツを LLM に渡すアプリを対象とする。sink はプロンプトそのものであり、データと指示の境界なく連結された信頼できないテキストや、システムプロンプトに補間された呼び出し側の値が該当する。`fetch_url` のような LLM ツールは、A10 SSRF など非 LLM の対応物と同じように制約する。sink はツールであってプロンプトではない。 | ID | カテゴリ | パターン | 修正 | | ----- | ------------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------- | ------------------------------------------------------------------------------------------------ | | LLM01 | Prompt Injection (LLM) | RAG ドキュメント・取得コンテンツ・ツール結果・role 引数など、信頼できない値や呼び出し側が制御する値が、データとしての枠付けなしにプロンプトへ到達 | 信頼できないコンテンツをデータとして区切る。呼び出し側の値は固定の列挙された指示にマッピングする | | A01 | Broken Access Control | 認証なし、IDOR、path traversal | 認証ミドルウェア、所有権チェック | | A01 | CSRF | 状態変更リクエスト (POST/PUT/PATCH/DELETE) に CSRF トークン検証なし | Double Submit Cookie | | A01 | Open Redirect (Taint) | URL パラメータ → `location.href` でバリデーションなし | ドメイン許可リストまたは相対のみ | | A02 | Cryptographic Failures | `password: 'plaintext'` | bcrypt/argon2 ハッシュ化 | | A02 | Timing Attack | トークン/署名の `===` 比較 | 定数時間比較。全バイトを XOR し最後に判定 | | A02 | Sensitive Data Exposure | localStorage/sessionStorage に保存された JWT | httpOnly cookie に置き換える | | A03 | Injection | `db.query(\`SELECT...${id}\`)` | パラメータ化クエリ、ORM | | A03 | Injection | `exec(\`ping ${host}\`)` | 入力バリデーション、ライブラリで代替 | | A03 | XSS | 静的に存在する `dangerouslySetInnerHTML` | デフォルト エスケープ、DOMPurify | | A03 | XSS (Taint) | サニタイザなしの `dangerouslySetInnerHTML={{ __html }}` | 境界で DOMPurify.sanitize() を呼ぶ | | A03 | XSS (Taint) | 関数引数 → `innerHTML` でサニタイズなし | 関数境界でサニタイズ | | A03 | XSS (Taint) | ユーザー制御 URL を持つ `` | https/http のみのプロトコル許可リスト | | A04 | Insecure Design | origin チェックなしの `postMessage` ハンドラ | `event.origin` の厳密比較 | | A05 | Security Misconfiguration | `cors({ origin: '*' })` | 明示的な origin 許可リスト | | A05 | Security Misconfiguration | オプションなしの `cookie: {}` | secure, httpOnly, sameSite: 'strict' | | A05 | Security Misconfiguration | エラー応答の `err.stack` で `NODE_ENV` ガードなし | 本番では汎用メッセージ、内部だけにログ | | A07 | Authentication Failures | 認証エンドポイント (login, register, password-reset) にレート制限なし | auth ルートグループに rate limiter ミドルウェア | | A08 | Prototype Pollution | `...body` スプレッドでリクエストオブジェクト構築 | 明示的なフィールド代入 | | A09 | Logging Failures | `logger.info({ password })` | センシティブ フィールドを除外 | | A10 | SSRF | `fetch(userInputUrl)` | URL バリデーション、許可リスト | ## 報告 重大度は finding-schema.md の 4 段から `low` を外した critical/high/medium を使う。security の finding に「気付いたら直す」段は無く、報告した時点で対処を要求する。独立した脆弱性はそれぞれ個別の finding として報告する。1 つのファイルに別個の問題が 2 つある場合 (例: path traversal と別の prompt injection)、一方を他方の注記に畳み込まず、両方を別々の finding として列挙する。 | シグナル | 重大度 | 必須出力 | | ---------------- | -------- | --------------------------------- | | 確実な悪用 | critical | 完全な悪用シナリオ + 具体的な修正 | | 明確な脆弱性 | high | 攻撃ベクター + 具体的な修正 | | 可能性のある問題 | medium | verification_hint + 修正提案 | | 投機的のみ | none | 報告しない | ## 参照ファイル | トピック | 範囲 | ファイル | | -------------- | ---------------------------------------- | ------------------------------------------------------ | | Basic | A01, A02, A07 | ${CLAUDE_SKILL_DIR}/references/owasp-basic.md | | Injection | A03 | ${CLAUDE_SKILL_DIR}/references/owasp-injection.md | | Advanced | A04-A06, A08-A10 | ${CLAUDE_SKILL_DIR}/references/owasp-advanced.md | | Cloud access | IAM、シークレット、ネットワーク | ${CLAUDE_SKILL_DIR}/references/cloud-access-network.md | | Cloud 運用 | ロギング、CI/CD、CDN、バックアップ | ${CLAUDE_SKILL_DIR}/references/cloud-operations.md | | Taint (markup) | HTML と属性の sink | ${CLAUDE_SKILL_DIR}/references/frontend-taint-html.md | | Taint (data) | オリジン越え、ナビゲーション、ストレージ | ${CLAUDE_SKILL_DIR}/references/frontend-taint-data.md | ## Taint レビュー ワークフロー 上表の Taint (markup) と Taint (data) にこの手順を適用する。 1. ユーザー入力・API レスポンス・URL パラメータの taint source を特定する 2. DOM 操作・ナビゲーション・ストレージの sink までデータフローを追跡する 3. すべての source-to-sink 経路でサニタイズまたはバリデーションが存在することを検証する 4. サニタイズがエラー経路や条件分岐でバイパスされないことを確認する