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>

lib/path: improve debug message on redirect

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

fstrim: add --dry-run

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

fstrim: use new ul_path_* API, improve sysfs context usage

* improve has_discard() function to initialize sysfs context only once
* use new ul_path_* API

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

lib/sysfs: fix sysfs_blkdev_get_wholedisk()

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

lib/path: simplify debug message

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

lscpu: use new ul_path_* API

* use ul_path_* API for /sys/devices/system/cpu paths
* use ul_path_* API for /proc
* rename is_compatible() to is_devtree_compatible() as it works
  with the devices tree only

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

lib/path: add ul_prefix_fopen(), improve cpuset funcs

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

lib/path: don't use extra '/'

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

ipcs: use new ul_path_* API

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

lib/path: add more debug messages

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

chcpu: use new ul_path_* API

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

ctrlaltdel: use new ul_path_* API

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

lib/path: make ul_path_read_ usable with NULL handler

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

rfkill: don't use lib/path

It seems like crazy overkill for this trivial purpose.

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

chmem: use new ul_path_* API

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

lsmem: use new ul_path_* API

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

lib/path: add ul_path_read_buffer()

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

lib/path: add ul_path_get_abspath()

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

zramctl: use new ul_path_* API

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

lib/sysfs: fix ul_path_read_string() usage

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

libfdisk: use new ul_path_* API

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

libblkid: use new ul_path_* API

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

lib/sysfs: add sysfs_blkdev_get_parent()

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

lib/sysfs: make possible to call sysfs_blkdev_deinit_path() in loop

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

lib/path: fix read string to be backwardly compatible

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

lib/path lib/sysfs: add debug

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

lib/loopdev: remove obsolete macro

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

lib/loopdev: use new ul_path_* API

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

lib/sysfs: add ul_new_sysfs_path() shortcut

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

lib/sysfs: new implementation

* reuse ul_path_* API

* allow to use prefix for sysfs paths, so we can use snapshots from
  sysfs for regression tests

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