Merge branch 'patch-1' of https://github.com/pmarillo/util-linux

* 'patch-1' of https://github.com/pmarillo/util-linux:
  Fix translation typo

hwclock: rely on kernel for RTC_UIE_ON on Aplha and Sparc

It's kernel business to return EINVAL for unsupported RTC_UIE_ON. We
(userspace) should not make decisions about things that we do not
control.

If kernel is wrong then fix the kernel, don't hide the problem by
crazy ifdefs in userspace.

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

mesg: do not print 'ttyname failed' message unless --verbose requested

Lots of people are confused why mesg(1) is priting this message.  Usual
cause seems to be an uninteractive shell trying to turn running 'mesg n'
from a /root/.profile where command invocation is by default on debian based
systems.  This might be rare case when failing silently is better.

[kzak@redhat.com: - add note to the man page
                  - fix if(isatty()) logic]

Reference: https://www.google.com/search?q=mesg+ttyname+failed
Review: https://marc.info/?l=util-linux-ng&m=153319988631233&w=2
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
Signed-off-by: Karel Zak <kzak@redhat.com>

Fix translation typo

tests: update lscpu tests

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

fdisk: fix error message on 'd' command

Command (m for help): d
No partition is defined yet!
Could not delete partition 93840461057817    <--- !

Don't print 'Could not delete..' if no partition is defined yet.

Addresses: https://github.com/karelzak/util-linux/issues/667
Signed-off-by: Karel Zak <kzak@redhat.com>

umount: add --quiet option

The command umount supports things like --all-targets and --recursive
to umount all nodes in specified tree. Sometimes it makes sense to
aggressively use wildcards like /dev/sdb* and in this case --quiet
seems like a good choice

umount --quiet --all-targets /dev/sdb*

to suppress 'not mounted' error messages. The new option suppress only
these error messages and nothing else.

Addresses: https://github.com/karelzak/util-linux/issues/672
Signed-off-by: Karel Zak <kzak@redhat.com>

lscpu: add 'Address sizes'

Addresses: https://github.com/karelzak/util-linux/issues/674
Signed-off-by: Karel Zak <kzak@redhat.com>

fstrim: add -A|--fstab to trim according to /etc/fstab

Addresses: https://github.com/karelzak/util-linux/issues/673
Signed-off-by: Karel Zak <kzak@redhat.com>

fstrim: add control struct

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

tests: update fdisk dialogs

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

libfdisk: Accept negative numbers for last sector input

[kzak@redhat.com: - add note to the man page
                  - add '-' to the dialog query
                  - cleanup functions names and libfdisk.sym]

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

hwclock: remove ntpd from man page

It seems better to use generic "NTP daemon" in the man page than
points to specific ntpd(1) implementation as some distros use for
example chronyd(1) rather than old ntpd(1).

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

lscpu: fix resource leak [coverity scan]

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

losetup: fix mem leak, improve code [coverity scan]

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

libblkid: use xstrncpy() for PT id [coverity scan]

It's probably unnecessary change as blkid_parttable_set_id() is used
by MBR code and we use proper buffer size there, but better safe than
sorry.

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

docs: add note about --disable-all-programs

Addresses: https://github.com/karelzak/util-linux/issues/666
Signed-off-by: Karel Zak <kzak@redhat.com>

cfdisk: fix compiler warnings [-Wcast-qual]

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

blkid: fix compiler warnings [-Wcast-qual]

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

login: use const qualifier for username from PAM or struct passwd [-Wcast-qual]

It seems more robust to use 'const' qualifier for username if this
variable points to external resources like PAM or struct passwd. The
patch introduces new variable cmd_username for username specified on
login(1) command line.

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

libmount: (utils) fix compiler warnings [-Wcast-qual]

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

lib/colors:: fix compiler warnings [-Wcast-qual]

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

setterm: fix compiler warnings [-Wcast-qual]

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

setterm: fix compiler warnings [-Wcast-qual]

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

swapon: fix compiler warnings [-Wcast-qual]

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

mkswap: fix compiler warnings [-Wcast-qual]

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

libfdisk: fix compiler warnings [-Wcast-qual]

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

libmount: fix compiler warning [-Wcast-qual]

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

lib/loopdev: fix compiler warning [-Wcast-qual]

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

lib/mangle: const quialifier cleanup

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

include/list, include/c: update and container_of()

* update container_of() to version without "const" qualifier

* use container_of() in list_entry()

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

libblkid: const qualifier cleanup

* use "const" in functions where we don't modify arguments

* use "const" when cast from const buffer to any struct (superblock etc.)

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

include/all-io: const cleanup

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

libmount: cleanup const in umount code

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

lib/color-names: const cleanup

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

libmount: add missing const

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

libmount: use const char in swaps parser

The parser does not modify the string, so let's keep it "const char".

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

libmount: use const char in mountinfo parser

The parser does not modify the string, so let's keep it "const char".

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

libmount: use const char in fstab parser

The parser does not modify the string, so let's keep it "const char".

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

lib/strutils: follow const in parse_size()

* don't cast from char to const char

* don't share endptr from strtoxxx() with rest of the code
  as the end pointer is char, but code works with const chars

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

include/strutils: remove unnecessary cast

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

hwclock: fix warning

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

build-sys: add -Wdiscarded-qualifiers

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

mount: add details about mtab and --namespace to mount.8

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

libmount: add since tag for new functions

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

lsblk: add note about --list -I <maj,...>

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

hwclock: add --delay <seconds>

* add command line option --delay <seconds>

* read RTC type from /sys/class/rtc/rtc<N>/name

* default to 0.5 (500ms) for rtc_cmos or when RTC type is impossible
  determine; otherwise delay is 0.

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

build-sys: add -Wno-cast-function-type for python

 libmount/python/pylibmount.c:158:19: warning: cast between incompatible function types
   from ‘PyObject * (*)(PyObject *)’ {aka ‘struct _object * (*)(struct _object *)’}
     to ‘PyObject * (*)(PyObject *, PyObject *)’ {aka ‘struct _object * (*)(struct _object *, struct _object *)’}
   [-Wcast-function-type]

This is generic problem in all Python C code and gcc v8. The another
possible (and probably more correct) way is to add unused argument to
all API functions. Unfortunately, this solution is pretty invasive. The
question is if gcc is not too paranoid in this case.

For more details see https://bugs.python.org/issue33012. It seems
Python guys also prefer CFLAGS modification for now.

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

libfdisk: fix compiler warning [-Wmaybe-uninitialized]

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

lslogins: remove redundant condition

[util-linux-2.32.1/login-utils/lslogins.c:601]: (style) Redundant condition: If '!EXPR', the comparison 'EXPR != '$'' is always true.

Reported-by: David Binderman <dcb314@hotmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

lib/strutils: remove redundant condition

[util-linux-2.32.1/lib/strutils.c:122]: (style) Redundant condition: If 'EXPR == '0'', the comparison 'EXPR' is always true.

Reported-by: David Binderman <dcb314@hotmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

mkfs.cramfs: remove redundant condition

util-linux-2.32.1/disk-utils/mkfs.cramfs.c:362]: (style) Redundant condition: If 'EXPR >= 16777216', the comparison 'EXPR' is always true.

Reported-by: David Binderman <dcb314@hotmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

lib/pager: fix compiler warning [-Wrestrict]

lib/pager.c: In function ‘pager_preexec’:
lib/pager.c:148:12: warning: passing argument 2 to restrict-qualified parameter aliases with argument 4 [-Wrestrict]

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

docs: add fdisk to TODO

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

docs: add hwclock to TODO

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

libblkid: Check for a secondary LUKS2 header.

This patch adds search for a secondary LUKS2 header,
if the primary one is corrupted.

This patch is primarily needed for wipefs that should wipe
both signatures (otherwise LUKS2 header recovery can use
secondary header and revert wipefs action).

Signed-off-by: Milan Broz <gmazyland@gmail.com>

* break up large strings for PySys_WriteStdout()

tests: add mountinfo with empty source

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

libmount: (mountinfo) parse empty strings in source

The source of a mount in /proc/self/mountinfo can unfortunately be an
empty string. Before this patch, 'mount' and 'mountpoint' fail as
following:

  $ sudo mount -t tmpfs "" /tmp/bb
  $ mount
  mount: /proc/self/mountinfo: parse error at line 64 -- ignored
  $ mountpoint /tmp/bb
  /tmp/bb is not a mountpoint

This patch fixes the parsing. It is unfortunately more complex because
sscanf() does not handle fields with empty strings easily.

Other projects have their own parser for mountinfo and have similar
issues. I know of runc and runtime-tools (I'll send a patch for those
two) but there are probably others.

Signed-off-by: Alban Crequy <alban@kinvolk.io>

lsblk: add PATH column

$ lsblk -o+PATH
NAME        MAJ:MIN RM   SIZE RO TYPE MOUNTPOINT    PATH
sda           8:0    0 223.6G  0 disk               /dev/sda
├─sda1        8:1    0   200M  0 part /boot/efi     /dev/sda1
├─sda2        8:2    0   200M  0 part /boot         /dev/sda2
├─sda3        8:3    0 130.3G  0 part               /dev/sda3
├─sda4        8:4    0    50G  0 part /             /dev/sda4
├─sda5        8:5    0  35.1G  0 part               /dev/sda5
└─sda6        8:6    0   7.8G  0 part               /dev/sda6

Requested-by: Marcel Partap <mpartap@gmx.net>
Signed-off-by: Karel Zak <kzak@redhat.com>

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

* 'master' of https://github.com/hjmallon/util-linux:
  libblkid: Add more APFS checking to avoid false positives

logger: Define SD_JOURNAL_SUPPRESS_LOCATION.

The normal journald functions add the location in the C source code files to
the log messages. This is nice for a big C based project, but logger is used in
scripts so it would be more useful to let users specify the location in the
script by adding the CODE_FUNC, CODE_FILE and CODE_FILE fields to the log
message.

It is already possible to do this, but it will result in two versions of these
fields: one for the location in logger.c and one for the location in the
script.

libblkid: Add more APFS checking to avoid false positives

libfdisk: reduce number of asprintf() calls, check return value

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

libfdisk: Fix multipath partition seperators for user-friendly names

The current code assumes "-part" is the only partition sepereator
but this is not true for some distros.

For example in Ubuntu 18.04 fdisk does not print the correct names for
mpatha:

~# ls -l /dev/mapper/mpatha*
lrwxrwxrwx 1 root root 7 Feb  1 04:39 /dev/mapper/mpatha -> ../dm-0
lrwxrwxrwx 1 root root 7 Feb  1 04:38 /dev/mapper/mpatha1 -> ../dm-4
lrwxrwxrwx 1 root root 7 Feb  1 04:38 /dev/mapper/mpatha2 -> ../dm-5
lrwxrwxrwx 1 root root 7 Feb  1 04:38 /dev/mapper/mpatha3 -> ../dm-6

~# fdisk -l /dev/mapper/mpatha
Device                   Boot     Start        End   Sectors  Size Id Type
/dev/mapper/mpatha-part1           2048  419432447 419430400  200G 83 Linux
/dev/mapper/mpatha-part2      419432448  838862847 419430400  200G 83 Linux
/dev/mapper/mpatha-part3      838862848 1258291199 419428352  200G 83 Linux

Instead of assuming a partition seperator of "-part" this patch uses
access to check the file system for a partition seperator of "p" or
the absense of a partition seperator. If neither of these work the patch
defaults to "-part" like we had before this patch.

libfdisk: use \x<hex> to read/write partition name

Addresses: https://github.com/karelzak/util-linux/issues/656
Signed-off-by: Karel Zak <kzak@redhat.com>

fallocate: remove unwanted debug message

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

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

* 'master' of https://github.com/hjmallon/util-linux:
  libblkid: Add very basic APFS recognition

Merge branch 'quick-fix-20180625' of https://github.com/sjitech/util-linux

* 'quick-fix-20180625' of https://github.com/sjitech/util-linux:
  exec_shell: prevent ".: applet not found" error when SHELL env is not set.

Merge branch 'fixes' of https://github.com/rudimeier/util-linux

* 'fixes' of https://github.com/rudimeier/util-linux:
  wipefs: add missing ifdef
  lib/path: remove unused expression
  ipcs,chmem: fix access() usage

fallocate: add man page note about --collapse-range granularity

Reported-by: Radka Skvarilova <rskvaril@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: Add very basic APFS recognition

wipefs: add missing ifdef

Seen on OSX:
misc-utils/wipefs.c:822:5: warning: implicit declaration of function 'rereadpt' is invalid in C99 [-Wimplicit-function-declaration]
                                rereadpt(fd, devname);

Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

lib/path: remove unused expression

 error: expression result unused [-Werror,-Wunused-value]
                        prefix = optarg, "failed to parse range start";
                                         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

ipcs,chmem: fix access() usage

Some mistakes happened lately when switching from path_exist()
to ul_path_access(). See f09a98de and 8ca31279.

This caused ipcs test failures when running i386 binaries on x86_64
hosts, because the syscall fallback was always used. That's why I
reviewed all similar changes and found another one in chmem.

Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

exec_shell: prevent ".: applet not found" error when SHELL env is not set.

When SHELL env is not set, it cause xstrdup(NULL) be executed, and result in weird error message ".: applet not found" instead of /bin/sh being used.

docs: update TODO

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

include/path: make sure PATH_MAX is defined

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

Merge branch 'more' of https://github.com/kerolasa/util-linux

* 'more' of https://github.com/kerolasa/util-linux:
  more: remove function like preprocessor defines
  more: remove global variables, add struct more_control
  more: move couple functions

choom: use new ul_path_* API

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

lib/path: allow dir-path formatting

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

tests: add simple LVM lsblk dump

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

lsblk: fix ul_path_read_string() usage

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

tests: add simple ATA and NVME lsblk dump

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

tests: add lsblk test script

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

lsblk: fix COL_SCHED

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

tests: add simple & stupid script to generate lsblk /sys and /proc dump

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

lsblk: document --sysroot

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

lsblk: fix sysfs_devno_to_devname() usage

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

lsblk: follow --sysroot when read mountpoints

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

lsblk: use prefix for DM name

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

lsblk: add --sysroot

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

docs: add note about lsblk to TODO

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

lsblk: use new ul_path_* API

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

lib/sysfs: cleanup function names

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

blockdev: use new ul_path_* API

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

partx: use new ul_path_* API

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

resizepart: use new ul_path_* API

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

blkzone: use new ul_path_* API

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

fstrim: use new ul_path_* API

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