None matching mokutils passwords

Bug #1560940 reported by Dave Morley on 2016-03-23
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubiquity (Ubuntu)
High
Mathieu Trudel-Lapierre

Bug Description

STEPS:
Requirements a secure boot enabled pc/kvm
1. Install latest pending iso on a secureboot enable machine
2. On the setup page select update from the web and 3rd party drivers
3. Enable disable secureboot
4. Type in the letter a or the password moo1234

EXPECTED:
I expect the the passwords to not say they don't match when they do and enable me to continue the install process

ACTUAL:
Passwords say they don't match even when it is one letter this prevents the install from proceeding

Related branches

Dave Morley (davmor2) wrote :
Ubuntu QA Website (ubuntuqa) wrote :

This bug has been reported on the Ubuntu ISO testing tracker.

A list of all reports related to this bug can be found here:
http://iso.qa.ubuntu.com/qatracker/reports/bugs/1560940

tags: added: iso-testing
Mario Limonciello (superm1) wrote :

I think the reason that this is actually happening is that this password is less than 8 characters. The secure boot plugin enforces that you have at least 8 characters but doesn't tell you in the U/I about this.

the validation for strength (human_password_strength) only enforces that the password be greater than 6 characters.

Dave Morley (davmor2) wrote :

Indeed but then the error message is wrong, they are both short but they match. It should throw up a TOO SHORT warning not don't match.

Mario Limonciello (superm1) wrote :

mokutil has a requirement of a length of 8.

The easiest solution here is to just set human_password_strength's too short to something less than 8. the implication for that is that on partman a password less than 8 won't be accepted for crypto (whereas previously it would have).

Changed in ubiquity (Ubuntu):
status: New → In Progress
importance: Undecided → High
assignee: nobody → Mathieu Trudel-Lapierre (cyphermox)

Fixing the prompts to show "Short password" (which should already be translated but somehow didn't show up) if the password length is less than 8, and I capped the size to 16 characters since this is also the maximum for mokutil.

Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubiquity - 2.21.57

---------------
ubiquity (2.21.57) xenial; urgency=medium

  [ Mathieu Trudel-Lapierre ]
  * Automatic update of included source packages: localechooser
    2.65ubuntu4. (LP: #1551285)
  * scripts/simple-plugins: better handle passing the MokPW key to mokutil
    and the result from the command -- we explicitly can ignore errors here.
    (LP: #1567445)
  * plugin-viewer-gtk.py: fix plugin-viewer-gtk to allow showing most plugins
    correctly; useful for debugging.
  * ubiquity/plugins/ubi-prepare.py: rework password validation behavior for
    MokPW without changing user-visible strings: we only need to show feedback
    for a "good" password (of the allowable lengths), and correct the behavior
    for mismatched passwords. Also make it obvious if the chosen password is
    too short. (LP: #1560940)

  [ Shih-Yuan Lee (FourDollars) ]
  * Really make oem-config-prepare quiet when started by a non-priviledged
    user.

 -- Mathieu Trudel-Lapierre <email address hidden> Wed, 13 Apr 2016 14:36:37 -0400

Changed in ubiquity (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers