Bug 243784 - sub-mount maps do not work in autofs v5
Summary: sub-mount maps do not work in autofs v5
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: autofs
Version: 5.0
Hardware: All
OS: Linux
low
low
Target Milestone: ---
: ---
Assignee: Ian Kent
QA Contact: Brock Organ
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-06-11 21:20 UTC by Jeff Moyer
Modified: 2007-11-30 22:07 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-06-12 12:29:07 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Jeff Moyer 2007-06-11 21:20:13 UTC
Version-Release number of selected component (if applicable):
autofs-5.0.1-0.rc2.46

How reproducible:
100%

Steps to Reproduce:
# cat /etc/auto.master
/test   /etc/auto.test --debug

# cat /etc/auto.test
sub     -fstype=autofs /etc/auto.sub

# cat /etc/auto.sub
*       localhost:/export/&

Default /etc/sysconfig/autofs, default /etc/nsswitch.conf.

It seems like parse_sun no longer deals with submount maps properly.
Here is the debug log:

automount[9321]: master_do_mount: mounting /test
automount[9321]: lookup_nss_read_map: reading map file /etc/auto.test
automount[9321]: handle_packet: type = 3
automount[9321]: handle_packet_missing_indirect: token 7, name sub, request pid 9331
automount[9321]: lookup_mount: lookup(file): looking up sub
automount[9321]: lookup_mount: lookup(file): sub -> -fstype=autofs /etc/auto.sub
automount[9321]: parse_mount: parse(sun): expanded entry: -fstype=autofs
/etc/auto.sub
automount[9321]: parse_mount: parse(sun): gathered options: fstype=autofs
automount[9321]: parse_mount: parse(sun): dequote("/etc/auto.sub") -> /etc/auto.sub
automount[9321]: parse_mapent: parse(sun): gathered options: fstype=autofs
automount[9321]: parse_mapent: parse(sun): invalid location
automount[9321]: handle_packet: type = 3
automount[9321]: handle_packet_missing_indirect: token 8, name sub, request pid 9331
automount[9321]: lookup_mount: lookup(file): looking up sub
automount[9321]: master_notify_state_change: sig 15 switching /test from 1 to 5
automount[9321]: st_prepare_shutdown: state 1 path /test
automount[9321]: expire_proc: exp_proc = 1084385600 path /test
automount[9321]: expire_cleanup: got thid 1084385600 path /test stat 0
automount[9321]: expire_cleanup: sigchld: exp 1084385600 finished, switching
from 5 to 7
automount[9321]: umount_multi: path /test incl 0

Comment 1 Ian Kent 2007-06-12 03:37:57 UTC
Yes, I see.
Obviously we let this through in version 4.

Strictly speaking the location requires a ":" because
it begins with a "/". The connectathon submount tests
all have a colon in front of the path location and have
always been like that since I started working with it.

I'm not sure what should be done here as allowing this
has other side affects such as being able to reliably
identify a multi-mount.

Ian


Comment 2 Jeff Moyer 2007-06-12 12:29:07 UTC
Actually, in the view autofs v4 versions I've tested, they don't allow this
either!  So, this is not a bug.  Submount maps that supply an explicit path
require a : escape, just like everyone else.

Sorry for the noise!


Note You need to log in before you can comment on or make changes to this bug.