Only check the vendor ID if the device has one set

Bug #1856896 reported by Robert Ancell
8
This bug affects 1 person
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://github.com/fwupd/fwupd/commit/d5f965da2f8847e494e5c2a25a123bb8c31c7850

[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
Revision history for this message
Mario Limonciello (superm1) wrote :

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.

Revision history for this message
Robert Ancell (robert-ancell) wrote :

I've made an upload that should be able to replace the existing SRU and get them both checked. I've also checked the fwupd 0.8 source and am pretty sure this doesn't have the check there to be updated.

Changed in fwupd (Ubuntu Xenial):
status: New → Invalid
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)
Revision history for this message
Timo Aaltonen (tjaalton) wrote :

this also needs a matching fwupd-signed upload.. bionic will be covered, but eoan needs it

Revision history for this message
Łukasz Zemczak (sil2100) wrote : Please test proposed package

Hello Robert, or anyone else affected,

Accepted fwupd into bionic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/fwupd/1.2.10-1ubuntu2~ubuntu18.04.2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-bionic to verification-done-bionic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-bionic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in fwupd (Ubuntu Bionic):
status: New → Fix Committed
tags: added: verification-needed verification-needed-bionic
Revision history for this message
dann frazier (dannf) wrote :

I've uploaded a new fwupd for eoan with an additional fix (for bug 1858590), as well as an fwupd-signed that covers both.

Revision history for this message
Timo Aaltonen (tjaalton) wrote : Proposed package upload rejected

An upload of fwupd to bionic-proposed has been rejected from the upload queue for the following reason: "You need to use -v when building the source package if there's an upload in -proposed already, otherwise the other bug would not get notified of the new version or closed automatically.".

Revision history for this message
Timo Aaltonen (tjaalton) wrote : Please test proposed package

Hello Robert, or anyone else affected,

Accepted fwupd into eoan-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/fwupd/1.2.10-1ubuntu4 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-eoan to verification-done-eoan. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-eoan. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in fwupd (Ubuntu Eoan):
status: New → Fix Committed
tags: added: verification-needed-eoan
Revision history for this message
Timo Aaltonen (tjaalton) wrote :

funny that the rejection comment did end up here, though the latest .changes didn't mention this bug

Revision history for this message
Robert Ancell (robert-ancell) wrote :

Timo - the rejected version is the one Dann uploaded, right?

Revision history for this message
Timo Aaltonen (tjaalton) wrote :

Hello Robert, or anyone else affected,

Accepted fwupd into bionic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/fwupd/1.2.10-1ubuntu2~ubuntu18.04.3 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-bionic to verification-done-bionic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-bionic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Revision history for this message
Timo Aaltonen (tjaalton) wrote :

Hello Robert, or anyone else affected,

Accepted fwupd-signed into eoan-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/fwupd-signed/1.10.2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-eoan to verification-done-eoan. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-eoan. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Can we get this verified for bionic? 18.04.4 is nearing, so I'd like to move as many things from -proposed to -updates as possible.

Mathew Hodson (mhodson)
no longer affects: fwupd (Ubuntu Xenial)
Revision history for this message
Yuan-Chen Cheng (ycheng-twn) wrote :

since there is no test case in this bug, shall we mark verification-done?
(ps: already verification-done for LP: #1820768)

Changed in oem-priority:
status: Confirmed → Fix Committed
Timo Aaltonen (tjaalton)
tags: added: verification-done verification-done-bionic verification-done-eoan
removed: verification-needed verification-needed-bionic verification-needed-eoan
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package fwupd - 1.2.10-1ubuntu4

---------------
fwupd (1.2.10-1ubuntu4) eoan; urgency=medium

  [ Robert Ancell ]
  * d/p/0001-trivial-libfwupd-skip-tests-if-machine-id-is-empty-t.patch:
    - Only check the vendor ID if the device has one set (LP: #1856896)

  [ dann frazier ]
  * d/p/0001-efi-use-a-wildcard-section-copy-for-final-EFI-genera.patch
    - This avoids a crash on exec of fwupdaa64.efi (LP: #1858590)

 -- dann frazier <email address hidden> Mon, 13 Jan 2020 12:17:07 -0700

Changed in fwupd (Ubuntu Eoan):
status: Fix Committed → Fix Released
Revision history for this message
Łukasz Zemczak (sil2100) wrote : Update Released

The verification of the Stable Release Update for fwupd has completed successfully and the package is now being released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package fwupd - 1.2.10-1ubuntu2~ubuntu18.04.3

---------------
fwupd (1.2.10-1ubuntu2~ubuntu18.04.3) bionic; urgency=medium

  * d/p/0001-dont-semver-conversion.patch, d/p/0001-version-handling.patch
    d/p/0001-plain_support_in_version.patch:
    backport regression fix that we can install firmware with the same
    without --allow-reinstall in command line. the patch already merged
    in upstream 1.2.X branch. (LP: #1820768)

fwupd (1.2.10-1ubuntu2~ubuntu18.04.2) bionic; urgency=medium

  * d/p/0001-trivial-libfwupd-skip-tests-if-machine-id-is-empty-t.patch:
    - Only check the vendor ID if the device has one set (LP: #1856896)

fwupd (1.2.10-1ubuntu2~ubuntu18.04.1) bionic; urgency=medium

  * Backport to bionic (LP: #1820768)
    - meson-0.45-bc.patch: Fix build with meson 0.45

  [ Steve Langasek ]
  * Drop added Recommends: on bolt which is not in flavor seeds and adds a
    new service.

 -- Yuan-Chen Cheng <email address hidden> Thu, 09 Jan 2020 16:25:38 +0800

Changed in fwupd (Ubuntu Bionic):
status: Fix Committed → Fix Released
Rex Tsai (chihchun)
Changed in oem-priority:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.