Bug 55857

Summary: Graphical install allows space in lilo label
Product: [Retired] Red Hat Linux Reporter: Eric Sandeen <sandeen>
Component: anacondaAssignee: Brent Fox <bfox>
Status: CLOSED RAWHIDE QA Contact: Brock Organ <borgan>
Severity: medium Docs Contact:
Priority: medium    
Version: 7.2   
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2001-11-08 17:27:32 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Eric Sandeen 2001-11-07 20:41:35 UTC
From Bugzilla Helper:
User-Agent: Mozilla/4.78 [en] (X11; U; Linux 2.4.9-13SGI_XFS_PR1 i686)

Description of problem:
Grub allows spaces in boot labels, lilo doesn't .  Anaconda allows them in
both, which will cause a failure.

Version-Release number of selected component (if applicable):


How reproducible:
Always

Steps to Reproduce:
1. Run graphical installation
2. Choose lilo for the bootloader
3. Enter a boot label with spaces

	

Actual Results:  Lilo will fail

Expected Results:  I'd expect anaconda to restrict me to valid boot labels
so that lilo will not fail.  ;-)

Additional info:

Works for me - 

--- anaconda-7.2/iw/bootloader_gui.py.orig	Wed Nov  7 14:30:09 2001
+++ anaconda-7.2/iw/bootloader_gui.py	Wed Nov  7 14:30:48 2001
@@ -179,7 +179,7 @@
             cur = text[i]
 # lilo did not allow ' '!, grub does
 #            if cur == ' ' or cur == '#' or cur == '$' or cur == '=':
-            if cur == '#' or cur == '$' or cur == '=':
+            if cur == '#' or cur == '$' or cur == '=' or
(self.lilo_radio.get_active() and cur == " "):
                 entry.emit_stop_by_name ("insert_text")
                 return
             i = i + 1

Comment 1 Brent Fox 2001-11-08 17:21:08 UTC
Yes, I've verified this problem.

Comment 2 Eric Sandeen 2001-11-08 17:27:27 UTC
To be a bit more specific... the text installer handles this properly.

Comment 3 Brent Fox 2001-11-09 04:38:11 UTC
Fixed in cvs on anaconda-7-2-branch and on head.  Thanks for your bug report.