Bug 109014
Summary: | pam_console locks with mount point in fstab which device file does not exist | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Behdad Esfahbod <behdad> | ||||
Component: | pam | Assignee: | Tomas Mraz <t8m> | ||||
Status: | CLOSED WORKSFORME | QA Contact: | |||||
Severity: | high | Docs Contact: | |||||
Priority: | medium | ||||||
Version: | rawhide | CC: | johnsonm, t8m | ||||
Target Milestone: | --- | Keywords: | EasyFix, Triaged | ||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2004-09-22 08:13:18 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: | |||||||
Attachments: |
|
Description
Behdad Esfahbod
2003-11-04 12:28:29 UTC
Created attachment 96150 [details]
Patch to fix.
Patch to fix the problem. Here is the problem: In
pam/modules/pam_console/chmod.c, in function change_file, which is supposed to
chmod the file, in two cases it calls change_via_fstab, once if it's a
directory, and once if the file does not exist! Which the second case is not
needed, as a file that does not exist can't be a mount point either. But then
it calls change_via_fstab, and if the non-existant file happens to be a mount
point (which of course cannot be mounted), then the corresponding device file
is passed to change_file, which again calls change_via_fstab, ...
The patch simply removes the dead case.
The patch is obvious and just needs to be applied. This shouldn't hang even without the patch because if the /123 file doesn't exist it will try only /mnt/floppy123 and this doesn't exist too so it recurses once more to change_via_fstab but this won't find anything and it fails. No infinite recursion. Humm, tested on FC2 now and works. Maybe something changed in the mean time... Feel free to close. They say time heals everything! |