login: fix LOGIN_CHOWN_VCS code

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

tests: search for "none" by findmnt

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

libmount: add mnt_fs_streq_target() and export all mnt_fs_streq_*

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

blkid: add DEVNAME= to export output format

Reported-by: Balamurugan Arumugam <barumuga@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

tests: wait a moment between mount and umount

It seems that without sleep between mount and umount are the test
unstable, because kernel is not able to provide updated
/proc/self/mountinfo so quickly (really?).

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

libmount: add mnt_fs_streq_srcpath()

We have to be careful with "none" or another dummy sources for pseudo
filesystems. These strings should be canonicalized or compared as a
paths.

The function is not exported by library API.

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

libmount: cosmetic changes around "none"

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

tests: update libmount tests

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

libmount: don't treat "none" differently

This causes more problems than it solves. In the latest edition:

  # mount -t proc none foo
  mount: foo: mount failed: Invalid argument

A check for source and target fails in mnt_context_apply_fstab()
because, even though they were indeed specified on the cmdline,
__mnt_fs_set_source_ptr() altered this and NULL'd out the source.

If you're able to mount this device via other means, other tools start
reporting oddities, such as mount's output:

  (null) on /foo type proc (rw,relatime)

or findmnt:

  TARGET      SOURCE FSTYPE OPTIONS
  /foo               proc   rw,relatime

Simply treat "none" like any other source when passed in.

[kzak@redhat.com: - don't translate NULL to "none" in mnt_fs_set_source()]

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

mountpoint: account for error from in mnt_fs_get_target

commit 04f087ec didn't take into consideration that mnt_fs_get_target()
could return an error, and would therefore show false positives, such
as:

$ mkdir foo; mountpoint foo
foo is a mountpoint

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

wipefs: always print devname

It seems too smart to print devname according to number of devices.

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

wipefs: support more device arguments for wipefs

[kzak@redhat.com: - print devname only if more devices specified]

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

wipefs: add quiet flag (-q)

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

libblkid: properly reset chain in probe_do_wipe()

The command wipefs is not able to remove all signatures from the
device if the first detected signature is linux raid. The another
superblock signatures are ignored. For example:

  # pvcreate /dev/sdb1
  # mdadm --create md8 --metadata=0.90 --chunk=64 --level=0 \
        --raid-devices=2 /dev/sdb1 /dev/sdb2

  # wipefs -a /dev/sdb1

will delete the linux raid signature only.

Reported-by: Milan Broz <mbroz@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

tests: add tests for mount --make-* stuff

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

mountpoint: fix a minor bug with 0:0 devno

zhi@debian-lizhi:~$ mountpoint /sys
/sys is not a mountpoint
zhi@debian-lizhi:~$

The problem is in mountpoint.c, function dir_to_device(), which uses 0 as an
error return value, but for sysfs 0 is its device number:

zhi@debian-lizhi:~$ cat /proc/self/mountinfo
14 19 0:0 / /sys rw,nosuid,nodev,noexec,relatime - sysfs none rw
15 19 0:3 / /proc rw,nosuid,nodev,noexec,relatime - proc none rw

My change is on dir_to_device, seperating device number from return value.

Signed-off-by: Zhi Li <lizhi1215@gmail.com>

lslocks: add --notruncate, minor fixes in man page

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

lslocks: add lslocks.8 man page

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

lslocks: new command

The new lslocks(8) program is meant to replace the deprecated lslk(8). It is
designed for simplicity and removes unnecessary Unix legacy outputs and
options:

- Don't output inode number, whence and maj:min device numbers.
- Don't provide nonblocking syscall options stat(2) and readlink(2)
- Remove lslk's alternate default kernel name list file path (-k)

The option to use nonblocking calls was previously intended for NFS partitions;
however this should be transparent to utility programs considering that
timeouts can occur generically (fuse - sshfs, NFS, netdevs, etc).

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

include: add /proc/locks path to pathnames

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

build-sys: s/ruman/ruman1/

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

docs: do not overwrite ddate.1 manual

The following warning meant either english manual page overwrote ru
version, or otherway around.  Neither way that not intented.  This
patch will change ru version of the manual page to be installed to
<prefix><mandir>/ru/man1 directory.

man/ru/Makefile.am:4: warning: mandir was already defined in condition TRUE, which includes condition BUILD_DDATE ...
configure.ac:1: ... `mandir' previously defined here

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

mount: (new) add 'internal-only(i)' to non-root allowed options

Commit ce433404 introduced the change, which causes something like mount -i
<user-mount-path> to break which worked well before util-linux-2.21, now it
gives message 'mount: only root can use "--internal-only" option' when that
shouldn't be the case when it is already in fstab.

Signed-off-by: Raghavendra D Prabhu <rprabhu@wnohang.net>

build-sys: add a crosscompile path for scanf_cv_alloc_modifier

This allows to building the package cross again. Since the return value
is not used I guess there is little loss.

Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>

tests: make output and diff dirs usage more robust

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

build-sys: enhance readability of the autotools files

Several horizontal lists are turned to vertical, and sorted to
alphabetical order. Additionally spaces are converted to tabs where
ever possible.

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

build-sys: remove unused function and header checks

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

login: support crazy shadow-utils syntax in login.defs

standard syntax for FOO=data:

ENV_FOO   data

additional syntax:

ENV_FOO  FOO=data

Reported-by: Colin Guthrie colin@mageia.org
Signed-off-by: Karel Zak <kzak@redhat.com>

test: test 'none' source for mount(8)

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

libmount: allow empty source for mount(2) syscall

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

libmount: use mount.<type> -s for NFS only

Unfortunately, it seems that for example mount.cifs don't care about
the API, so we need exception like the original mount(8).

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

mount: (new) cleanup mount -a return codes

New return codes:

  0 : all mounted (or all ignored)
 64 : some mounted, some failed
 32 : all failed

Note that already mounted or ignored (filtered out by -t or -O)
filesystems don't affect the final return code.

The original mount(8) returns 0 instead of 64, so the situation
"some mounted, some failed" cannot be detected.

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

libmount: canonicalize targets from fstab on mount -a

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

libmount: optimize for paths with tailing slash

mnt_table_find_target() canonicalizes paths by libmnt cache, but it's
overkill if the difference between paths is tailing slash only.

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

lib/canonicalize: always remove tailing slash

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

tests: test non-canonical paths in fstab

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

libmount: canonicalize all paths from (fs)tab

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

more: fix typos

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

sys-utils: fix typos

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

misc-utils: fix typos

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

losetup: add --detach-all to the list of options

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

docs: corrections to FSF license files, and postal address

The COPYING and Documentation/licenses/COPYING* files are being
replaced by files from GNU web site.

http://www.gnu.org/licenses/gpl-2.0.txt
http://www.gnu.org/licenses/lgpl-2.1.txt

Postal addresses to FSF in other files are updated to match with the
address in license files.

Reference: http://lists.gnu.org/archive/html/freefont-announce/2005-04/msg00001.html
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: don't create empty man/ru directory

Reported-by: Tom Gundersen <teg@jklm.no>
Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: release++ (v2.21)

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

docs: update v2.21-ReleaseNotes

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

docs: update AUTHORS file

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

po: merge changes

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

po: update uk.po (from translationproject.org)

po: update pl.po (from translationproject.org)

po: update nl.po (from translationproject.org)

po: update fr.po (from translationproject.org)

po: update cs.po (from translationproject.org)

umount: respect fs search path

This brings search path support to umount to match existing behavior
in fsck and mount.

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

fdisk: update information about GNU Parted in fdisk.8

The support for resizing partitions and filesystems has been
removed from parted since version 3.0.

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

mount: distinguish between MS_COMMENT flags (netdev, nofail, ...)

Since util-linux 2.12h (year 2004) there is only one flag for all
dummy options like _netdev or nofail. Unfortunately it means that when
mount(8) composes the final mount options string for mtab (or for
mount.<type> helpers) the string is generated incorrectly.

Reported-by: Dave Reisner <d@falconindy.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

mount: clear flags before options parsing

for example "mount -a" calls the mount options parser more than once

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

tests: add wall to build-sys tests

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

libmount: clean up context docs

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

docs: add note about mount(8) options to TODO

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

lib/strutils: move array bounds check in string_to_idarray() to appropriate place.

string_to_idarray() will incorrectly exit with an error when the last element
of the passed in array gets filled. However it should only exit with an error
if there is more input. To fix this move the array bounds check.

Signed-off-by: Chandan B Rajenda <chandan@linux.vnet.ibm.com>
Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

libblkid: add BLKID_PARTS_MAGIC to blkid_do_wipe() docs

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

docs: update copyright years

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

libmount: cleanup docs

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

libblkid: update docs

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

libmount: remove teaser proto for mnt_table_get_name

This function was never implemented, but the proto has existed since the
inception of the library. As the libmnt_table struct doesn't include a
member necessary to carry the name of the table, simply discard this
prototype. If ever a need to break ABI comes up, perhaps this could be
revisited.

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

docs: trivial documentation fixes and improvements

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

cytune.8: fix formatting

W: manual-page-warning /usr/share/man/man8/cytune.8.gz 178: warning:
macro `chars',' not defined (possibly missing space after `ch')

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

build-sys: fix building of wall

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

mkfs.bfs: add setlocale

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

docs: add note about losetup to TODO

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

docs: add note about fdisk to TODO

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

tests: make minix tests more robust

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

unshare: add note about MS_SHARED to unshare.1

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

build-sys: add tools/ to EXTRA_DIST

Reported-by: Petr Uzel <petr.uzel@suse.cz>
Signed-off-by: Karel Zak <kzak@redhat.com>

lib/fsprobe: mar this internal API as deprecated

The fsprobe has been added many years ago to provide unified API for

- built-in libblkid
- e2fsprogs libblkid
- udev libvolume_id

these all is unnecessary as we strictly depends on built-in libblkid.

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

login: add LOGIN_RETRIES, cleanup retries check code

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

setsid: add "+" to getopt_long()

otherwise setsid wants to interpret <program> arguments

$ setsid sh -i
setsid: invalid option -- 'i'

Usage:
 setsid [options] <program> [arguments ...]

Signed-off-by: Harald Hoyer <harald@redhat.com>

tunelp: free allocation, use xalloc

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

switch_root: close open file after usage [cppcheck]

[sys-utils/switch_root.c:159]: (error) Resource leak: cfd

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

chcpu: use appropriate variable type [cppcheck]

[sys-utils/chcpu.c:251]: (style) Checking if unsigned variable 'maxcpus' is less than zero.

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

taskset: use appropriate variable type [cppcheck]

[schedutils/taskset.c:182]: (style) Checking if unsigned variable 'ncpus' is less than zero.

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

libblkid: verify arroy bound before reference [cppcheck]

[libblkid/src/read.c:428]: (style) Array index end is used before limits check

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

libblkid: allow return value to indicate error [cppcheck]

[libblkid/src/probe.c:1207]: (style) Checking if unsigned variable 'len' is less than zero.

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

libblkid: clarify operation order [cppcheck]

[libblkid/src/cache.c:265]: (style) Suspicious condition (assignment+comparison), it can be clarified with parentheses

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

sfdisk: free variable which got the allocation [cppcheck]

[fdisk/sfdisk.c:358]: (error) Memory leak: ss

[kzak@redhat.com: - free after error]

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

fdisk: remove redundant declaration [cppcheck]

fdisk.h:116:12: warning: redundant redeclaration of 'valid_part_table_flag' [-Wredundant-decls]

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

raw: remove unnecessary return [cppcheck]

[disk-utils/raw.c:171]: (style) Consecutive return, break, continue, goto or throw statements are unnecessary.

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

mkfs.cramfs: check blocksize user input [cppcheck]

Enhance user input checking and error messaging, while fixing
cppcheck warning.

[disk-utils/mkfs.cramfs.c:729]: (style) Checking if unsigned variable 'blksize' is less than zero.

[kzak@redhat.com: fix coding style]

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

mkfs.cramfs: close open file after usage [cppcheck]

[disk-utils/mkfs.cramfs.c:913]: (error) Resource leak: fd

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

libblkid: improve reiserfs and btrfs min. size limits

Reported-by: Rogier Goossens <goossens.rogier@gmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

flock: add missing word »be« to flock.1

flock: fix flock.1 grammar by removing »s«

tools: use gpg-aggent in ko-release-gen

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

build-sys: release++ (v2.21-rc2)

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

docs: update v2.21-ReleaseNotes

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

libblkid: remove >> 9 from GPT code

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

docs: update AUTHORS file

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

po: merge changes

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

tests: add --memcheck to

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

agetty: check localtime() result

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