lsfd: don't define a local variable in the middle of a block

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd.1.doc: use delimited literal block notation in the EXAMPLE section

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

findmnt: filter entries before add to the tree

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

Merge branch 'fix_syscall_usage' of https://github.com/dengraf/util-linux

* 'fix_syscall_usage' of https://github.com/dengraf/util-linux:
  Do not include sys/syscall.h if not available

findmnt: make sure all entries are in tree output

For example /proc/self/mountinfo when mounted in chroot environment
does not contain all nodes and parent-child relation is not always
possible to create.

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

Do not include sys/syscall.h if not available

Some platforms do not provide sys/syscall.h. The configure script already
checks for the existance of the file. Include sys/syscall.h only in case
HAVE_SYS_SYSCALL_H has been set.

libmount: (--all) continue although /proc is not mounted

Now 'mount --all' ends with error if /proc is not mounted and there is
some other entry before /proc in fstab. This commit improves this
situation and ignores all mount table related errors if the table is
empty.

This is important for situation when there is for example "/" as the
first line in fstab.

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

lsfd.1.adoc: remove redundant parenthesis from the examples

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd.1.adoc: add caution about the CLI stability

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd.1.adoc: fix a typo

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsblk: (adoc) improve --all description

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

mount: (adoc) add hint about /proc and /sys to --all description

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

fsck: clear SIGCHLD inherited setting

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

more: clear SIGCHLD inherited setting

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

unshare: clear SIGCHLD inherited setting

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

nsenter: clear SIGCHLD inherited setting

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

vipw: improve child error handling

Always set SIGCHLD handler to default, even if the caller of vipw has
set SIGCHLD to ignore. If SIGCHLD is ignored no zombie processes would
be created, which in turn could mean that kill is called with an already
recycled pid.

Also improved error message if child process fails.

Proof of Concept:

1. Compile nochld:
--
 #include <signal.h>
 #include <unistd.h>
 int main(void) {
char *argv[] = { "vipw", NULL };
signal(SIGCHLD, SIG_IGN);
execvp("vipw", argv);
return 1;
 }
--
2. Run nochld
3. Suspend child vi, which suspends vipw too:
`kill -STOP childpid`
4. Kill vi:
`kill -9 childpid`
5. You can see with ps that childpid is no zombie but disappeared
6. Bring vipw back into foreground
`fg`
7. See misleading warning message

You will get an improperly formatted warning message. Also the wake up
kill call sent SIGCONT to "childpid" which could have been assigned
to another process already.

This is definitely not a vulnerability. It would take super user
operations, at which point an attacker would have already elevated
permissions.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

Merge branch 'nsenter-all-avoid-entering-nonexisting-ns' of https://github.com/Jongy/util-linux

* 'nsenter-all-avoid-entering-nonexisting-ns' of https://github.com/Jongy/util-linux:
  nsenter: Do not try to enter nonexisting namespaces when --all is used

nsenter: Do not try to enter nonexisting namespaces when --all is used

This PR changes is_same_namespace() to is_usable_namespace(). If a namespace file
is missing for the running PID, then it is considered not usable and we do not try
to enter it.

Signed-off-by: Yonatan Goldschmidt <yon.goldschmidt@gmail.com>

tests: (hardlink) add info about number of files to test

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

hardlink: add verbose messages when skip file

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

irqtop: fix options parsing

Reported-by: Peter Seiderer <ps.report@gmx.net>
Signed-off-by: Karel Zak <kzak@redhat.com>

bash-completion: fix irqtop

Forgot update the option as introduced by commit
17f7caa45105f0cbf8bf9b562468fba2c5d4a549.

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

irqtop: small coding style change

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

irqtop: add -c/--cpu-stat option

Since a23aecc1bf("irqtop: add per-cpu stats"), irqtop always shows
per-cpu stats. Test on a modern AMD server with 2 socket(256 CPU),
irqtop shows messy output with too many columns.

In this patch, add -c/--cpu-stat option to enable/disable per-cpu stats.
And 'auto' option auto-detect window size, only show per-cpu stats
if the length of per-cpu stats is shorter than the width of window.

[kzak@redhat.com: - shorten the commit message
                  - rename --cpu to --cpu-stat
                  - use scols_table_enable_nowrap() rather than trim in irqtop.c
                  - reduce --help for the new option]

Signed-off-by: zhenwei pi <pizhenwei@bytedance.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

bash-completion: add --json to dmesg

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

mkswap: add --quiet

* add --quiet to suppress output and warning messages
* check for mutually exclusive arguments --check vs. --quiet

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

Merge branch 'sysfs_partitions' of https://github.com/Portisch/util-linux

* 'sysfs_partitions' of https://github.com/Portisch/util-linux:
  sysfs: fallback for partitions not including parent name

sysfs: fallback for partitions not including parent name

Merge branch 'travis' of https://github.com/evverx/util-linux

* 'travis' of https://github.com/evverx/util-linux:
  ci: no longer refer to Travis CI

dmesg: add --json output format

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

lib/jsonwrt: check if JSON handler is initialized

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

Merge branch 'ci-coverity' of https://github.com/evverx/util-linux

* 'ci-coverity' of https://github.com/evverx/util-linux:
  ci: add a GHAction sending data to Coverity

sfdisk: fix typo in --move-data when check partition size

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

sfdisk: update docs, add examples to the man page

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

libfdisk: (script) rewrite start= and size= parsing

* let's use the same code for both supported formats
* add support for '+,' to enlarge by start move  (see previous commit)

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

libfdisk: enlarge partition by move start down

Now it's possible move start of the partition only when offset or
absolute value is specified. This commit implements resize to "use all
available free space before the current start".

We already support the same for end of the partition (resize to use all
free space after the partition).

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

docs: update IRC address

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

ci: no longer refer to Travis CI

and also add a link to Coveralls coverage reports
to Documentation/howto-test to make it a little easier
to find them.

Signed-off-by: Evgeny Vereshchagin <evvers@ya.ru>

build-sys: disable IPC tools on Darwin

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

ci: add a GHAction sending data to Coverity

To get it to work a secret named COVERITY_SCAN_TOKEN should
be added to the util-linux repository:
https://docs.github.com/en/actions/security-guides/encrypted-secrets#creating-encrypted-secrets-for-a-repository

It has to match the util-linux project token, which
can be found at
https://scan.coverity.com/projects/karelzak-util-linux?tab=project_settings

Signed-off-by: Evgeny Vereshchagin <evvers@ya.ru>

ci: build coverage reports on Coveralls

Signed-off-by: Evgeny Vereshchagin <evvers@ya.ru>

docs: update github URL

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

cifuzz: switch to the util-linux organization

Signed-off-by: Evgeny Vereshchagin <evvers@ya.ru>

build-sys: fir distcheck for fileeq.h

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

build-sys: (meson) fix hardlink

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

lsfd.1.adoc: Add missing underscores

login: improve coding style

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

login: Restore tty size after calling vhangup()

If login receives the tty to work on via stdin, stdout and stderr,
login might end up closing the remaining open file descriptors to
the tty just before it calls vhangup(). When the last open file
descriptors to a tty are closed, it's configured size is reset to
0x0. To avoid this from happening, save the size before closing
the stdin, stdout and stderr file descriptors and reapply the size
after the tty is re-opened.

Fixes #1484

lsfd: remove unused --sysroot

It's unnecessary to redirect to any /proc dump if we can use 'test_mkfds'.

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

lsfd: delete an unnecessary semicolon

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: introduce -p/--pid option, pids filter working in the early stage

    $ time sudo ./lsfd -Q '(PID == 1) or (PID == 2)' > /dev/null

    real 0m0.508s
    user 0m0.230s
    sys 0m0.267s

    $ time sudo ./lsfd -p 1,2 > /dev/null

    real 0m0.088s
    user 0m0.036s
    sys 0m0.033s

    $ [ $(./lsfd -p 1,2) = $(./lsfd -Q '(PID == 1) or (PID == 2)') ]
    $ echo $?
    0

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

Merge branch 'mount_fuzz' of https://github.com/evverx/util-linux

* 'mount_fuzz' of https://github.com/evverx/util-linux:
  mount_fuzz: reject giant files early

Merge branch 'topic/hardlink'

tests: update hardlink --maximum-size

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

hardlink: add new option: -S/--maximum-size

Support limiting comparison by maximum file size.

[kzak@redhat.com: - resolve conflicts]

Signed-off-by: Daniele Pizzolli <dan+dev@toel.it>
Signed-off-by: Karel Zak <kzak@redhat.com>

hardlink: add --cache-size

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

lib/fileeq: fix for small memsiz

hardlink: rename --buffer-size to --io-size

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

mount_fuzz: reject giant files early

It should help to address https://github.com/google/oss-fuzz/issues/6703

Signed-off-by: Evgeny Vereshchagin <evvers@ya.ru>

hardlink: rewrite files content comparison

Example:

 # hardlink --ignore-time --dry-run /var/lib

Old version, based on memcmp(), nothing cached:

 Mode:           dry-run
 Files:          93453
 Linked:         36172 files
 Compared:       0 xattrs
 Compared:       25755615 files
 Saved:          80.11 MiB
 Duration:       204.557185 seconds

New version, --method=memcmp, 32 bytes cached:

 Mode:           dry-run
 Method:         memcmp
 Files:          93453
 Linked:         36172 files
 Compared:       0 xattrs
 Compared:       25755615 files
 Saved:          80.11 MiB
 Duration:       5.248426 seconds

New version, zero-copy (Linux crypto API), sha1, crc32 and sha256:

 Mode:           dry-run
 Method:         sha1
 Files:          93453
 Linked:         36172 files
 Compared:       0 xattrs
 Compared:       25755615 files
 Saved:          80.11 MiB
 Duration:       2.870973 seconds

 Mode:           dry-run
 Method:         crc32
 Files:          93453
 Linked:         36172 files
 Compared:       0 xattrs
 Compared:       25755615 files
 Saved:          80.11 MiB
 Duration:       2.582651 seconds

 Mode:           dry-run
 Method:         sha256
 Files:          93453
 Linked:         36172 files
 Compared:       0 xattrs
 Compared:       25755615 files
 Saved:          80.11 MiB
 Duration:       3.152825 seconds

The default is sha256 to be robust..

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

include/fileeq: add functions to compare files content

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

lsfd: introduce -p/--pid option, pids filter working in the early stage

    $ time sudo ./lsfd -Q '(PID == 1) or (PID == 2)' > /dev/null

    real 0m0.508s
    user 0m0.230s
    sys 0m0.267s

    $ time sudo ./lsfd -p 1,2 > /dev/null

    real 0m0.088s
    user 0m0.036s
    sys 0m0.033s

    $ [ $(./lsfd -p 1,2) = $(./lsfd -Q '(PID == 1) or (PID == 2)') ]
    $ echo $?
    0

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

chsh: fflush stdout before reading from stdin

Same problem as described in https://github.com/karelzak/util-linux/pull/1481

Signed-off-by: Lorenzo Beretta <vc.net.loreb@gmail.com>

chfn: flush stdout before reading stdin and fix uninitialized variable

Same problem as described in https://github.com/karelzak/util-linux/pull/1481

Signed-off-by: Lorenzo Beretta <vc.net.loreb@gmail.com>

Merge branch 'refactor-tests' of https://github.com/Ajchler/util-linux

* 'refactor-tests' of https://github.com/Ajchler/util-linux:
  tests: split additional tests into subtests
  tests: split several tests into subtests
  tests: split test into subtest
  tests: split cal/colorw test into subtests
  tests: split cal/color test into subtests

build-sys: disable libmount when missing mntent.h

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

build-sys: include xlocale.h for locale_t on MacOS

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

build-sys: (hardlink) check for llistxattr and lgetxattr

It seems check for sys/xattr.h is not enough. The header file exists
on MacOS, but without these functions.

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

libbuid: use _UL_LIBUUID_UUID_H to cover uuid.h

It seems _UUID_UUID_H is too generic.

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

docs: add hint for non-public reports

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

Merge branch 'flush' of https://github.com/ericonr/util-linux

* 'flush' of https://github.com/ericonr/util-linux:
  vipw: flush stdout before getting answer.

agetty: use getttynam() if available

/etc/ttys seems to be a rather archaic concept that is not meant to
exist on Linux. Nevertheless it does. glibc has getttynam() which
correctly parses /etc/ttys. So let's give it a try before falling back
to the built in defaults. One can set the terminal type for a
specific tty using e.g.:

echo 'ttyS0 "" xterm' > /etc/ttys

[kzak@redhat.com: - improve configure.ac part
                  - log error on failed strdup()]

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

Merge branch 'uniformize-title-markups' of https://github.com/rffontenelle/util-linux

* 'uniformize-title-markups' of https://github.com/rffontenelle/util-linux:
  docs: Uniformize references to section titles

vipw: flush stdout before getting answer.

Otherwise the question is displayed only after the user presses Return,
and the program looks like it's hanging.

This happens at least on musl libc.

Reported by @loreb.

Signed-off-by: Érico Nogueira <erico.erc@gmail.com>

logger: fix --size use for stdin

The stdin version counts log header into the message size, but
for example when it reads message from argv[] it counts only message
itself.

 $ logger --stderr  --size 3 "abcd"
 <13>Oct 21 18:48:29 kzak: abc

 $ echo "abcd" | logger --stderr  --size 3
 logger: cannot allocate 18446744073709551597 bytes: Cannot allocate memory

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

Revert "logger: verify --size of large enough for message header"

This reverts commit 2fdea5a26480d46ec9445b0ea17fc984f84eecc9.

build-sys: use set +e before patch --try in ./autogen.sh

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

logger: realloc buffer when header size changed

This is probably paranoid optimization, but when we generate a new
header we need to be sure that buffer is not smaller than calculated
maximal size of user's data.

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

logger: verify --size of large enough for message header

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

wdctl: update man page

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

lsfd: (adoc) add more exapmles

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: remove --source option

-Q '(SOURCE == ...)' works well for the same purpose.
See https://github.com/karelzak/util-linux/pull/1418#issuecomment-945643880

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: add KTHREAD column

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: (test) add a case for displaying symlinks

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: (test) add a case for displaying COMMAND column

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: (test) add a case for displaying socket pairs

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: (test) add a case for displaying DEV column

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: (helper) improve the code converting file descriptor numbers

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: (test) add a case for displaying a character device

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: add --debug-filter option

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: (test) add a case for testing SIZE column

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: (test) add a case for testing FLAGS (wronly,nonblock) column

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: (helper) add "nonblock" parameter to pipe-no-fork factory

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: (helper) update examples in the help message

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: (helper) add "dentries" parameter to directory factory

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: (helper) add "dir" parameter to directory factory

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: (test) test POS column

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: (helper) add "offset" parameter to ro-regular-file factory

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

lsfd: (helper) accept an integer argument for a parameter

Signed-off-by: Masatake YAMATO <yamato@redhat.com>