Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.

Bug 101265

Summary:	HTTP install adds extra leading / to some paths
Product:	[Retired] Red Hat Linux Beta	Reporter:	Mike Shaver <shaver>
Component:	anaconda	Assignee:	Jeremy Katz <katzj>
Status:	CLOSED RAWHIDE	QA Contact:	Mike McLean <mikem>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	beta1	CC:	gboyce, l0cksley
Target Milestone:	---
Target Release:	---
Hardware:	i386
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2006-04-24 18:59:58 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 Mike Shaver 2003-07-30 14:49:32 UTC

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4b) Gecko/20030523

Description of problem:
When selecting HTTP install, and providing a site path of

/~jacob/severn/

the Red Hat installer will request some files (the first such file is the
hdlist) with the prefix

//~jacob/severn/

which doesn't go to the same place.  (I believe this to be a characteristic of
the matching that mod_userdir does to know that it's supposed to head over to
public_html.)

Version-Release number of selected component (if applicable):
Severn beta 1 installer.

How reproducible:
Always

Steps to Reproduce:
1. Set up an install under a userdir (~msw/public_html)
2. Select HTTP install
3. Wait for it to tell you that it can't load the header list
4. Inspect your HTTP logs to see the extra leading /
5. (Optional) add a RedirectMatch rule so that you can install anyway

Comment 1 Mike Shaver 2003-07-30 15:44:28 UTC

Severn beta1 bugs are supposed to be marked i386, I now recall.

Comment 2 Bill Nottingham 2003-08-05 03:32:47 UTC

*** Bug 101522 has been marked as a duplicate of this bug. ***

Comment 3 Michael Fulbright 2003-08-13 17:03:10 UTC

I think I have this fixed. Please try again in our next beta.

Comment 4 l0cksley 2004-06-25 16:42:41 UTC

*** Bug 123607 has been marked as a duplicate of this bug. ***

Comment 5 l0cksley 2004-06-25 18:16:40 UTC

I've downloaded anaconda-10.0-5.src.rpm and tried to trace
the logMessages.

In mountUrlImage() in loader2/urlinstall.c,
we start in the URL_STAGE_MAIN state
and call urlMainSetupPanel() in loader2/urls.c.
This function gets the URL from the user
and stores it in a (struct iurlinfo).
It also forces ui->prefix to have a leading slash.

Returning to mountUrlImage(),
we change to the URL_STAGE_FETCH state
and call loadUrlImages(), which calls
loadSingleUrlImage(), which calls
urlinstStartTransfer() in loader2/urls.c.
This function generates the "transferring http://..."
logMessage with the extra slash I see in virtual terminal 3.
The bad URL is then passed to httpGetFileDesc()
in loader2/ftp.c, which presumably fails.

There is a block near the end of mountUrlImage()
with the comment

"sanitize url so we dont have problems like bug #101265"
"basically avoid duplicate /'s"

but I think this bug occurs before it is reached.

Comment 6 Jeremy Katz 2006-04-24 18:59:58 UTC

Mass-closing lots of old bugs which are in MODIFIED (and thus presumed to be
fixed).  If any of these are still a problem, please reopen or file a new bug
against the release which they're occurring in so they can be properly tracked.