mount: don't canonicalize the source of a pseudofs

In line with previous patches, the source of a pseudofs is strictly
semantic; do not attempt to resolve it to a filesystem path.

As a side effect of this, nspec needs to be initialized to NULL since
it may not be assigned null by spec_to_devname when the FS is deemed
a pseudofs.

Signed-off-by: Dave Reisner <dreisner@archlinux.org>

libmount: avoid resolving pseudofs source on update

Similar to the previous commit, when mounting a pseudofs, trust the
source of the fs, as it cannot be matched to backing device on the
filesystem.

Similar situation, but slightly different bug:

  # mkdir /foo /root/foo
  # echo 'foo  /foo  tmpfs  defaults  0  0' >> /etc/fstab
  # cd /root
  # mount -a
  # mount | grep /foo
  /root/foo on /foo type tmpfs (rw,relatime)

This fixes the pure libmount based mount utility, but the legacy mount
tool will still fail to handle this properly.

Signed-off-by: Dave Reisner <dreisner@archlinux.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

libmount; fix possible mem-leak

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

libmount: always trust the source of a pseudofs

Since nodev FSs have no backing block device, it's unreasonable to try
and resolve the source back to a path on the filesystem. Always trust
the source of the FS given in /etc/fstab to avoid remounting the device
when a directory of the same name exists in the current working
directory.

Example reproduction:

  # mkdir /foo
  # echo 'foo  /foo  tmpfs  defaults  0  0' >> /etc/fstab
  # mount -a
  # cd /root; mkdir foo
  # mount -a
  # mount | grep /foo
  foo on /foo type tmpfs (rw,relatime)
  /root/foo on /foo type tmpfs (rw,relatime)

[kzak@redhat.com: - use fs->flags to detect pseudo FS]

Signed-off-by: Dave Reisner <dreisner@archlinux.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: use CDROM_GET_CAPABILITY ioctl

And replace the current archaic logic of is_ide_cdrom_or_tape().

Signed-off-by: Davidlohr Bueso <dave@gnu.org>

mount: append inverting options for mount.<type>

 fstab:
server://foo   /mnt/foo    nfs    user,exec

The mount(8) does not append the option "exec" to /sbin/mount.<type>
helper's command line. This is no problem when executed by non-root
user as it reads the options from fstab only.

.. but when executed by root (UID=0) then the mount.<type> helper
follows the command where the "exec" option is missing. This is bug.

original version (strace output):

 execve("/sbin/mount.nfs", ["/sbin/mount.nfs",
        "sr.net.home:/mnt/store", "/mnt/store", "-o",
        "rw,nosuid,nodev,noauto,user"], [/* 21 vars */]) = 0

fixed version:

 execve("/sbin/mount.nfs", ["/sbin/mount.nfs",
        "sr.net.home:/mnt/store", "/mnt/store", "-o",
        "rw,nosuid,nodev,noauto,user,exec"], [/* 21 vars */]) = 0
                                    ^^^^^

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

lib,tt: don't trim last line char

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

losetup: cleanup headers, cmdline checks

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

losetup: cleanup man page

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

losetup: cleanup usage()

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

losetup: check  to control debug output

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

losetup: print info about one device

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

losetup: rewrite loop setup

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

losetup: rewrite -f

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

losetup: rewrite -d and -D

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

losetup: rewrite -c

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

losetup: rewrite -j

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

losetup: rewrite -a

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

lib,loopdev: add missing functions

 - improve loop_info usage (don't call ioctl more than once)
 - add functions to get devno and inode of the backing file
 - add function for compare any file with backing file by devno + inode
   or by filename

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

libblkid; fix memory leak in blkid_probe_get_wholedisk_probe()

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

prlimit: use xalloc libs

Signed-off-by: Davidlohr Bueso <dave@gnu.org>

losetup: move to sys-utils directory

Now losetup(8) is independent on mount/ stuff at all.

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

losetup: remove mount-specific headers

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

losetup: remove unnecessary header files

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

losetup: remove 32-bit ioctls

 ... kernel 2.4 ?! Who cares?

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

losetup: remove #ifdef LOOP_SET_FD nonsense

It's build-system task to enable/disable utils according to accessible
system features.

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

losetup: remove obsolete #ifdef MAIN

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

libmount: don't use "new" in header file

Reported-by: ben <benscott@nwlink.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

tests: update fdisk tests

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

fdisk: improve some messages

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

fdisk: Fix bad invalid flag 0x00000 warning message

This splits check_dos_label() and dos_init() off from get_boot() and gets
rid of the invalid flag 0x00000 warning message due to a check for MBR
signs in zeroized buffer:

memset(MBRbuffer, 0, 512);

if (what == create_empty_dos)
goto got_dos_table;
[...]
got_dos_table:
if (!valid_part_table_flag(MBRbuffer)) {
[...]
if (!valid_part_table_flag(pe->sectorbuffer))
fprintf(stderr, _("Warning: invalid flag 0x%04x of partition "
[...]

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

fdisk: avoid an intermediate variable in command_prompt()

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

fdisk: don't use get_existing_partition() if disk label isn't dos

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

fdisk: remove unused guess_device_type() function

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

fdisk: get rid of get_boot(create_empty_sun) call

create_sunlabel() should create a new empty SUN disklabel without checking data
itself writes to memory and initialize internal related fdisk variables.

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

fdisk: get rid of type_open global variable

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

fdisk: rename try() and tryprocpt() functions

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

fdisk: drop one parameter from try() function

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

fdisk: move code for renaming SGI bootfile to SGI module

This moves the code for renaming SGI bootfile from command_prompt() to
sgi_set_bootfilename() function.

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

fdisk: remove possibly_osf_label variable that is only assigned

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

fdisk: remove unused 'require' member in enum action

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

agetty: add static and const qualifiers

Remove also a static qualifier in main() function.

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

libblkid: minor docs changes

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

wipefs: use xalloc lib

Signed-off-by: Davidlohr Bueso <dave@gnu.org>

lists: add list sorting routine

We need a list sorting function, just to mention one example user that could
benefit is the lib/tt code to sort columns.

This patch adds list_sort() which uses the Merge Sort algorithm, behaving
nicely in O(nlog(n)), heavily based on the kernel's implementation[1].  The
private data (void *priv) passed to the comparison function as been removed to
adopt a qsort(3)-like syntax, and IMHO we don't really need it anyways.

[1]: http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=blob;f=lib/list_sort.c;h=d7325c6b103f0be078ff3672c35c468ed35738f1;hb=HEAD

[kzak@redhat.com: - use size_t in list_sort()]

Signed-off-by: Davidlohr Bueso <dave@gnu.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

findmnt: support alternative location of fstab

Signed-off-by: Dave Reisner <dreisner@archlinux.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

findmnt: use warnx for libmnt error callback

libmount doesn't set errno appropriately here:

$ findmnt -s
findmnt: /etc/fstab: parse error at line 20: Success

Signed-off-by: Dave Reisner <dreisner@archlinux.org>

setterm: order man page items alphabetically

Signed-off-by: Jari Aalto <jari.aalto@cante.net>

prlimit: return proper type

The add_prlim() function's definition currently returns a bogus structure.

Signed-off-by: Davidlohr Bueso <dave@gnu.org>

docs: add hwclock -c to TODO file

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

partx: don't treat empty partition table as error

Phillip wrote:
 If a partition table was detected, but it had no partitions
 in it, a poorly worded error message was printed and further
 action halted.  For partx -a, this gave an unnecessary error.

Reported-by: Phillip Susi <psusi@cfl.rr.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

partx: cleanup usage() and man page, add undocumented --verbose

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

partx: don't print error when partition already doesn't exist

If you run partx -d to delete partitions, and there are gaps
in the sequence ( partition 2 exists, but 1 doesn't ), then it
would complain that it had errors trying to remove the partitions
that already don't exist. Changed to ignore the error when
errno == ENXIO.

Signed-off-by: Phillip Susi <psusi@cfl.rr.com>

agetty: don't use log_err() for non-fatal errors

The TIOCSCTTY ioctl requires that caller is session leader -- so it
depends on initd (or we have to add setsid() to aggety). It seems that the
traditional way is to setup tty in agetty and session in login(1).

It means that all session related things (TIOCSCTTY, vhangup, ...) in the
command agetty should be optional. (Note that vhangup() is called when
--hangup is explicitly specified on command line, so log_err() makes
sense there.)

Reported-by: Andrew Walrond <andrew@walrond.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

logger: fix remote logging

Reported-by: Hervé Quillévéré <herve_quillevere@herveq.tk>
Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: add log_err to smatch non-return functions

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

build-sys: check HAVE_ definitions with #ifdef [smatch scan]

Fix to `warning: undefined preprocessor identifier' messages.

[kzak@redhat.com: replace "#ifdef XXX #ifdef YYY" with
                  "#if defined(XXX) && defined(YYY)"]

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: check HAVE_ definitions with #ifdef [smatch scan]

Fix to `warning: undefined preprocessor identifier' messages.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: add missing headers to dist

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

libmount: add support for mount -a --fork

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

libmount: add "+" prefix for options pattern (e.g findmnt -O)

Examples:

 * findmnt --fstab -O noauto
   Returns all entries where is not "auto" option (including entries
   with "noauto" option.

 * findmnt --fstab -O +noauto
   Returns all entries where is literally "noauto" option.

Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=758457
Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: add -a to umount(8) sample

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

libmount: add mnt_context_umount_next()

This is "umount -a" backend.

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

libmount: improve verbose message

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

Merge branch 'login-utils' of https://github.com/kerolasa/lelux-utiliteetit

* 'login-utils' of https://github.com/kerolasa/lelux-utiliteetit: (23 commits)
  last: fix few compiler warnings
  setpwnam: remove non-ANSI definitions [smatch scan]
  last: possible buffer overflow fixed [smatch scan]
  vipw: fix coding style
  setpwnam: fix coding style
  selinux_utils: fix coding style
  newgrp: fix coding style
  logindefs: fix coding style
  islocal: fix coding style
  chsh: fix coding style
  chfn: fix coding style
  chfn, setpwnam: let config.h be in control of extensions in use
  chfn, chsh: new file pamfail.h for error printing
  chsh: build bug: do not override config.h definition
  chfn, chsh, setpwnam: get true/false from stdbool.h
  login: add version printing option
  newgrp: add version and help options
  vipw: stop printing non-sense version string
  vipw: use libc error printing facilities
  chsh: use libc error printing facilities
  ...

libmount: add new exprimental umount(8)

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

include,optutils: add option_to_longopt()

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

libmount: use USAGE_* from mount(8) sample

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

last: fix few compiler warnings

Calling abort() as default seems appropriate as the default should be
impossible to reach.

last.c:355:3: warning: switch missing default case [-Wswitch-default]

And simple line removals.

last.c:110:13: warning: redundant redeclaration of 'optind' [-Wredundant-decls]
last.c:111:15: warning: redundant redeclaration of 'optarg' [-Wredundant-decls]

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

setpwnam: remove non-ANSI definitions [smatch scan]

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

last: possible buffer overflow fixed [smatch scan]

last.c +447 ttyconv(11) error: buffer overflow '(arg)' 3 <= 3

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

vipw: fix coding style

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

setpwnam: fix coding style

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

selinux_utils: fix coding style

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

newgrp: fix coding style

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

logindefs: fix coding style

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

islocal: fix coding style

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

chsh: fix coding style

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

chfn: fix coding style

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

chfn, setpwnam: let config.h be in control of extensions in use

The autotools will set _GNU_SOURCE which removes necessity to have
any _*_SOURCE definition in *.c files.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

chfn, chsh: new file pamfail.h for error printing

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

libmount: add sanitize_env() to mount(8) sample

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

libmount: remove magic constatnt from mount(8)

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

docs: add x-mount-mkdir to TODO file

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

agetty: map NL to CR-NL on output in initial termiossettings

From: Jacoby Hickerson <hickersonjl@gmail.com>
Signed-off-by: Dennis Jensen <dennis.h.jensen@siemens.com>

Merge branch 'master' of https://github.com/maurizio-lombardi/util-linux

* 'master' of https://github.com/maurizio-lombardi/util-linux:
  mkfs.minix: The total number of zones is limited to 65536 only on V1 filesystems
  mkfs.minix: Some bug fixes:
  mkfs.minix: The s_blocksize field of the MinixV3 superblock must be initialized with a valid block size, not the total number of blocks of the device!
  mkfs.minix: inode numbers are 32-bit wide in V3 filesystems
  mkfs.minix: In Minix V3 filesystems the maximum filename lenght is 60 chars and the dirsize is 64 bytes.
  mkfs.minix: Fix super_set_nzones()
  mkfs.minix: remove redundant code

libmount: fix compiler warning [-Wimplicit-function-declaration]

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

do: fix czech LABEL= translation

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

chsh: build bug: do not override config.h definition

The ONLY_LISTED_SHELLS is defined by --disable-chsh-only-listed
configuration option.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

chfn, chsh, setpwnam: get true/false from stdbool.h

Requires C99, which should not be a problem.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

login: add version printing option

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

newgrp: add version and help options

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

vipw: stop printing non-sense version string

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

vipw: use libc error printing facilities

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

chsh: use libc error printing facilities

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

chsh: align with howto-usage-function.txt

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

chfn: use libc error printing facilities

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

chfn: align with howto-usage-function.txt

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

fallocate: clean up fallbacks for FALLOC_FL_* flags

Reported-by: Voelker, Bernhard <bernhard.voelker@siemens-enterprise.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add mkfs.carmfs test for small FS with duplicate files

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