[PATCH] Revert "debian: Allow empty passwords"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The workaround that has been used so for doesn't work.

For the user account, the password is indeed removed in the end of the
installation, but then login in from GDM is impossible. Although it
works from a VT, this is not something obvious that every user would
try. So, requiring the user password seems the best to do for now and
when another workaround is found the user password can be set to
optional again.

For the root account, the password is not removed in the of the
installation and ends up being set as "dummyPa55w0rd", something that
the user would never guess, unless they have access to the libosinfo
code. So, requiring the admin password seems the best to do for now and
when another workaround is found the admin password can be set to
optional again.

This reverts commit 087a8f8a23895e2d792b0f6d26a9d0c01a6c7e87.

 Conflicts:
	data/install-script/debian.org/debian-preseed-desktop.xml.in

Signed-off-by: Fabiano Fidêncio <fidencio@xxxxxxxxxx>

Resolves:
https://bugzilla.redhat.com/show_bug.cgi?id=1336108
https://bugzilla.redhat.com/show_bug.cgi?id=1336109
---
 .../debian.org/debian-preseed-desktop.xml.in       | 36 +++-------------------
 .../debian.org/debian-preseed-jeos.xml.in          | 18 +----------
 2 files changed, 5 insertions(+), 49 deletions(-)

diff --git a/data/install-script/debian.org/debian-preseed-desktop.xml.in b/data/install-script/debian.org/debian-preseed-desktop.xml.in
index fd69bf2..a786a30 100644
--- a/data/install-script/debian.org/debian-preseed-desktop.xml.in
+++ b/data/install-script/debian.org/debian-preseed-desktop.xml.in
@@ -14,6 +14,10 @@
       <param name="user-fullname" policy="optional"/>
       <param name="user-password" policy="optional"/>
       <param name="admin-password" policy="optional"/>
+
+      <!-- This MUST be nonempty. Can we enforce this? -->
+      <param name="user-password" policy="required"/>
+      <param name="admin-password" policy="required"/>
       <param name="avatar-location" policy="optional"/>
       <param name="avatar-disk" policy="optional"/>
 
@@ -130,34 +134,16 @@ d-i time/zone select <xsl:value-of select="config/l10n-timezone"/>
 
 ## Root
 d-i passwd/root-login boolean true
-<xsl:choose>
-  <xsl:when test="config/admin-password = ''">
-# Real password will be set later
-d-i passwd/root-password password dummyPa55w0rd
-d-i passwd/root-password-again password dummyPa55w0rd
-  </xsl:when>
-  <xsl:otherwise>
 d-i passwd/root-password password <xsl:value-of select="config/admin-password"/>
 d-i passwd/root-password-again password <xsl:value-of select="config/admin-password"/>
-  </xsl:otherwise>
-</xsl:choose>
 
 ## User
 d-i user-setup/allow-password-weak boolean true
 d-i user-setup/encrypt-home boolean false
 d-i passwd/user-fullname string <xsl:call-template name="user-fullname"/>
 d-i passwd/username <xsl:value-of select="config/user-login"/>
-<xsl:choose>
-  <xsl:when test="config/user-password = ''">
-# Real password will be set later
-d-i passwd/user-password password dummyPa55w0rd
-d-i passwd/user-password-again password dummyPa55w0rd
-  </xsl:when>
-  <xsl:otherwise>
 d-i passwd/user-password password <xsl:value-of select="config/user-password"/>
 d-i passwd/user-password-again password <xsl:value-of select="config/user-password"/>
-  </xsl:otherwise>
-</xsl:choose>
 
 
 ### Partitioning
@@ -201,20 +187,6 @@ d-i grub-installer/bootdev string <xsl:call-template name="target-disk"/>
 ### Cleanup
 d-i finish-install/reboot_in_progress note
 
-<xsl:choose>
-  <xsl:when test="config/admin-password = ''">
-# Delete root password
-d-i preseed/late_command string in-target passwd -d root
-  </xsl:when>
-</xsl:choose>
-
-<xsl:choose>
-  <xsl:when test="config/user-password = ''">
-# Delete user password
-d-i preseed/late_command string in-target passwd -d <xsl:value-of select="config/user-login"/>
-  </xsl:when>
-</xsl:choose>
-
 ### Remove the installation DVD from the apt-get's sources.list
 d-i preseed/late_command string in-target sed -i '/cdrom/d' /etc/apt/sources.list
 
diff --git a/data/install-script/debian.org/debian-preseed-jeos.xml.in b/data/install-script/debian.org/debian-preseed-jeos.xml.in
index 6d1c405..57dc8e4 100644
--- a/data/install-script/debian.org/debian-preseed-jeos.xml.in
+++ b/data/install-script/debian.org/debian-preseed-jeos.xml.in
@@ -4,7 +4,7 @@
     <expected-filename>preseed.cfg</expected-filename>
     <can-pre-install-drivers>true</can-pre-install-drivers>
     <config>
-      <param name="admin-password" policy="optional"/>
+      <param name="admin-password" policy="required"/>
       <param name="l10n-keyboard" policy="optional" value-map="http://x.org/x11-keyboard"/>
       <param name="l10n-timezone" policy="optional"/>
       <param name="l10n-language" policy="optional"/>
@@ -88,17 +88,8 @@ d-i partman/confirm_nooverwrite boolean true
 
 d-i passwd/root-login boolean true
 d-i passwd/make-user boolean false
-<xsl:choose>
-  <xsl:when test="config/admin-password = ''">
-# Real password will be set later
-d-i passwd/root-password password dummyPa55w0rd
-d-i passwd/root-password-again password dummyPa55w0rd
-  </xsl:when>
-  <xsl:otherwise>
 d-i passwd/root-password password <xsl:value-of select="config/admin-password"/>
 d-i passwd/root-password-again password <xsl:value-of select="config/admin-password"/>
-  </xsl:otherwise>
-</xsl:choose>
 
 tasksel tasksel/first multiselect standard ssh-server
 
@@ -114,13 +105,6 @@ d-i apt-setup/security_host string
 
 d-i finish-install/reboot_in_progress note
 
-<xsl:choose>
-  <xsl:when test="config/admin-password = ''">
-# Delete password
-d-i preseed/late_command string in-target passwd -d root
-  </xsl:when>
-</xsl:choose>
-
 ### Remove the installation DVD from the apt-get's sources.list
 d-i preseed/late_command string in-target sed -i '/cdrom/d' /etc/apt/sources.list
 
-- 
2.7.4

_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Fedora Users]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux