Project

General

Profile

Actions

Bug #37610

closed

HostCommon.crypt_passwords reencrypts Base64 based passwords for Grub, leading to errors

Added by Evgeni Golov 5 months ago. Updated 2 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Unattended installations
Target version:

Description

crypt() only accepts passwords up to 512 characters (at least in modern libxcrypt >= 4.4.4 like present on EL9)

While we usually do not pass so long passwords into it, there is a situation where we (accidentally) do while using Base64(-Windows) passwords.
This is usually guarded by password_base64_encrypted?, but because we are changing the password, it returns false.
The later code then works fine for the root password, but when we also try to encrypt the grub password, we pass in the (already "encrypted") root password, which now is longer than 512 characters.


Related issues 1 (0 open1 closed)

Related to Foreman - Bug #33811: Reprovisioning a host using new HostGroup does not inherit root password from the new HostGroupClosedDominik MatoulekActions
Actions #1

Updated by Evgeni Golov 5 months ago

  • Related to Bug #33811: Reprovisioning a host using new HostGroup does not inherit root password from the new HostGroup added
Actions #2

Updated by The Foreman Bot 5 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/10229 added
Actions #3

Updated by The Foreman Bot 5 months ago

  • Pull request deleted (https://github.com/theforeman/foreman/pull/10229)
Actions #4

Updated by The Foreman Bot 5 months ago

  • Pull request https://github.com/theforeman/foreman/pull/10229 added
Actions #5

Updated by Ewoud Kohl van Wijngaarden 4 months ago

  • Category set to Unattended installations
  • Triaged changed from No to Yes
  • Found in Releases 3.2.0 added
Actions #6

Updated by The Foreman Bot 4 months ago

  • Fixed in Releases 3.12.0 added
Actions #7

Updated by Evgeni Golov 4 months ago

  • Status changed from Ready For Testing to Closed
Actions #8

Updated by The Foreman Bot 4 months ago

  • Pull request https://github.com/theforeman/foreman/pull/10249 added
Actions #9

Updated by The Foreman Bot 4 months ago

  • Fixed in Releases 3.11.2 added
Actions #10

Updated by Evgeni Golov 4 months ago

  • Fixed in Releases 3.11.1 added
  • Fixed in Releases deleted (3.11.2)
Actions #11

Updated by Evgeni Golov 4 months ago

  • Target version set to 3.11.1
Actions #12

Updated by The Foreman Bot 2 months ago

  • Pull request https://github.com/theforeman/foreman/pull/10314 added
Actions

Also available in: Atom PDF