Actions
Bug #37610
closedHostCommon.crypt_passwords reencrypts Base64 based passwords for Grub, leading to errors
Status:
Closed
Priority:
Normal
Assignee:
Category:
Unattended installations
Target version:
Difficulty:
Triaged:
Yes
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.
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
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
Updated by The Foreman Bot 5 months ago
- Pull request deleted (
https://github.com/theforeman/foreman/pull/10229)
Updated by The Foreman Bot 5 months ago
- Pull request https://github.com/theforeman/foreman/pull/10229 added
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
Updated by Evgeni Golov 4 months ago
- Status changed from Ready For Testing to Closed
Applied in changeset foreman|79c9292704d428a475c9efea1e822ef4ba9cbc38.
Updated by The Foreman Bot 4 months ago
- Pull request https://github.com/theforeman/foreman/pull/10249 added
Updated by Evgeni Golov 4 months ago
- Fixed in Releases 3.11.1 added
- Fixed in Releases deleted (
3.11.2)
Updated by The Foreman Bot 2 months ago
- Pull request https://github.com/theforeman/foreman/pull/10314 added
Actions