Only check the vendor ID if the device has one set
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OEM Priority Project |
Fix Released
|
High
|
Yuan-Chen Cheng | ||
fwupd (Ubuntu) |
Fix Released
|
High
|
Unassigned | ||
Bionic |
Fix Released
|
High
|
Robert Ancell | ||
Disco |
Won't Fix
|
High
|
Unassigned | ||
Eoan |
Fix Released
|
High
|
Robert Ancell | ||
Focal |
Fix Released
|
High
|
Unassigned |
Bug Description
[Impact]
fwupd will fail to install firmware where the firmware has a vendor-id but the device does not have one set. The vendor-id is currently optional but in about 6 months time it will be required. In this case devices that can't store vendor-ids will not be able to install firmware. There is an upstream fix to skip this check if the device doesn't have vendor IDs.
https:/
[Test Case]
There's not a specific test case for this, as no firmware currently meets this requirement. Thus the only test is that you can still successfully install firmware. Future firmware will trigger this case.
[Regression Potential]
Change is simple and unlikely to introduce a significant bug. This will cause a behaviour change, but it is done by the upstream maintainer and intentional.
Changed in fwupd (Ubuntu Focal): | |
status: | New → Fix Released |
Changed in fwupd (Ubuntu Eoan): | |
assignee: | nobody → Robert Ancell (robert-ancell) |
Changed in fwupd (Ubuntu Disco): | |
assignee: | nobody → Robert Ancell (robert-ancell) |
Changed in fwupd (Ubuntu Xenial): | |
assignee: | nobody → Robert Ancell (robert-ancell) |
Changed in fwupd (Ubuntu Disco): | |
importance: | Undecided → Medium |
Changed in fwupd (Ubuntu Bionic): | |
importance: | Undecided → High |
Changed in fwupd (Ubuntu Disco): | |
importance: | Medium → High |
Changed in fwupd (Ubuntu Eoan): | |
importance: | Undecided → High |
Changed in fwupd (Ubuntu Bionic): | |
assignee: | nobody → Robert Ancell (robert-ancell) |
Changed in fwupd (Ubuntu Focal): | |
importance: | Undecided → High |
Changed in fwupd (Ubuntu Xenial): | |
importance: | Undecided → High |
description: | updated |
description: | updated |
Changed in fwupd (Ubuntu Disco): | |
status: | New → Won't Fix |
assignee: | Robert Ancell (robert-ancell) → nobody |
Changed in oem-priority: | |
importance: | Undecided → High |
status: | New → Confirmed |
tags: | added: oem-priority |
Changed in oem-priority: | |
assignee: | nobody → Yuan-Chen Cheng (ycheng-twn) |
no longer affects: | fwupd (Ubuntu Xenial) |
tags: |
added: verification-done verification-done-bionic verification-done-eoan removed: verification-needed verification-needed-bionic verification-needed-eoan |
Changed in oem-priority: | |
status: | Fix Committed → Fix Released |
IIRC xenial is not affected by this.
For bionic it would be good to either combine this with the existing SRU in process or wait for that one to finish.