MalDet · Static Security Report
4 findings likely need attention. Most notable: Variable `instance` is likely modified and later used on error. In some cases this coul... (100% confidence).
Generated from finding counts. A plain-English summary was not available for this scan.
MalDet scores two different questions separately, because they call for different responses: code can be badly written without being hostile, and hostile code is not always badly written.
| Detector | Looks for | High | Medium | Low | Total |
|---|---|---|---|---|---|
| semgrep | Insecure coding patterns across many languages | 0 | 5 | 0 | 5 |
4 of 5 findings were rated more likely genuine than noise by the classifier.
Identical findings are grouped, so a single rule firing many times takes one row. Ordered worst first. “Seen” is how many times that finding occurred.
| Severity | Detector | Finding | Seen | Example location |
|---|---|---|---|---|
| medium | semgrep | Variable `instance` is likely modified and later used on error. In some cases this could result in panics due to a nil dereference | 1 | /internal/instance/resource_instance_file.go:255 |
| medium | semgrep | GitHub Actions step uses a mutable tag or branch reference. Tags and branch names can be silently repointed by the action owner, enabling supply-chain attacks — as seen in the trivy-action and kics-github-action compromises. Pin the reference to a full 40-character commit SHA instead, e.g. `uses: ac | 3 | /.github/actions/cluster/action.yml:40 |
| medium | semgrep | Do not use `math/rand`. Use `crypto/rand` instead. | 1 | /internal/acctest/testutils.go:5 |