GHSA-85jx-fm8m-x8c6: zot’s create-only policy allows overwrite attempts of existing latest tag (update permission not required)
zot’s dist-spec authorization middleware infers the required action for `PUT /v2/{name}/manifests/{reference}` as `create` by default, and only switches to `update` when the tag already exists and `reference != "latest"`.as a result, when `latest` already exists, a user who is allowed to `create` (but not allowed to `update`) can still pass the authorization check for an overwrite attempt of `latest`.affected componentfile: `pkg/api/authz.go` (`DistSpecAuthzHandler`)condition: `slices.Contains(tags, reference) && reference != "latest"` (line 352 at the pinned commit)severityHIGH
category: CWE-863 (incorrect authorization)note: impact depends on how a deployment uses `latest` (for example, if `latest` is treated as a protected or “push-once” tag), and on how access control is provisioned (users with `create` but without `update`). the attached poc demonstrates a real overwrite of `latest` (tag digest changes) under a create-only policy.steps to reproduceconfigure access control so user `attacker` has `create` but not `update` on a repository.ensure the repository has an existing tag named `latest`.attempt to push a new manifest to `/v2/acme/app/manifests/latest` (example repository name).observe that the authorization check is evaluated as `create` (not `update`) for `latest`, so the request passes authorization even though the tag already exists.the attached poc demonstrates this deterministically with `canonical.log` and `control.log` markers.expected vs actualexpected: overwriting an existing tag should require `update` permission, including `latest` (or `latest` should be explicitly documented as exempt).actual: when `reference=="latest"` and the tag exists, the middleware keeps the action as `create` instead of switching to `update`.security impactthis can break least-privilege expectations in deployments that rely on the `create` vs `update` split to prevent tag overwrites (for example, “push-once” policies). if `latest` is used as a high-trust tag in ci/cd, this can create supply-chain risk because a create-only principal can overwrite an existing `latest` tag while other existing tags correctly require `update`.suggested fixremove the special-case exemption for `latest` when determining whether an existing tag requires `update` permission (treat `latest` the same as other tags), or document and enforce an explicit policy rule for `latest`.notes / rationaleoci distribution spec does not define a standard authorization model; this report is about zot’s own create vs update semantics and the observable behavior in `DistSpecAuthzHandler`.zot documentation describes immutable tags as being enforceable via authorization policies (create-only “push once”, update disallowed). if `latest` is exempt, this control does not apply to `latest` unless documented otherwise.[addendum.md](https://github.com/user-attachments/files/24986139/addendum.md)
[poc.zip](https://github.com/user-attachments/files/24986140/poc.zip)
[PR_DESCRIPTION.md](https://github.com/user-attachments/files/24986141/PR_DESCRIPTION.md)
[RUNNABLE_POC.md](https://github.com/user-attachments/files/24986142/RUNNABLE_POC.md)
CVSS v3.1: 7.7