None matching mokutils passwords

Bug #1560940 reported by Dave Morley
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubiquity (Ubuntu)
Fix Released
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

Tags: iso-testing

Related branches

Revision history for this message
Dave Morley (davmor2) wrote :
Revision history for this message
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
Revision history for this message
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.

Revision history for this message
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.

Revision history for this message
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)
Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

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.

Revision history for this message
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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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