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>

mount: do not treat arm/sparc specially.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Martin Schlemmer <azarah@gentoo.org>

mount: don't umount sysfs when running umount -a

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Martin Schlemmer <azarah@gentoo.org>

po: typo in french translation of mount error.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Martin Schlemmer <azarah@gentoo.org>

mount: fix -fv so that it doesn't incorrectly spit out an error that nothing was done.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Martin Schlemmer <azarah@gentoo.org>

sys-utils: move some man pages from category 8 to 1

The dmesg, ipcrm, ipcs, renice and setsid are user-accessible commands
and belong in man1 more than to man8.

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

docs: fix info about devel/master branchs

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

docs: remove deprecated section from README

The deprecated stuff is already described in the DEPRECATED file.

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

build-sys: add SUID_CFLAGS

The patch allows to define special CFLAGS for typical suid programs
(like mount, umount, chfn, ...). Some distributions use for example
"-fpic" for suid binaries.

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

mount: add support for mtab "uhelper" option

The helper is an external /sbin/umount.<suffix> program where the
suffix is a value from the uhelper= option from /etc/mtab.

The uhelper (unprivileged umount helper) is possible to used when
non-root user wants to umount a mountpoint which is not defined in the
/etc/fstab file (e.g devices mounted by HAL).

This option is already supported by HAL upstream.

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

mount: update mtab correctly when mount --move

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

schedutils: define SCHED_BATCH when compile with old glibc

This is temporary workaround and it will be removed in 2.14 when
minimal number of people will use old systems where is not defined
SCHED_BATCH in (bits/)sched.h.

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

check exit status of autotools

Make sure each autotool run in autogen.sh worked and if not, abort.
Signed-off-by: Mike Frysinger <vapier@gentoo.org>

todo: add request for cal highlight control option

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

todo: add entry about replay

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

tests: refresh mtablock output in expected/ directory

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

mount: relative atime support

Add the "relatime" (relative atime) option support to mount.  Relative
atime only updates the atime if the previous atime is older than the
mtime or ctime.  Like noatime, but useful for applications like mutt
that need to know when a file has been read since it was last
modified.

Cc: Adrian Bunk <bunk@stusta.de>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: Karel Zak <kzak@redhat.com>
Signed-off-by: Valerie Henson <val_henson@linux.intel.com>

tests: add missing header

minor patch, which adds a missing string.h header to
mnt_test_sysinfo.c

Signed-Off-By: Matthias Koenig <mkoenig@suse.de>

misc-utils: add scriptreplay manpage

a really minor patch which adds the manpage for scriptreplay.

Signed-Off-By: Matthias Koenig <mkoenig@suse.de>

mount: shared-subtree support

This patch builds shared-subtree semantics awareness into the mount
command.  Updates the man page for mount too.

The patch also fix a conflict between MS_COMMENT and MS_UNBINDABLE
(-- kzak).

Signed-off-by: Karel Zak <kzak@redhat.com>
Signed-off-by: Ram Pai <linuxram@us.ibm.com>

tests: enable mtablock test when uid=0 only

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

docs: add the DEPRECATED file

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

losetup: add to man page info about deprecated cryptoloop

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

mount: fix mtab_lock

* the lock function uses F_SETLK / F_SETLKW as a conditional wait.
  It's more reliable and better for performance to close the
  MOUNTED_LOCK file in unlock_mtab(), otherwise concurrent process will
  be wait by while () { link() } loop instead on fcntl(F_SETLKW).

  Thanks to Jeff Moyer <moyer@redhat.com> who found the problem two
  year ago.

* when open(MOUNTED_LOCK) failed, we need to try everything again, but
  the original code didn't zeroize "we_created_lockfile" and the old
  version in particular case left lock_mtab() without locked /etc/mtab.
  This is nasty bug.

* the original locking code had bad performance due too long sleep
  (1s),  between attempts. Now we're more aggressive and we use
  5000ms. The result is that more processes is able to lock mtab in
  short time slice.

  Thanks to Peter Rockai <prockai@redhat.com> who found the problem
  and suggest a first version of the code with usleep.

* now we don't count number of attempts anymore, but we count sum of
  time which we spend in the mtab_lock(). The number of attempts is
  not important (and it also depends on CPU performance, load,
  scheduler, ...), the important thing is how long we spend with
  locking. Now time  limit is 30s.

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

tests: add lock_mtab() performance and reliability test

The test starts concurrently many processes that use lock_mtab() as
lock for access to same file.

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

tests: make clean need to remove diffs and outputs

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

losetup: add support read-only loops

This feature has been already supported by mount, but it wasn't accessible by
losetup command. Now you can use "losetup -r".

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

losetup: add -a option to list all used loop devices

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

mount: add -s and -f and note to man page for external mount helpers

The mount -s (for nfs only) and -f options must be exported to
external /sbin/mount.<type> helpers.

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

mount: call /sbin/mount.<type> also when mounting without "-t"

The mount uses /sbin/mount.<type> when the type is *defined* on
command line or in fstab only. It's not a consistent solution, because
we also support fylesystem type autodetection. The patch allows to
mount OCFS2 by label for example.

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

tests: add test for /sbin/mount.<type> call

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

newgrp: add support for /etc/gshadow

The original newgrp command doesn't expect group pasword in /etc/gshadow
although almost all distributions use this file (and the gpasswd command).

The newgrp from util-linux is deprecated and better is use shadow-utils only.
Unfortunately, shadow-utils are broken too (see RH version where is bugfix).
In this case it's better fix util-linux version at least...

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

newgrp: check result from getgrnam() more carefully

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

hwclock: make ggc happy and check return values from fgets, read and write

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

hwclock: remove tailing white-spaces and clean up clock.h

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