Bug 28072

Summary: anaconda's loader.c has incorrect umount() call
Product: [Retired] Red Hat Linux Reporter: Jim Wright <jwright>
Component: anacondaAssignee: Erik Troan <ewt>
Status: CLOSED RAWHIDE QA Contact: Brock Organ <borgan>
Severity: low Docs Contact:
Priority: medium    
Version: 7.0   
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: 2001-02-20 19:33:34 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 Jim Wright 2001-02-17 01:37:54 UTC 
From Bugzilla Helper:
User-Agent: Mozilla/4.75 [en] (X11; U; Linux 2.2.16-22 i686)


In loader/loader.c, the following code appears:

    mlLoadModule("nfs", NULL, modLoaded, *modDepsPtr, NULL, NULL, flags);

    if (doPwMount(ksPath, "/tmp/nfskd", "nfs", 1, 0, NULL, NULL)) {
        logMessage("failed to mount %s", ksPath);
        return 1;
    }

    fullFn = malloc(strlen(file) + 20);
    sprintf(fullFn, "/tmp/nfskd/%s", file);
    copyFile(fullFn, location);

    umount("/tmp/nfs");

The last line should be

    umount("/tmp/nfskd");


Reproducible: Always
Steps to Reproduce:
1. do a kickstart install using a config file on an nfs server

	

Actual Results:   notice that /tmp/nfskd does not get unmounted


Expected Results:   /tmp/nfskd should be unmounted after the kickstart
config file is copied
Comment 1 Erik Troan 2001-02-20 20:30:07 UTC 
<blush> Fixed.
Comment 2 Glen Foster 2001-03-02 18:05:34 UTC 
Our installer team-lead thinks we should really fix this before next release.