libfdisk: fix fdisk_script_get_table()

Make sure we never return NULL and we reuse the table in code.

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

libfdisk: (docs) add notes about fdisk_enable_wipe()

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

libfdisk: add fdisk_script_set_table()

This small change improves possibility to modify by script described
PT and use script API in another tools as primary way to create
partitions.

All you need is to compose script by fdisk_script_set_header() and
fdisk_script_set_table() and than apply by fdisk_apply_script().

Requested-by: Lennart Poettering <lennart@poettering.net>
Signed-off-by: Karel Zak <kzak@redhat.com>

lscpu: add a new columns to --cache

References: https://www.kernel.org/doc/Documentation/ABI/testing/sysfs-devices-system-cpu
Signed-off-by: Karel Zak <kzak@redhat.com>

docs: we have 2019 already

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

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

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

docs: update v2.34-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 zh_CN.po (from translationproject.org)

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

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

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

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

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

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

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

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

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

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

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

tests: update build-sys output

It seems, no more libgcc_s...

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

libsmartcols: fix docs

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

libmount: fix docs typo

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

rev: be careful with close()

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

findmnt: (verify) ignore passno for XFS

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

lib/sysfs: fix reference counting for parent

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

cfdisk: free libfdisk items

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

fstrim: update man page, reuse libmnt_iter

* add info about read-only to the man page
* don't be systemd specific, people aso use crond
* reuse libmnt_iter

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

cfdisk: simplify code

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

fstrim: properly de-duplicate fstrim -A

fstab can contain tag based mounts. De-duplication by source has to be
done after resolving the full source path.

Perform the table iteration twice. First time, prepare for
de-duplication, second time perform the TRIM itself.

Signed-off-by: Stanislav Brabec <sbrabec@suse.cz>

fstrim -a/-A: Skip read-only volumes

Calling TRIM on some read-only volumes can fail with:
fstrim: /win: FITRIM ioctl failed: Bad file descriptor

Skipping all read-only mounts seems to be safe and logical strategy.

Fixes opensuse#1106214.

Signed-off-by: Stanislav Brabec <sbrabec@suse.cz>

fstrim: Fix fstrim_all() comment

"convert LABEL=" does not happens in mnt_fs_get_source(), but later in
mnt_resolve_spec(). To make this more clean, move the comment before this
chunk of code.

Signed-off-by: Stanislav Brabec <sbrabec@suse.cz>

lib/canonicalize: fix compiler warning [-Wsign-compare]

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

libmount: (docs) add reference to v2.33 and v2.34

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

libsmartcols: (docs) add reference to v2.33 and v2.34

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

libfdisk: (docs) add reference to v2.33

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

umount: be more strict about --all

If you by accident execute

umount ls -al /mnt/

then umount --all is executed and another arguments silently ignored.
It seems better to be more strict in this case.

Reported-by: Harald Dunkel <harald.dunkel@aixigo.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

wall: make sure ut_line is not empty

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

libblkid: fix detection of dm-integrity superblock version

Kernel 5.2 can use superblock version 3 for dm-integrity.
Let's remove the explicit version check to be compatible
with future extensions.

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

hardlink: fix bad formatting in hardlink.1

* fix regex\f to regex\fR (issue #805)
* replace \fP with \fR to be consistent with our another man pages

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

docs: add lscpu --caches to ReleaseNotes

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

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

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

docs: update v2.34-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 hr.po (from translationproject.org)

libmount: add selinuxfs between pseudo filesystems

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

libmount: add bpf between pseudo filesystems

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

findmnt: fix filters use on --list

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

tests: add asan build-sys test

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

fdisk: add note about -S and -H

It's seems people still follow old advices for SSD/4K devices. Let's
make CHS deprecation more visible at the begin of the man page...

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

ul: make sure buffers are zeroized [coverity scan]

column: make code more robust [coverity scan]

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

setpriv: fix memory leak in local scope [coverity scan]

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

setarch: avoid NULL dereference [coverity check]

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

rtcwake: terminate mode and excl[]

* terminate string read from /proc
* terminate array for exclusive options

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

build-sys: add info about ASAN

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

renice: make code more readable for static analyzer [coverity scan]

It seems coverity and clag have no clue about relation between argv[]
and argc. Let's make code more readable for them...

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

readprofile: be more explicit with used types [cppcheck]

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

lscpu: check scols_line_set_data() return value [coverity scan]

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

lscpu: make lookup() use more consistent [coverity scan]

We usually check lookup() return value. Let's do it in this case too.
It seems static analyzers will be happy with consistent code.

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

mesg: avoid 'ttyname failed: Success' message

The ttyname(3) can fail to access /dev/ path, and that will cause function
to fail without setting errno value with result of rather confusing error
message.  Lets start setting stdin permission via /proc when this happens as
a go-around, with hope kernel following symlink does not fail.  Ok, noted,
that hopes of symlink follow working are pretty slim.

Based on patch from Sami Kerola <kerolasa@iki.fi>.

Reference: https://github.com/lxc/lxd/issues/1724
Signed-off-by: Karel Zak <kzak@redhat.com>

lib/ttyutils: introduce get_terminal_stdfd()

Let's use separate function to detect what is the current terminal.

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

libblkid: remove unneeded fields from struct bcache_super_block

This started as fix to following warning.

    libblkid/src/superblocks/bcache.c:33:3: warning: ISO C99 doesn’t
    support unnamed structs/unions [-Wc99-c11-compat]

But when sent to review Karel told out rather than getting rid of warnings
lets get rid of everything unnecessary.

[kzak@redhat.com: - use original struct member names rather than 'padN']

Reference: https://github.com/karelzak/util-linux/pull/802
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
Signed-off-by: Karel Zak <kzak@redhat.com>

login: retire use of __FUNCTION__ macro

This fixes following warning about deprecatd __FUNCTION__ that is another
name for __func__ since GCC (pre 2.95, released July 1999).

    login-utils/login.c:860:10: warning: ISO C does not support
    ‘__FUNCTION__’ predefined identifier [-Wpedantic]

Co-Author: Sami Kerola <kerolasa@iki.fi>
Reference: https://c0x.coding-guidelines.com/6.4.2.2.html
Reference: https://github.com/karelzak/util-linux/pull/802
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
Signed-off-by: Karel Zak <kzak@redhat.com>

ipcs: check return value when read from /proc [coverity scan]

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

hwclock: don't use uninitialized value [coverity scan]

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

fallocate: make posix_fadvise() use more readable for analyzers

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

dmesg: make strtok() use more robust

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

blkzone: remove never read value

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

whereis: make subdirs scan more robust

* call strchr() only once
* avoid things like strcat(buf, strchr(dir, '*') + 1)
* make it more readable
* improve debug messages

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

lsblk: make sure __process_one_device() has proper arguments [coverity scan]

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

lsblk: check ul_path_scanf() return value [coverity scan]

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

lsblk: check stat() return code [coverity scan]

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

logger: make code more robust for static analyzer [clang scan]

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

hardlink: avoid uninitialized variables [clang scan]

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

libfdisk: assert if self_pte() returns NULL

The self_pte() can return NULL if partitions array is not large enough,
but that should also be impossible and definitely a bug.

libfdisk/src/dos.c:984:8: warning: potential null pointer dereference [-Wnull-dereference]
libfdisk/src/dos.c:1031:8: warning: potential null pointer dereference [-Wnull-dereference]

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

libfdisk: (bsd) improve checksum calculation [-Waddress-of-packed-member]

Let's keep compilers and static analyzers happy. The idea is to use
memcpy() to copy from buffer to variable and use all label as unsigned
char rather than vectorize by unsigned short.

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

libblkid: (silicon raid) improve checksum calculation [-Waddress-of-packed-member]

Let's keep compilers and static analyzers happy. The idea is to use
memcpy() to copy from buffer to variable and use all label as unsigned
char rather than vectorize by uint16_t.

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

libfdisk: (sgi) improve checksum calculation [-Waddress-of-packed-member]

Let's keep compilers and static analyzers happy. The idea is to use
memcpy() to copy from buffer to variable and use all label as unsigned
char rather than vectorize by uint32_t.

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

include/c: use __has_attribute

The __has macros are more portable (supported by gcc as well as
clang).

The old __GNUC_PREREQ is deprecated and it should be removed in
future. (Well, the __has macros are supported since gcc 5, so we
should be patient as some old stable distros (e.g. RHEL 7) use gcc
4.x).

This patch helps clang to correctly analyze our xalloc.h stuff.

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

bash-completion: add hardlink completion

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

bash-completion: update options before release

blkid: 5e91d5dd716ebc6144bcb0cabb0ec847a678be9e --no-part-details
lsblk: dc4662f0e755929de9cbe98b1b343b492fd620f4 --dedup
lsblk: 0bd05f5ee4876ffd13c98acd56c2bff9971f28f1 --merge
lscpu: 0e86bc8406959d0d56194d4f404b780e6b70caa4 --caches
lscpu: 2011528bce58dd31d5ee0287f2367da1f2d6cfa9 --bytes

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

lib/ismounted: use xstrncpy()

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

zramctl: use xstrncpy()

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

whereis: use xstrncpy()

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

vipw: use xstrncpy()

It's already used by vipw, so use it everywhere.

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

lib/path: use xstrncpy()

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

lib/sysfs: use xstrncpy()

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

losetup: use xstrncpy()

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

sfdisk: use xstrcpy()

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

lib/colors: remove unnecessary goto

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

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

* '2019wk19' of https://github.com/kerolasa/util-linux:
  lib/colors: remove redundant if statement
  wipefs: fix variable / function shadowing [cppcheck]
  sulogin: fix variable / function shadowing [cppcheck]
  lscpu: remove redundant condition check [cppcheck]
  libmount: avoid possible null pointer dereference [cppcheck]
  lib/mangle: fix possible null pointer dereference [cppcheck]
  sfdisk: remove unnecessary size check [cppcheck]

Merge branch 'lscpu_midr_updates' of https://github.com/jlinton/util-linux

* 'lscpu_midr_updates' of https://github.com/jlinton/util-linux:
  lscpu: Add additional aarch64 models

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

* 'lscpu-another' of https://github.com/kerolasa/util-linux:
  lscpu: fix floating point exception
  include/strutils: fix potential null pointer dereference
  lscpu: remove extra space from field key name

lib/colors: remove redundant if statement

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

wipefs: fix variable / function shadowing [cppcheck]

[misc-utils/wipefs.c:636] -> [misc-utils/wipefs.c:310]: (style) Local
variable usage shadows outer function

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

sulogin: fix variable / function shadowing [cppcheck]

[login-utils/sulogin.c:398] -> [login-utils/sulogin.c:171]: (style) Local
variable set shadows outer function
[login-utils/sulogin.c:398] -> [login-utils/sulogin.c:830]: (style) Local
variable set shadows outer function

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

lscpu: remove redundant condition check [cppcheck]

[sys-utils/lscpu.c:1783] -> [sys-utils/lscpu.c:1785]: (warning) Either the
condition 'desc' is redundant or there is possible null pointer dereference: desc.
[sys-utils/lscpu.c:1840] -> [sys-utils/lscpu.c:1842]: (warning) Either the
condition 'desc' is redundant or there is possible null pointer dereference: desc.

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