0
0

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?

AIコーディングで発生しやすいセキュリティ脆弱性のパターンと対策コード例

0
Posted at

読者が抱える課題

GitHub CopilotやChatGPTなどのコード生成AIの導入が進む一方で、AIが生成したコードをそのまま採用することで、SQLインジェクションやSSRF、ハードコードされた認証情報などの脆弱性がコードベースに混入するリスクが高まっています。AIはコンテキストに基づいた「動作するコード」を生成しますが、「安全なコード」を常に保証するわけではありません。

この記事で分かること

  • AIコーディングにおいて脆弱性が作り込まれやすい代表的なパターン
  • 安全なコード(良い例)と脆弱なコード(悪い例)の比較実装
  • AI生成コードを安全に利用するためのレビューチェックリストと導入運用フロー

対象読者・前提条件

  • AIコードアシスタントを日常業務で利用しているITエンジニア・開発者
  • Webアプリケーションのコードレビューを担当するリーダー・レビュアー
  • 前提知識:Pythonおよび一般的なWebセキュリティ(OWASP Top 10程度)の基礎知識

AIコーディングで生成されやすい脆弱性パターンと対策

パターン1:SQLインジェクション(文字列連結によるクエリ構築)

AIは単純なクエリ作成を指示された際、プレースホルダーを使わずにF文字列や文字列連結を用いたコードを提案することがあります。

悪い例(脆弱なコード)

# AIが生成しがちな不安全なコード例
def get_user_bad(cursor, user_id):
    # 入力値が直接SQL文に連結されているためSQLインジェクションが可能
    query = f"SELECT id, username, email FROM users WHERE id = '{user_id}'"
    cursor.execute(query)
    return cursor.fetchone()

良い例(安全なコード)

# パラメータ化クエリを使用した安全なコード例
def get_user_good(cursor, user_id):
    # プレースホルダーを使用してSQLとデータを分離する
    query = "SELECT id, username, email FROM users WHERE id = %s"
    cursor.execute(query, (user_id,))
    return cursor.fetchone()

パターン2:SSRF(Server-Side Request Forgery)

外部API連携などのコードを生成させる際、ユーザー入力のURLに対するバリデーションが省略されたコードが生成されるケースがあります。

悪い例(脆弱なコード)

import requests

# AIが生成しがちな不安全なコード例
def fetch_user_avatar_bad(image_url):
    # 内部ネットワーク(例: http://169.254.169.254/)へのアクセス制限がない
    response = requests.get(image_url)
    return response.content

良い例(安全なコード)

from urllib.parse import urlparse
import ipaddress
import requests

def is_safe_url(url):
    allowed_schemes = {'http', 'https'}
    parsed = urlparse(url)
    if parsed.scheme not in allowed_schemes:
        return False
    try:
        # ドメイン解決やプライベートIPブロックの検証を実装する(独自バリデーション例)
        ip = ipaddress.ip_address(parsed.hostname)
        if ip.is_private or ip.is_loopback:
            return False
    except ValueError:
        # ホスト名がIPアドレスでない場合は必要に応じてDNSルックアップ等で検証を実施
        pass
    return True

def fetch_user_avatar_good(image_url):
    if not is_safe_url(image_url):
        raise ValueError("Unsafe URL requested")
    response = requests.get(image_url, timeout=5.0)
    return response.content

※上記は基本的な検証の例です。実際のプロダクション環境ではDNS Rebinding攻撃対策などを含めたライブラリやプロキシでの制御を検討してください。

発生パターン比較表

脆弱性の種類 AIが生成しやすい理由 主な対策
SQLインジェクション 可読性の高いF文字列などを好んで出力するため ORMの利用、パラメータ化クエリの強制
硬直化した秘密情報 プロンプトやサンプルコード内の仮の鍵をそのまま出力するため 環境変数・シークレット管理ツールの利用
SSRF / ディレクトリトラバーサル 入力値の検証ロジックを省略して最短で動くコードを作るため スキーマ検証、ホワイトリストによる入力制限
不十分な暗号化/ハッシュ 古い学習データからMD5やSHA-1などを選択することがあるため argon2idやbcryptなどの現代的なアルゴリズムを指定

レビュー用チェックリスト

AIが生成したコードをPR(プルリクエスト)に取り込む前に、以下の項目を確認してください。

  • 入力検証: 外部からの入力値に対して適切な型チェック、長さチェック、エスケープが実施されているか
  • データベース操作: 動的な文字列連結によるクエリ構築が行われていないか
  • 認証・認可: 認証情報(APIキー、パスワード)がコード内にハードコードされていないか
  • 外部通信: URLやIPアドレスを指定して外部通信を行う際、内部ネットワークへのリクエストが遮断されているか
  • エラーハンドリング: スタックトレースや内部実装の詳細がエラーレスポンスとしてクライアントに返却されていないか
  • 依存ライブラリ: 提案されたパッケージ名が存在し、タイポスクワッティング等の危険性がないか

注意点と運用上の判断基準

  1. コードの所有権と責任: AIが生成したコードであっても、リポジトリにマージされた時点で実装者の責任となります。「AIが出力したから」という理由で検証をスキップすることは避けてください。
  2. SAST(静的解析ツール)の導入: CodeQL、Bandit、SonarQubeなどのSASTツールをCI/CDパイプラインに組み込み、自動で脆弱性を検知する仕組みを構築してください。
  3. 最新仕様の確認: AIの学習データの更新時期によっては、非推奨の関数やセキュリティ上問題のある古い記述を出力する場合があります。利用するライブラリの公式ドキュメントで最新仕様を確認してください。

まとめ

AIコーディングアシスタントは開発スピードを大きく向上させますが、安全なコードを出力するとは限りません。開発者は「生成されたコードには脆弱性が含まれ得る」という前提を持ち、静的解析ツールと手動レビューを組み合わせた防御策を講じることが重要です。

0
0
0

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
0
0

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?