libblkid: (ntfs) validate that sector_size is a power of two

The NTFS prober reads data based off an offset of the sector size.
If the sector size is unaligned and the read data is cached then other
probers can read unaligned values.

Sector sizes for NTFS actually only make sense as power-of-two so
validate that and as a sideeffect avoid the unaligned reads.

Also add the reproducer from OSS-Fuzz that found this issue.

Fixes #2509

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

Merge branch 'login-terminal' of https://github.com/stoeckmann/util-linux

* 'login-terminal' of https://github.com/stoeckmann/util-linux:
  login: prevent undefined ioctl and tcsetattr calls
  login: use correct terminal fd during setup

Merge branch 'timestamp' of https://github.com/t-8ch/util-linux

* 'timestamp' of https://github.com/t-8ch/util-linux:
  lib/timeutils: (parse_timestamp_reference) report errors on overflow
  lib/timeutils: print error if timestamp can't be parsed
  lib/timeutils: test epoch timestamp

Merge branch 'ci/cache-openwrt-sdk' of https://github.com/t-8ch/util-linux

* 'ci/cache-openwrt-sdk' of https://github.com/t-8ch/util-linux:
  ci: cache openwrt sdk

Merge branch 'ci/gcc-13' of https://github.com/t-8ch/util-linux

* 'ci/gcc-13' of https://github.com/t-8ch/util-linux:
  ci: also use GCC 13 for sanitizer builds
  ci: don't combine -Werror and -fsanitize

Merge branch 'analyzer1' of https://github.com/t-8ch/util-linux

* 'analyzer1' of https://github.com/t-8ch/util-linux:
  libsmartcols: handle nameless tables in export format
  ldattach: don't call exit() from signal handler
  lslogins: fix realloc() loop allocation size
  more: avoid out-of-bound access
  libfdisk: handle allocation failure in fdisk_new_partition
  lib/env: fix function name remote_entry -> remove_entry
  libmount: gracefully handle NULL path in mnt_resolve_target()
  libblkid: avoid memory leak of cachefile path
  lib/env: avoid underflow of read_all_alloc() return value

Merge branch 'realloc' of https://github.com/t-8ch/util-linux

* 'realloc' of https://github.com/t-8ch/util-linux:
  treewide: use (x)reallocarray() when applicable

Merge branch 'ci/clang-17' of https://github.com/t-8ch/util-linux

* 'ci/clang-17' of https://github.com/t-8ch/util-linux:
  ci: build with clang 17

Merge branch 'login-comment' of https://github.com/stoeckmann/util-linux

* 'login-comment' of https://github.com/stoeckmann/util-linux:
  login: Use pid_t for child_pid
  login: move comment

Merge branch 'lsfd--resolve-mqueue-nodev' of https://github.com/masatake/util-linux

* 'lsfd--resolve-mqueue-nodev' of https://github.com/masatake/util-linux:
  lsfd: collect the device number for mqueue fs in the initialization stage
  tests: (lsfd) show the entry for mqueue in /proc/self/mountinfo

lib/timeutils: (parse_timestamp_reference) report errors on overflow

Instead of silently returning '0' on 'minus'-underflow and silently
wrapping on 'plus'-overflow return ERANGE.

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

lib/timeutils: print error if timestamp can't be parsed

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

lib/timeutils: test epoch timestamp

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

ci: cache openwrt sdk

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

ci: also use GCC 13 for sanitizer builds

Now that -Werror is not used anymore with -fsanitize the false-postives
are not a problem anymore and we get the better runtime checks of newer
sanitizers.

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

ci: don't combine -Werror and -fsanitize

As explained in [0]:

    Note that sanitizers tend to increase the rate of false positive
    warnings, most notably those around -Wmaybe-uninitialized. We recommend
    against combining -Werror and [the use of] sanitizers.

[0] https://gcc.gnu.org/onlinedocs/gcc-13.2.0/gcc/Instrumentation-Options.html#index-fsanitize_003dundefined

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

libsmartcols: handle nameless tables in export format

fputs(NULL) is not allowed.

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

ldattach: don't call exit() from signal handler

Instead use _exit().

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

lslogins: fix realloc() loop allocation size

If stat() fails the realloc loop would always try to allocate zero
bytes.

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

more: avoid out-of-bound access

The realloc() needs to happen before that memory is used.

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

libfdisk: handle allocation failure in fdisk_new_partition

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

lib/env: fix function name remote_entry -> remove_entry

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

libmount: gracefully handle NULL path in mnt_resolve_target()

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

libblkid: avoid memory leak of cachefile path

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

lib/env: avoid underflow of read_all_alloc() return value

read_all_alloc() returns a negative error on failure.
When casting this to an unsigned type the failure check "< 1"
will not work.

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

treewide: use (x)reallocarray() when applicable

reallocarray() prevents overflow of the multiplication.
It also avoids issues with operator precedence like in libmount/src/context.c:

    pids = realloc(cxt->children, sizeof(pid_t) * cxt->nchildren + 1);

This only allocated one additional byte, and not enough space for
another child.

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

login: prevent undefined ioctl and tcsetattr calls

Do not call tcsetattr if tcgetattr fails, because the content of
tt and ttt is undefined in that case.

Also do not just warn if ioctl fails, but also avoid calling it again
after tty has been re-opened.

I've solved this by setting struct variables to values which cannot be
valid at this point. If they do have these exact values, then the
calls will be prevented.

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

login: use correct terminal fd during setup

The function get_terminal_name iterates through standard file
descriptors until it finds a terminal. This means that it's not
guaranteed that STDIN_FILENO (i.e. 0) is actually a terminal.

Do not modify permissions on possible files. Instead, retrieve
the file descriptor which was used by get_terminal_name as well.

Proof of Concept (as root):

1. Create a temporary file with a mode different than TTYPERM.
```
install -m 700 /dev/null /tmp/test
ls -l /tmp/test
-rwx------ 1 root root 0 Sep 21 20:15 /tmp/test
```

2. Run login within a terminal with adjusted stdin.
```
login < /tmp/test
host login:
            Hangup
```

3. Check permissions of input file.
```
ls -l /tmp/test
-rw------- 1 root root 0 Sep 21 20:15 /tmp/test
```

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

ci: build with clang 17

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

login: Use pid_t for child_pid

Since child_pid keeps track of a PID, it should be of type pid_t
and not of type int.

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

login: move comment

In 0b4d75fae55b4a5ff8f65df8551c56cf1eeb9b08 the variable "timeout" has
been moved from global to local scope, but its comment was not.

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

test: (lsfd) add a case for testing BPF-MAP.TYPE and BPF-MAP.TYPE.RAW columns

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

test: (mkfds::bpf-map) new factory

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

lsfd: add BPF-MAP.TYPE, BPF-MAP.TYPE.RAW, and BPF-MAP.ID columns

Based on these new columns, the NAME column for "bpf-map" is enhanced.

An example output:

  # ./lsfd -p 1 -Q '(TYPE == "bpf-map")' | head
  COMMAND PID USER ASSOC  XMODE    TYPE       SOURCE MNTID INODE NAME
  systemd   1 root    13 rw---- bpf-map anon_inodefs    15   106 id=305 type=hash-of-maps

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

lsfd: (man) add bps(8) and ss(8) to the "SEE ALSO" section

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

build-sys: fix libmount/src/hooks.c use

Reported-by: Samuel Thibault <samuel.thibault@aquilenet.fr>
Signed-off-by: Karel Zak <kzak@redhat.com>

autotools: fix non-Linux build

Signed-off-by: Karel Zak <kzak@redhat.com>
Co-Author: Samuel Thibault <samuel.thibault@aquilenet.fr>

Merge branch 'lsfd--monitor-flag-for-poll-syscall' of https://github.com/masatake/util-linux

* 'lsfd--monitor-flag-for-poll-syscall' of https://github.com/masatake/util-linux:
  tests: (test_mkfds::mkfds-multiplexing) make the output of ts_skip_subtest visible
  tests: (test_mkfds::mkfds-multiplexing) dump /proc/$pid/syscall for debugging
  lsfd: (man) write about XMODE.m and classical system calls for multiplexing
  tests: (lsfd) add a case testing XMODE.m for classical syscalls for multiplexing
  tests: (test_mkfds) add ppoll multiplexer
  tests: (test_mkfds) add poll multiplexer
  tests: (test_mkfds) add pselect6 and select multiplexers
  tests: (test_mkfds) add a new factory "multiplexing"
  tests: (test_mkfds) allow to add factory-made fds to the multiplexer as event source
  tests: (test_mkfds, refactor) make the function for waiting events plugable
  tests: (test_mkfds) include locale headers first to define _GNU_SOURCE
  tests: (test_mkfds) use err() when a system call fails
  lsfd: (comment) fix a typo
  lsfd: mark XMODE.m on fds monitored by select(2) and pselect6(2)
  lsfd: mark XMODE.m on fds monitored by poll(2) and ppoll(2)
  lsfd: cache the result of checking whether "XMODE" column is enabled or not
  procfs: add a helper function to access /proc/$pid/syscall
  lsfd: (refactor) make the code for traversing threads reusable

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

* 'kraj/lscpu' of https://github.com/kraj/util-linux:
  lscpu: Use 4K buffer size instead of BUFSIZ

lsfd: collect the device number for mqueue fs in the initialization stage

Though lsfd reads device minor numbers for file-systems having "nodev"
from /proc/$pid/mountinfo, we observed lsfd failed to resolve the
values of SOURCE column for mqueue files on s390 CI/CD env. It seems
that /proc/$pid/mountinfo doesn't provide enough information.

This change makes lsfd open a mqueue file in lsfd's initialization
stage as a new data source for resolving; lsfd can collect an
actually-used minor number from the file descriptor with fstat(2).

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

Merge branch 'vla-alloca' of https://github.com/t-8ch/util-linux

* 'vla-alloca' of https://github.com/t-8ch/util-linux:
  buildsys: warn on usage of alloca()
  more: remove usage of alloca()
  buildsys: warn on usage of VLAs
  lib/mbsedit: remove usage of VLA
  lscpu: remove usage of VLA
  fdisk: remove usage of VLA
  libfdisk: (dos) remove usage of VLA
  libblkid: (stratis) remove usage of VLA
  lib/path: remove usage of VLA

tests: (lsfd) show the entry for mqueue in /proc/self/mountinfo

The entry is useful debugging the lsfd and the test case for mqueue.

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

Merge branch 'tests/caps' of https://github.com/t-8ch/util-linux

* 'tests/caps' of https://github.com/t-8ch/util-linux:
  tests: fix capability testing

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

* 'master' of https://github.com/gjioui/util-linux:
  Add Phytium FTC862 cpu model. fix:#2486

Merge branch 'PR/scriptreplay-flow'

lscpu: Use 4K buffer size instead of BUFSIZ

Some lines in /proc/cpuinfo can be large e.g. flags and can then
truncate them in displaying them

BUFSIZ can vary quite a bit  e.g. glibc/linux systems its 8192
but on musl/linux and OSX its 1024, on mingW it is 256, some tests e.g.
x86_64-64cpu-linux6.2.tar.gz has added really long line for cpu flags
line which is greater than 1024 characters and hence this test fails
on musl because lscpu -s reports truncated string

Fixes x86_64-64cpu-linux6.2 tests

Signed-off-by: Khem Raj <raj.khem@gmail.com>
s

buildsys: warn on usage of alloca()

alloca() is susceptible to security issues, ensure it is not
introduced.

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

more: remove usage of alloca()

alloca() is susceptible to security issues, avoid it.

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

buildsys: warn on usage of VLAs

Variable-length-arrays are susceptible to security issues, ensure none
are introduced.

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

lib/mbsedit: remove usage of VLA

Variable-length-arrays are susceptible to security issues, avoid them.

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

lscpu: remove usage of VLA

Variable-length-arrays are susceptible to security issues, avoid them.

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

fdisk: remove usage of VLA

Variable-length-arrays are susceptible to security issues, avoid them.

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

libfdisk: (dos) remove usage of VLA

Variable-length-arrays are susceptible to security issues, avoid them.

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

libblkid: (stratis) remove usage of VLA

Variable-length-arrays are susceptible to security issues, avoid them.

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

lib/path: remove usage of VLA

Variable-length-arrays are susceptible to security issues, avoid them.

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

tests: fix capability testing

The old capability testing logic would incorrectly interpret the following
output from getpcaps as cap_wake_alarm being supported:

$ getpcaps 0
0: cap_wake_alarm=i

Instead of realying on brittle command output parsing at a test helper
to detect capabilities.

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

libblkid: prune unneeded buffers

When a new buffer is cached that is a superset of another existing
buffer the old buffer can be removed as future requests can be satisfied
by the new one.

As probe functions can have local references to buffered data, delay the
cleanup until the probefunc is finished to avoid accessing freed data.

For the bcachefs.img testfile this reduces the final (maximal) cache
from 34338 bytes in 54 buffers
  to 24760 bytes in 40 buffers.

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

libblkid: add remove_buffer helper

Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

Add Phytium FTC862 cpu model.
fix:#2486

tests: (test_mkfds::mkfds-multiplexing) make the output of ts_skip_subtest visible

tests: (test_mkfds::mkfds-multiplexing) dump /proc/$pid/syscall for debugging

lsfd: (man) write about XMODE.m and classical system calls for multiplexing

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

tests: (lsfd) add a case testing XMODE.m for classical syscalls for multiplexing

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

autotools: fix typos

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

scriptreplay: support ctrl+s and ctrl+g

The old scriptreplay supported XON/XOFF flow control. The new
implementation uses cfmakeraw() and it disables it by default. Let's
enable it by IXON iflag.

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

tests: (test_mkfds) add ppoll multiplexer

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

tests: (test_mkfds) add poll multiplexer

tests: (test_mkfds) add pselect6 and select multiplexers

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

tests: (test_mkfds) add a new factory "multiplexing"

tests: (test_mkfds) allow to add factory-made fds to the multiplexer as event source

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

tests: (test_mkfds, refactor) make the function for waiting events plugable

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

tests: (test_mkfds) include locale headers first to define _GNU_SOURCE

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

tests: (test_mkfds) use err() when a system call fails

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

lsfd: (comment) fix a typo

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

lsfd: mark XMODE.m on fds monitored by select(2) and pselect6(2)

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

lsfd: mark XMODE.m on fds monitored by poll(2) and ppoll(2)

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

lsfd: cache the result of checking whether "XMODE" column is enabled or not

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

procfs: add a helper function to access /proc/$pid/syscall

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

lsfd: (refactor) make the code for traversing threads reusable

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

Merge branch 'lsfd--fix-the-way-to-stop-mkfds-1' of https://github.com/masatake/util-linux

* 'lsfd--fix-the-way-to-stop-mkfds-1' of https://github.com/masatake/util-linux:
  tests: (lsfd::filter-floating-point-nums) use --raw output to make the case more robust
  tests: (lsfd::column-xmode) do rm -f the file for testing before making it
  tests: make ts_skip_capability accepts the output of older version of getpcaps
  tests: (lsfd::mkfds-unix-dgram) don't depend on the number of whitespaces in the output
  tests: (lsfd) avoid race conditions (part 1)
  tests: (test_mkfds) monitor stdin by default

Merge branch 'gitignore-lsclocks' of https://github.com/bcarnes/util-linux

* 'gitignore-lsclocks' of https://github.com/bcarnes/util-linux:
  add missing lsclocks to .gitignore

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

* 'patch-1' of https://github.com/ThomasKaiser/util-linux:
  Revise/add HiSilicon core names

Merge branch 'waitpid--suppress-message' of https://github.com/masatake/util-linux

* 'waitpid--suppress-message' of https://github.com/masatake/util-linux:
  waitpid: warn of "exited" only when --verbose is given

Merge branch 'PR/libmount-stat.h'

Merge branch 'setpgid' of https://github.com/DaanDeMeyer/util-linux

Revise/add HiSilicon core names

add missing lsclocks to .gitignore

tests: (lsfd::filter-floating-point-nums) use --raw output to make the case more robust

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

tests: (lsfd::column-xmode) do rm -f the file for testing before making it

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

tests: make ts_skip_capability accepts the output of older version of getpcaps

In https://git.kernel.org/pub/scm/libs/libcap/libcap.git/commit/?id=8a2ac848a2065641e20a8b50384b68e8781cc6fc,
the default output format of getpcaps command was changed.

ts_skip_capability() assumes only the newer format, a format
introduced in the commit.

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

tests: (lsfd::mkfds-unix-dgram) don't depend on the number of whitespaces in the output

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

tests: (lsfd) avoid race conditions (part 1)

For terminating test_mkfds process, the test cases
used SIGCONT. test_mkfds used pselect for waiting for the signal.
After receiving the signal test_mkfds exited.

There is a race condition in this logic. If a test case sends SIGCONT
to test_mkfds process before the process calls pselect, the signal
may be lost. As the result, test_mkfds keeps waiting forever in pselect.

This change replaces the code sending SIGCONT with writing to the pipe
connecting to the stdin of test_mkfds. test_mkfds also monitors
its stdin in the pselect call. Unlike the signal, the written bytes to
the pipe are queued.

This change doesn't touch tests/ts/lsfd/option-inet and option-summary
test cases. They don't use coproc. So fixing the race condition in the test
cases can be much complicated. I will make a dedicated pull request (part 2)
for them.

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

tests: (test_mkfds) monitor stdin by default

After making file descriptors as specified in arguments, test_mkfds
waits for an event with pselect. When test_mkfds gets the event, it
exits.

The original code monitored stdin when the test_mkfds process ran in
the foreground. However, monitoring stdin is useful even when it runs
in the background. ts/lsfd/mkfds-* test cases run test_mkfds as a
co-process.  So the test cases can terminate their test_mkfds
processes with echo command if the processes monitor stdin.

This change also adds -X/--dont-monitor-stdin option. As the name
shows, with the option, test_mkfds doesn't monitor stdin as before.
Some test cases run test_mkfds directly. In such a case,
tests/functions.sh may connect stdin of test_mkfds to
/dev/null. /dev/null is always ready to be read.

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

waitpid: warn of "exited" only when --verbose is given

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

Merge branch 'libblkid/protection' of https://github.com/t-8ch/util-linux

* 'libblkid/protection' of https://github.com/t-8ch/util-linux:
  libblkid: protect shared buffers against modifications
  libblkid: (vfat) avoid modifying shared buffer
  libblkid: (jmicron_raid) use checksum APIs
  libblkid: (jmicron_raid) avoid modifying shared buffer
  libblkid: (zonefs) avoid modifying shared buffer

Merge branch 'PR/losetup-json' of github.com:karelzak/util-linux-work

* 'PR/losetup-json' of github.com:karelzak/util-linux-work:
  losetup: make --output-all more usable
  losetup: add MAJ a MIN for device and backing-file
  losetup: fix JSON MAJ:MIN

libmount: make.stx_mnt_id use more robust

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

libmount: report statx in features list

Let's make statx() support visible in "mount -V" output.

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

libmount: fix statx() includes

Using sys/stat.h and linux/stat is too tricky.h together. It seems
better to rely on libc and use sys/stat.h only. Users affected
by old libc must update to use recent util-linux.

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

Add new setpgid utility

This program allows running a command in a new process group and
optionally makes the new process group the foreground process group
of the ctty.

This is useful when running programs through wrappers programs (think
bubblewrap, ...) and wanting to make sure that SIGINT is only sent to
the innermost process. This is possible by putting the innermost process
in a new process group and making that process group the foreground process
group of the controlling terminal.

By adding a separate utility to util-linux, we can apply this to any
program even if the program itself doesn't implement this functionality.