Changed ubuntu-keyring paths breaks upgrade to focal.
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Landscape Client |
Fix Committed
|
High
|
Simon Poirier | ||
landscape-client (Ubuntu) |
Fix Released
|
Medium
|
Unassigned | ||
Bionic |
Fix Released
|
Medium
|
Unassigned | ||
Focal |
Fix Released
|
Medium
|
Unassigned | ||
Groovy |
Won't Fix
|
Medium
|
Unassigned | ||
Hirsute |
Won't Fix
|
Medium
|
Unassigned | ||
Impish |
Fix Released
|
Medium
|
Unassigned | ||
Jammy |
Fix Released
|
Medium
|
Unassigned |
Bug Description
[Impact]
* When launching an Ubuntu release-upgrade through landscape-client, the
upgrade-tool fails GPG verification due to trusted apt key having changed
location as of 18.04 LTS.
* The proposed patch extends gpg lookup path to include all
/etc/
when verifying the upgrade-tool signature.
[Test Case]
* Install and register the landscape-client against a landscape-server
on a series supporting an upgrade.
* Wait for it to sync up packages.
* On the computer packages page, there is a link at the bottom to request a
release upgrade of that machine, if a supported version is available.
* The upgrade fails and /var/log/
a failed gpg verification.
[Where problems could occur]
* One thing which has been considered in this fix is how someone could have
worked around the issue by re-creating the old key path. The fix covers
such a case by still reading the deprecated trusted.gpg file.
* Although some care has been taken to only load valid gpg keys from apt
trusted keychain, there could be unforeseen scenarios where invalid data
gets read from the keychain. In such a case, the strict nature of gpg would
reject the signature verification, thus being no worse than without the fix.
* The affected callsite is used for verifying the release-upgrader code prior
to running it. One bad thing which we could imagine with this code path is
falsely accepting an invalid file signature, which may create a security
issue. This would likely take shape of injecting a gpg key, without
having root access, in the search path.
[Other Info]
* There is no way to directly verify this issue on 20.10 Groovy and later
(without faking a release) due to the lack of upgrade path to a supported
LTS. The ubuntu-keyring package having the same file layout, the same
validation failure is however to be expected if left unpatched.
[Original description]
Since bionic, ubuntu-keyring removed `/etc/apt/
This breaks signature verification for the upgrade-tool.
Trying to release-upgrade through landscape yields a failure on signature check:
2020-11-10 15:47:51,019 WARNING [MainThread] Invalid signature for upgrade-tool tarball: /usr/bin/gpg failed (out='', err='gpg: keybox '/etc/apt/
gpg: Signature made Fri Oct 16 03:28:09 2020 UTC
gpg: using RSA key 3B4FE6ACC0B21F32
gpg: Can't check signature: No public key
Related branches
- Andreas Hasenack (community): Approve
- git-ubuntu import: Pending requested
-
Diff: 171 lines (+149/-0)3 files modifieddebian/changelog (+7/-0)
debian/patches/0006-lp1903776-release-upgrade.patch (+141/-0)
debian/patches/series (+1/-0)
- Andreas Hasenack (community): Approve
- git-ubuntu import: Pending requested
-
Diff: 171 lines (+149/-0)3 files modifieddebian/changelog (+7/-0)
debian/patches/lp1903776-release-upgrade.patch (+141/-0)
debian/patches/series (+1/-0)
- Andreas Hasenack (community): Approve
- Canonical Server: Pending requested
-
Diff: 171 lines (+149/-0)3 files modifieddebian/changelog (+7/-0)
debian/patches/1903776-release-upgrade.patch (+141/-0)
debian/patches/series (+1/-0)
- Andreas Hasenack (community): Approve
- Canonical Server: Pending requested
-
Diff: 171 lines (+149/-0)3 files modifieddebian/changelog (+7/-0)
debian/patches/0004-lp1903776-release-upgrade.patch (+141/-0)
debian/patches/series (+1/-0)
Changed in landscape-client: | |
status: | New → Confirmed |
importance: | Undecided → Critical |
importance: | Critical → High |
assignee: | nobody → Simon Poirier (simpoir) |
Changed in landscape-client: | |
status: | Confirmed → In Progress |
Changed in landscape-client: | |
status: | In Progress → Fix Committed |
Changed in landscape-client (Ubuntu): | |
status: | New → Confirmed |
Changed in landscape-client (Ubuntu Hirsute): | |
status: | Confirmed → New |
description: | updated |
Changed in landscape-client (Ubuntu Hirsute): | |
assignee: | nobody → Simon Poirier (simpoir) |
Changed in landscape-client (Ubuntu Groovy): | |
assignee: | nobody → Simon Poirier (simpoir) |
Changed in landscape-client (Ubuntu Focal): | |
assignee: | nobody → Simon Poirier (simpoir) |
Changed in landscape-client (Ubuntu Bionic): | |
assignee: | nobody → Simon Poirier (simpoir) |
Changed in landscape-client (Ubuntu Hirsute): | |
status: | New → In Progress |
Changed in landscape-client (Ubuntu Groovy): | |
status: | New → In Progress |
Changed in landscape-client (Ubuntu Focal): | |
status: | New → In Progress |
Changed in landscape-client (Ubuntu Bionic): | |
status: | New → In Progress |
tags: | added: dist-upgrade |
Changed in landscape-client (Ubuntu Bionic): | |
importance: | Undecided → Medium |
Changed in landscape-client (Ubuntu Focal): | |
importance: | Undecided → Medium |
Changed in landscape-client (Ubuntu Groovy): | |
importance: | Undecided → Medium |
Changed in landscape-client (Ubuntu Hirsute): | |
importance: | Undecided → Medium |
Changed in landscape-client (Ubuntu Impish): | |
importance: | Undecided → Medium |
Changed in landscape-client (Ubuntu Jammy): | |
importance: | Undecided → Medium |
Changed in landscape-client (Ubuntu Bionic): | |
assignee: | Simon Poirier (simpoir) → nobody |
Changed in landscape-client (Ubuntu Focal): | |
assignee: | Simon Poirier (simpoir) → nobody |
Changed in landscape-client (Ubuntu Groovy): | |
assignee: | Simon Poirier (simpoir) → nobody |
Changed in landscape-client (Ubuntu Hirsute): | |
assignee: | Simon Poirier (simpoir) → nobody |
Changed in landscape-client (Ubuntu Impish): | |
assignee: | nobody → Simon Poirier (simpoir) |
Changed in landscape-client (Ubuntu Jammy): | |
assignee: | Simon Poirier (simpoir) → nobody |
Changed in landscape-client (Ubuntu Impish): | |
assignee: | Simon Poirier (simpoir) → nobody |
https:/ /github. com/CanonicalLt d/landscape- client/ pull/89