mount: clean up info about NFS in mount.8

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: add note about fcntl/ioctl unreliability on NFS to mount.8

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: add note about /etc/mtab unreliability to mount.8

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add mount by uuid with label in fstab

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add mount by uuid with devname in fstab

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add mount by label with uuid in fstab

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add mount by label with devname in fstab

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add mount by devname with uuid in fstab

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add mount by devname with label in fstab

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add more variants to {mount,fstab}-by-{label,uuid,devname}

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: add support for mixed usage of SPECes

This patch improves a way how the mount works with SPECes (devname,
LABEL or UUID) and nodes (mountpoints) from command line.

The patch adds support for mixed usage of SPECes -- it means on
command line is different SPEC than in the /etc/fstab file. For
example:

command line:
   mount LABEL=foo
fstab:
   UID=915b048a-998d-4ee5-9e6b-7fcaaf34d3c5 /mnt/foo auto default

The mount command doesn't strictly require same SPEC on command line
and in fstab anymore. You can be more creative and mix UUID, LABEL
or device name.

This implementation is more effective that convert *all* entries from
/etc/fstab to a real devnames.

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: getfs_* (fstab) interface has to work with canonicalize()

The getfs_by_dir() and getfs_by_devname() have to be able to work
with canonicalized and also with non-canonicalized paths.

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: cleanup blkid cache after test device deinitialization

The libblkid reuses cached information when a cache entry is younger
than 2 secs. That's wrong, because util-linux tests use devices
more quickly.

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: use verbose mode instead debug mode

The patch:

  commit 61d9d2ff1cb0e28e2b88099bbc5b919fa5004fce
  Date:   Wed Dec 27 23:35:56 2006 +0100

added debug mode to the mount command. It was mistake. It's better to
use old verbose mode instead a new debug mode. This patch transform
all staff from debug mode to to verbose mode.

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: parse SPEC before search in fstab

The mount blindly uses a SPEC from command line and tries to found the
SPEC in /etc/fstab. It's better to parse the SPEC and search by UUID,
LABEL or devname only.

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: clean up getfs* (fstab.c) interface

Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: minor fixes in configure.in

Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: factor out common parts from mount/Makefile.am

Signed-off-by: Stepan Kasal <skasal@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add mount by devname from fstab

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add mount by UUID from fstab test

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add mount by label from fstab test

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add support for fstab modification

The patch adds ts_fstab_add and ts_fstab_clean routines.

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add mount by devname test

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add mount by LABEL test

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: fix Makefile.am (add missing tests)

The patch adds missing tests to Makefile.am.

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add mount by UUID test

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add ts_log and --verbose support

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: "if [...]" clean up

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add swapon by devname test

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add swapon by UUID test

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: simplify devices usage

The patch simplifies devices usage in the tests. The patch also improves
errors checking and reporting.

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add functions for label, uuid and fstype detection

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: code refactoring -- new ts_udev_loop_support function

The patch moves generic and duplicated code to a new
ts_udev_loop_support function.

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: code refactoring -- new ts_device_init function

The patch moves generic loop device (de)initialization code to
ts_device_init and ts_device_deinit functions.

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: code refactoring -- new ts_skip_nonroot function

The patch moves "if $UID != 0" test to a new function. That's better
than duplicate the code on many places.

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: fix dependence on blkid

The util-linux-ng tests have to be useful with blkid and also with
volume_id.

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: fsprobe: add libvolume_id support

Signed-off-by: Matthias Koenig <mkoenig@suse.de>
Signed-off-by: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

mount: fsprobe: add libvolume_id support to configure.ac

The patch add new option --with-fsprobe=<name> (where the <name> is
blkid or volume_id). The blkid is default. The mount cannot be
compiled without a filesystem detection library.

Signed-off-by: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

mount: fix has_* functions (CVE-2007-0822)

The functions have to check for NULL pointer.

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: fsprobe: use blkid cache only when really necessary

The blkid_get_cache() parses /etc/blkid.tab, it's better do it only
when we really need to resolve a spec (label or uuid).

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: fsprobe: make fsprobe_get_devname functions more generic

The blkid supports NAME=value parsing, but use the library for
this simple task is overkill. (The libblkid requires initialized
blkid cache all time, for all calls.)

This patch makes the fsprobe_get_devname_for_mounting() and
fsprobe_get_devname() generic for all fsprobe implementations.

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: fsprobe: rename the rest of API routines to fsprobe_*

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: fsprobe: remove mount_guess_fstype.{c,h}

Signed-off-by: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

mount: fsprobe: remove non-blkid code

This patch removes old FS detection code and enables
blkid code only when HAVE_BLKID is defined.

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: fsprobe: rename files to fsprobe_*

Signed-off-by: Karel Zak <kzak@redhat.com>

disk-utils: fix libuuid usage in mkswap

Fix libuuid usage in mkswap.

Signed-off-by: Matthias Koenig <mkoenig@suse.de>

sys-utils: fix man page headers

The man pages have been moved from man8 to man1, but the
headers still refer to man1.

Signed-off-by: Karel Zak <kzak@redhat.com>

docs: refresh AUTHORS file

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: kill mount_guess_rootdev

It's not used.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: add AC_GNU_SOURCE

The widechar code doesn't compile correctly without _GNU_SOURCE (at
least according to reports from testers. I don't see the problem on
FC6.)

Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: remove -fomit-frame-pointer

The -fomit-frame-pointer option makes debugging impossible on some
machines. It is automatically enabled by -O<n> optimizations.

Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: add Automake option dist-bzip2

We distribute .tar.bz2 along with .tar.gz .

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: in configure.ac, change "po" -> "$srcdir/po"

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: use dist_noinst_HEADERS in include/Makefile.am

It is slightly more appropriate.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: in the clean targets use "find ... | xargs rm -f"

"find ... -exec rm {}" is unnecessarily slow.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: remove po/Makevars.template from EXTRA_DIST

The is no point in distributing it.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: remove config/include-Makefile.am from EXTRA_DIST

It gets distributed automaticaly.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: do not use wildcards in EXTRA_DIST

Wildcards in EXTRA_DIST break "make dist" for vpath build.
But plain directory names are ok.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: use dist_usrbinexec_SCRIPTS in misc-utils/Makefile.am

A slightly more concise way to distribute the script "reset".

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: let configure instantiate the misc-utils/*.pl scripts

The substitution of @PERL@ is most done with AC_CONFIG_FILES.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: --prefix defaults to /usr

Use prefix=/usr as the default setting.  If prefix is /usr, make
bindir and sbindir default to /bin and /sbin, respectively.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: set DISTCHECK_CONFIGURE_FLAGS in top-level makefile

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: make the getopt example directory relative to datadir

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: amend .gitignore

Add several built files to .gitignore files.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: remove swapargs.h, move the tests to main configure.ac

Do not use header swapargs.h, generated by script swap.configure;
use the standard AC_DEFINE method in configure.ac.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: replace AC_TRY_* by AC_*_IFELSE

The AC_TRY_* macros are deprecated.  Following the suggestion from
autoupdate, I have replaced them by AC_*_IFELSE.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: s/AC_HELP_STRING/AS_HELP_STRING/

The macro has been renamed.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: do not use internal Autoconf variables

This is a cleanup of configure.ac removing the internal ac_cv_*
variables.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: merge adjacent AC_CONFIG_HEADERS and AC_CONFIG_FUNCS calls

Merge adjacent calls to AC_CONFIG_HEADERS--the generated configure is
then smaller.  Likewise for AC_CONFIG_FUNCS.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: move -D flags to *_CPPFLAGS

Preprocessor options belong to *_CPPFLAGS; add $(AM_CPPFLAGS) as we
want to amend it, not override.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: do not set with_foo twice

It's redundant to set with_slang=$withval, and similar.
(Inspect the generated configure; it contains
  withval=$with_slang; with_slang=$withval
:-)

Signed-off-by: Stepan Kasal <skasal@redhat.com>

swapon: simplify an #if

Replace two immerced #ifdef's by an #if with a compound conditional.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: use dist_noinst_DATA to work around the bug with dist_man_MANS

Automake mishandles dist_man_MANS.  Using dist_noinst_DATA to work
around this is more elegant than using dist-hook.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: simplify "clean" in tests/Makefile.am

The construction with "distclean-local" does not seem necessary.
Moreover, Automake complains when "clean" is redefined.

Simply put the rm to "clean-local".

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: use dist_example_DATA

"dist_" prefix is more elegant than EXTRA_DIST.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: mount/Makefile.am tiny cleanup II

Style change: init the *_LDADD vars, then amend them with +=.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: do not build convenience libraries in lib/

Build lib/*.c individually for each project which uses it, so that the
right set of flags is applied each time.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: mount/Makefile.am tiny cleanup

Major part of mount_SOURCES and umount_SOURCES is the same; factor it
out to a new make variable.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: remove AC_PROG_RANLIB

libtoolize (1.5.16) says it's now obsolete.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: declare SUID_CFLAGS and SUID_LDFLAGS as precious

Use AC_ARG_VAR to declare the variables SUID_CFLAGS and SUID_LDFLAGS as
precious.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: fix the example configure call in README

Adapt the example to pass the variables as arguments to the configure
script, as this is the prefered way.  And a few cosmetic changes to the
paragraph.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: add SUID_LDFLAGS

Programs which are usually installed with the setuid bit do need their own
CFLAGS and LDFLAGS.  SUID_LDFLAGS is analogic to SUID_CFLAGS.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: do not play with DEFS, use AM_CPPFLAGS

If a -D flag is needed, it belongs to AM_CPPFLAGS.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: move the optimization flags to AM_CFLAGS

The optimization flags are not preprocessor flags.
Moreover, CPPFLAGS and CFLAGS shall be overridable at make time, the
configure script shall not touch them.
Setting AM_CFLAGS in config/include-Makefile.am seems to be TRW.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: do not kick off AM_CFLAGS by SUID_CFLAGS

SUID_CFLAGS are not meant to override the makefile-wide AM_CFLAGS.
(We do not use AM_CFLAGS currently, but we will.)

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: update po/POTFILES.in

Remove mount/nfsmount* files from po/POTFILES.in .

Signed-off-by: Stepan Kasal <skasal@redhat.com>

mount: remove nfsmount() from sundries.h

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: remove all NFS code

Use /sbin/{u,}mount.nfs{,4} from nfs-utils! The mount command
will use these to mount nfs filesystems instead of internal code.

The /sbin/{u,}mount.nfs{,4} is supported from nfs-utils-1.1.0
(currently -rc1).

Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: fix the final message of autogen.sh

The message shall not recommend calling ./configure when srcdir != "." .

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: call automake after autoconf

"automake" calls "autoconf" internally, to trace the expansion of
configure.ac.  So it is more natural to call "autoconf" before
"automake".

Signed-off-by: Stepan Kasal <skasal@redhat.com>

build-sys: remove config.h.in from VCS

config.h.in is generated, no need to keep it.

Signed-off-by: Stepan Kasal <skasal@redhat.com>

mount: add support for context, fscontext and defcontext selinux mount options

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: use realloc for xstrconcat functions

Everywhere in code is

s = xstrconcat3(s, t, u);

that's nonsense use malloc(new_s); free(s); return new_s;

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: use growable string for options

The parse_opt() routine uses fixed size of string for mount options.
This is useless for future selinux options where is not well defined
size of selinux context name.

The patch also makes code more readable and all option-string
operations share same code.

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: avoid duplicate entries in mtab when mount -f

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add mount -o remount test

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add mount --move test

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: doesn't rpc_pipefs and nfsd on umount -a

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: fix list logic in update_mtab

If mtab does not contain the new entry, then only mc0->prev is updated
to point to absent, but not the old mc0->prev's nxt pointer.  Because
we then use the nxt pointers to write the new mtab, absent is not
added to the new mtab.

(Note: fortunately, the mount doesn't use the update_mtab() for new
mounts, but for remount, move and umount only -- kzak)

If mtab is empty, absent->prev should be set to mc0, and not
mc0->prev, as it will be NULL.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Martin Schlemmer <azarah@gentoo.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

mount: fix memory usage in update_mtab

The update_mtab deallocates memory which was allocated by caller. It's nice
opportunity for double-free errors.

The patch fix a memory leak if we have to abort before mc0 are freed.  The
patch also fix a memory leak when we deallocate old (umounted) entry.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Martin Schlemmer <azarah@gentoo.org>
Signed-off-by: Karel Zak <kzak@redhat.com>