fix(cloud): Fix KMSKeyID check (#1359)

Fixes: aquasecurity/trivy#4628 Signed-off-by: Simar <simar@linux.com>
aquasecurity · Jun 30, 2023 · c3d4a38 · c3d4a38
1 parent bafdfd8
commit c3d4a38
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/internal/adapters/cloud/aws/rds/rds.go b/internal/adapters/cloud/aws/rds/rds.go
@@ -336,10 +336,15 @@ func (a *adapter) adaptDBSnapshots(dbSnapshots types.DBSnapshot) (*rds.Snapshots
 		DBSnapshotIdentifier: defsecTypes.String(*dbSnapshots.DBSnapshotIdentifier, metadata),
 		DBSnapshotArn:        defsecTypes.String(*dbSnapshots.DBSnapshotArn, metadata),
 		Encrypted:            defsecTypes.Bool(dbSnapshots.Encrypted, metadata),
-		KmsKeyId:             defsecTypes.String(*dbSnapshots.KmsKeyId, metadata),
+		KmsKeyId:             defsecTypes.String("", metadata),
 		SnapshotAttributes:   SnapshotAttributes,
 	}
 
+	// KMSKeyID is only set if Encryption is enabled
+	if snapshots.Encrypted.IsTrue() {
+		snapshots.KmsKeyId = defsecTypes.StringDefault(*dbSnapshots.KmsKeyId, metadata)
+	}
+
 	return snapshots, nil
 }