terraform.gcp.security.gcp-kms-prevent-destroy.gcp-kms-prevent-destroy

profile photo of semgrepsemgrep
Author
unknown
Download Count*
LGPL Commons Clause
License

Ensure KMS keys are protected from deletion

Run Locally

Run in CI

Semgrep CI docs

Defintion

Edit RuleView Source
Open in Playground 
rules:
  - id: gcp-kms-prevent-destroy
    patterns:
      - pattern: resource
      - pattern-inside: |
          resource "google_kms_crypto_key" "..." {
            ...
          }
      - pattern-not-inside: |
          resource "google_kms_crypto_key" "..." {
            ...
            lifecycle {
              prevent_destroy = true
            }
            ...
          }
    message: Ensure KMS keys are protected from deletion
    metadata:
      owasp:
        - A05:2017 - Broken Access Control
        - A01:2021 - Broken Access Control
      cwe:
        - "CWE-284: Improper Access Control"
      category: security
      technology:
        - terraform
        - gcp
      references:
        - https://owasp.org/Top10/A01_2021-Broken_Access_Control
      subcategory:
        - audit
      likelihood: LOW
      impact: LOW
      confidence: LOW
      license: Commons Clause License Condition v1.0[LGPL-2.1-only]
      vulnerability_class:
        - Improper Authorization
    languages:
      - hcl
    severity: WARNING

Examples

gcp-kms-prevent-destroy.tf

# ok: gcp-kms-prevent-destroy
resource "google_kms_crypto_key" "pass" {
  name            = "crypto-key-example"
  key_ring        = google_kms_key_ring.keyring.id
  rotation_period = "15552000s"

  lifecycle {
    prevent_destroy = true
  }
}

# fail
# ruleid: gcp-kms-prevent-destroy
resource "google_kms_crypto_key" "fail" {
  name            = "crypto-key-example"
  key_ring        = google_kms_key_ring.keyring.id
  rotation_period = "15552000s"

  lifecycle {
    prevent_destroy = false
  }
}

# fail
# ruleid: gcp-kms-prevent-destroy
resource "google_kms_crypto_key" "fail2" {
  name            = "crypto-key-example"
  key_ring        = google_kms_key_ring.keyring.id
  rotation_period = "15552000s"
}