GHSA-x5r6-jr56-89pv: Incus has Nil Dereferences on Restore via Malformed YAML
SummaryDetailsIt was found that backup.GetInfo() trusts the inline backup/index.yaml config when present and only falls back to parsing the legacy backup/container/backup.yaml file if result.Config == nil. As a result, an archive can carry a valid inline config that passes the initial import preflight while also carrying a malformed legacy backup/container/backup.yaml file that is reparsed later from the restored file system.ParseConfigYamlFile() accepts YAML documents with no container section, and multiple downstream consumers then dereference .Container without checking for nil. Confirmed examples in the instance restore and import flow include backup.UpdateInstanceConfig() and internalImportFromBackup().An authenticated user with permission to import instance backups may be able to crash the Incus daemon with a crafted backup archive whose inline backup/index.yaml is valid but whose extracted legacy backup.yaml omits container. The crash occurs in the restore path after archive extraction has begun.The flow is as follows:
A crafted backup archive contains a valid backup/index.yaml file together with a malformed backup/container/backup.yaml file that omits the container section.
backup.GetInfo() parses backup/index.yaml successfully, so bInfo.Config is populated from the inline config.
Because result.Config != nil, GetInfo() does not fall back to backup/container/backup.yaml.
instances_post.go then builds the request from bInfo.Config.Container, which succeeds because the inline config is valid.
Later, storage unpack extracts backup/container/backup.yaml into the instance volume as <mountPath>/backup.yaml.
backend.go then calls backup.UpdateInstanceConfig(..., mountPath), which reparses <mountPath>/backup.yaml through ParseConfigYamlFile().
Because ParseConfigYamlFile() accepts YAML with no container section, backup.Container == nil, and later access to backup.Container.Devices or backup.Container.ExpandedDevices can trigger a nil-pointer dereference.Affected Files:https://github.com/lxc/incus/blob/v6.22.0/internal/server/backup/backup_info.go#L87https://github.com/lxc/incus/blob/v6.22.0/internal/server/backup/backup_info.go#L115https://github.com/lxc/incus/blob/v6.22.0/internal/server/backup/backup_config_utils.go#L85https://github.com/lxc/incus/blob/v6.22.0/internal/server/backup/backup_config_utils.go#L159https://github.com/lxc/incus/blob/v6.22.0/internal/server/storage/backend.go#L809https://github.com/lxc/incus/blob/v6.22.0/cmd/incusd/api_internal.go#L749The initial backup-metadata parser prefers inline backup/index.yaml content:Affected Code:The legacy backup/container/backup.yaml file is only parsed if the inline config is absent:Affected Code:ParseConfigYamlFile() accepts an empty YAML document, or one with no container section, without validation:Affected Code:UpdateInstanceConfig() conditionally uses backup.Container at first but later dereferences it unconditionally:Affected Code:Another confirmed sink is present in internalImportFromBackup():Affected Code:This was confirmed as follows:Command:Output:It is recommended to validate the parsed legacy backup.yaml structure before any dereference and to fail with a standard error if Container is missing:Proposed Fix:That validation should be added at minimum in: backup.UpdateInstanceConfig() and internalImportFromBackup() before any backupConf.Container.* accessMore broadly, it is recommended to centralize backup-config validation so that both inline backup/index.yaml and extracted legacy backup.yaml files are checked against the same structural requirements before any restore or import consumer uses them.A patch is available at https://github.com/lxc/incus/releases/tag/v7.0.0.CreditThis issue was discovered and reported by the team at 7asecurity (https://7asecurity.com/)
CVSS v3.1: 6.5