All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Gordon Lack" <gml4410@ggr.co.uk>
To: autofs@linux.kernel.org
Subject: Bug in autofs v4.1.4 init.d script (and fix)
Date: Fri, 08 Sep 2006 12:31:37 +0100	[thread overview]
Message-ID: <45015498.22A9C700@ggr.co.uk> (raw)
In-Reply-To: 44EB1231.7ABAA89B@ggr.co.uk

   In the getfilemounts() function when the code comes acros a line
which starts with a "+" (include an NIS map here) instead of just
including that map it cats the entire /etc/auto.master file again and
actually include *all* NIS maps which are mentioned.  This means that if
you have n (> 1) NIS map inclusions then the inclusions are done <n>
times,and it also means that the inclusion for all maps is first done
when the first NIS map inclusion is found. 

   It is perfectly legitimate to have an /etc/auto.master file like
this:

+auto.master-TEST
/home   /etc/my-home-override.amap
+auto.master

   whereby the /home setting is meant to override the stanmdrad one in
auto.master.  The bug in the getfilemounts() function stops this
happening as auto.master will be included when +auto.master-TEST is
seen.

   The fix for this is to change the "while read auto_master_in" loop to
be:

        while read auto_master_in
        do
            case "$auto_master_in" in
            +*)
                nismap=`echo $auto_master_in | sed -e 's/^\+//'`
                catnismap $nismap
                ;;
            *)
                echo $auto_master_in
                ;;
            esac
        done

  reply	other threads:[~2006-09-08 11:31 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-07-07 15:02 -hosts and -null enabling for autofs v4.1.x Gordon Lack
2006-07-13 13:10 ` Ian Kent
2006-07-13 13:42 ` Jeff Moyer
2006-07-14 11:33   ` Gordon Lack
2006-07-14 13:31     ` Jeff Moyer
2006-07-14 15:58       ` Gordon Lack
2006-08-07  6:56         ` Ian Kent
2006-08-22 14:03 ` Bug (and patch) in autofs init.d script (v4.1.4) Gordon Lack
2006-08-22 14:18 ` Enhancement patch for autofs v4.1.4 init.d script Gordon Lack
2006-09-08 11:31   ` Gordon Lack [this message]
2006-09-12 12:58     ` Bug in autofs v4.1.4 init.d script for reload (and fix) Gordon Lack
2006-09-13  9:42       ` Gordon Lack
2006-11-10 11:58 ` autofsv5 bug. Non-existent dirs get "mounted"! Gordon Lack
2006-11-10 12:05   ` autofsv5 enhancement - map names in /proc/mounts Gordon Lack
2006-11-10 17:33     ` autofsv5 - mouting files (not dirs) Gordon Lack
2006-11-11  2:58       ` Ian Kent
2006-11-11  2:53     ` autofsv5 enhancement - map names in /proc/mounts Ian Kent
2006-11-13 10:09       ` Gordon Lack
2006-11-13 15:44         ` Ian Kent
2006-11-13 16:58           ` Gordon Lack
2006-11-14  2:00             ` Ian Kent
2006-11-14 10:00               ` Gordon Lack
2006-11-14 11:42                 ` Ian Kent

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=45015498.22A9C700@ggr.co.uk \
    --to=gml4410@ggr.co.uk \
    --cc=autofs@linux.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.