Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use SHA512 for storing hashed passwords #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

earlruby
Copy link

@earlruby earlruby commented Dec 8, 2018

The root password and grub password were encrypted using
an (insecure) MD5 hash. The resulting kickstart file would
build virtual machines that store the (insecure) MD5
hashed password for root in /etc/shadow.

Since current operating systems using glibc 2.7 (or later)
default to SHA512 hashes for /etc/shadow and support SHA512
for grub, and Python 2.7's crypt function will generate a
SHA512 has for any salt starting with '$6$', this patch
changes the algorithm to SHA512 for storing hashed passwords.

The root password and grub password were encrypted using
an (insecure) MD5 hash. The resulting kickstart file would
build virtual machines that store the (insecure) MD5
hashed password for root in /etc/shadow.

Since current operating systems using glibc 2.7 (or later)
default to SHA512 hashes for /etc/shadow and support SHA512
for grub, and Python 2.7's crypt function will generate a
SHA512 has for any salt starting with '$6$', this patch
changes the algorithm to SHA512 for storing hashed passwords.
@earlruby
Copy link
Author

/assign @clumens

@Ricky-Tigg
Copy link

#9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants