libblkid: check for private DM device before open

blkid_new_probe_from_filename() opens the device before calling
blkid_probe_set_device(), which checks sysfs_devno_is_dm_private()
and sets BLKID_FL_NOSCAN_DEV.  But the open() itself bumps the
kernel open count, so a concurrent DM_DEVICE_REMOVE ioctl sees
EBUSY even though blkid never actually probes the device.

Move the private-device check before open() in
blkid_new_probe_from_filename().  The sysfs UUID is readable
without opening the block device, so this closes the race window
between udev-worker's blkid builtin and device-mapper remove.

Note: blkid_verify() in verify.c already does the check before
open() -- this patch makes the probe path consistent.

Signed-off-by: Zdenek Kabelac <zkabelac@redhat.com>

waitpid: consistently write the abbreviation "PID" in all-uppercase

Also, use %d instead of %u, as all other comparable messages use %d.

CC: Christian Goeschel Ndjomouo <cgoesc2@wgu.edu>
Signed-off-by: Benno Schulenberg <bensberg@telfort.nl>

nsenter: (man) correct the markup and improve the formatting

Remove the mistaken angle brackets that were introduced last month
by commit f2a5997869, add the missing equals sign, and put both it
and the subsequent colon in bold as they are literal characters.

Also, indent the list of paths to make it clearer where it ends,
and so that the subsequent paragraph gets indented correctly.

CC: Xiao Liang <shaw.leon@gmail.com>
CC: Christian Goeschel Ndjomouo <cgoesc2@wgu.edu>
Signed-off-by: Benno Schulenberg <bensberg@telfort.nl>

nsenter: (usage) improve the alignment of the option descriptions

Perfect alignment is not possible, otherwise too little space remains
for the descriptions.

Also, slightly reword one description to fit within 80 columns again,
and align also a description that was missed by commit 986188d1da two
monts ago.

Signed-off-by: Benno Schulenberg <bensberg@telfort.nl>

nsenter: (usage) correct the formatting of eight option arguments

Angle brackets should wrap each placeholder separately: <file> and
<nsid>.  Angle brackets should not enclose metacharacters (|) nor
literal ones (= and :).

(The mistaken formatting was introduced by commit f2a5997869.)

CC: Xiao Liang <shaw.leon@gmail.com>
Signed-off-by: Benno Schulenberg <bensberg@telfort.nl>

lsmem: correct the grammar of an error message

That is: make the message identical to its twin nine lines back.

(The spello was introduced by commit 92d018a105 from five months ago.)

CC: Sumanth Korikkar <sumanthk@linux.ibm.com>
Signed-off-by: Benno Schulenberg <bensberg@telfort.nl>

Merge branch 'improve/4070' of https://github.com/echoechoin/util-linux

* 'improve/4070' of https://github.com/echoechoin/util-linux:
  man: add ENVIRONMENT section for smartcols support in various man pages
  man: hide LIBSMARTCOLS_JSON in man pages that do not support --json

man: add ENVIRONMENT section for smartcols support in various man pages

This update introduces an ENVIRONMENT section to the man pages
of several utilities. The section includes references to the common
environment variable for smart column output, ensuring consistent
documentation across tools that support the --json output format.

terminal-colors.d: (man) re-apply improvements lost in merge

Commit 3252142521 improved the man page formatting and wording,
but merge commit d708fc7db6 undid all those changes due to a
conflict resolution mess.  Re-apply the improvements:

 - do not show 'type' as an optional part in the synopsis
 - indent the list of file types for clarity
 - correct and improve several wordings
 - fix grammar ("which provide" not "which provides")
 - compact the examples section

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

mkfs.cramfs: Properly check mmap results

The POSIX manual page states that the error return value is MAP_FAILED.
Sync with all other mmap checks throughout util-linux.

Also, test an mmap call which previously was not tested.

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

umount: fix --graceful option cosmetics and tests

- fix missing comma in man page option syntax
- fix typo "succesfully" -> "successfully"
- move --graceful before --namespace in help text
- shorten help text to single line
- improve man page description, add root-only note
- add test for non-existent path with --graceful

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

Merge branch 'umount_graceful_option' of https://github.com/cgoesche/util-linux-fork

* 'umount_graceful_option' of https://github.com/cgoesche/util-linux-fork:
  umount: add test for --graceful option
  umount: new --graceful option to exit successfully even if filesystem is unmounted

Merge branch 'PR/meson-libcommon-duplicates' of https://github.com/karelzak/util-linux-work

* 'PR/meson-libcommon-duplicates' of https://github.com/karelzak/util-linux-work:
  lib/meson: move caputils.c from lib_common to per-binary sources
  lib/meson: remove unconditional procfs.c from lib_common_sources
  lib/meson: remove duplicated source files from lib_common_sources

fsck: Fix stack overflow with many options

If fsck is given too many options, a stack overflow can occur because
these options are temporarily stored in a fixed sized array on stack.

As with arguments, check if too many options are supplied and call errx
if this is the case.

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

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

* 'eject_oob' of https://github.com/stoeckmann/util-linux:
  eject: Fix OOB access with empty argument

chmem: check ul_path_read_buffer() return value

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

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

* 'pg_sigchld' of https://github.com/stoeckmann/util-linux:
  pg: Reset SIGCHLD to default

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

* 'eject_sigchld' of https://github.com/stoeckmann/util-linux:
  eject: Reset SIGCHLD to default for wait

Merge branch 'bugfix' of https://github.com/cuiweixie/util-linux

* 'bugfix' of https://github.com/cuiweixie/util-linux:
  fincore: should add break here

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

* 'master-branch-1' of https://github.com/Leefancy/util-linux:
  The incorrect order of fcntl() parameters resulted in the inability to clear O_NONBLOCK

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

* 'pager_fork_failure' of https://github.com/stoeckmann/util-linux:
  lib/pager: Translate error message
  lib/pager: Always reset signals before exit
  lib/pager: Properly handle fork error return value

docs: getopt: improve arg example in getopt-example

The option `wow!*\?` in original example is not the argument of
`--c-long`. Only `=` separated in optional option is allowed.

[karelzak@redhat.com: - fix tcsh quote style to use backtick-quote
 matching actual script output, remove orphaned comment about
 exclamation mark escaping]

Based-on-patch-by: Ben Song <bensongsyz@gmail.com>
Addresses: https://github.com/util-linux/util-linux/pull/4088
Signed-off-by: Karel Zak <kzak@redhat.com>

Merge branch 'patch-27' of https://github.com/mariobl/util-linux

* 'patch-27' of https://github.com/mariobl/util-linux:
  Put file types in namei.1.adoc in a table structure
  Convert code blocks into tables
  Remove backticks from table in terminal-colors.d.5.adoc

# Conflicts:
# lib/terminal-colors.d.5.adoc

lslogins: fix broken GROUP column output

The condition on COL_GROUP incorrectly checked `!grp->gr_name`
instead of `!user->group`. This caused the GROUP column to always
be empty because when gr_name is non-NULL (normal case) the
assignment was skipped, and when gr_name is NULL, xstrdup() would
crash.

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

pam_lastlog2: add -lpam to Makemodule.am

If we don't add this, we don't actually link with PAM; compare the
before and after of pam_lastlog2.so -- without -lpam it does not list
libpam.so.0 as a NEEDED dependency.

Signed-off-by: Morgan Jones <me@numin.it>
Signed-off-by: Karel Zak <kzak@redhat.com>

terminal-colors.d: (man) do not show 'type' as an optional part

That is: remove the square brackets from around 'type' in the synopsis.

Also, do not give the impression that a leading dot by itself is fine
before the 'type'.  That is: a dot is required only when 'name' and/or
'@term' is present.

Also, do not colorize the square brackets as if they were part of the
placeholders.  (And use ++double plus++ passthroughs for the opening
square brackets, to prevent asciidoctor from misinterpreting them.)

Indent the list of file types, for clarity.

And correct or improve some wordings, and remove an inconvenient
blank line in an example.

Signed-off-by: Benno Schulenberg <bensberg@telfort.nl>

findfs: (man) improve the markup, the layout, and the wording

Mark NAME as a placeholder instead of a literal, and rename it to
TAGNAME to make it clearer what "tag" in the description refers to.

Indent the list of possible tags, to make it clearer where the
description continues.

Drop the angle brackets around placeholders -- those are used
in --help output, but in man pages it is just italics.

Also add a missing section header.

Signed-off-by: Benno Schulenberg <bensberg@telfort.nl>

lib/meson: move caputils.c from lib_common to per-binary sources

caputils.c is GPL-licensed, and libcommon should only contain LGPL
or public domain code (as stated in lib/Makemodule.am). Move it out
of lib_common_sources and add it directly to the sources of binaries
that use it (unshare, nsenter, setpriv), matching autotools.

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

lib/meson: remove unconditional procfs.c from lib_common_sources

procfs.c was listed in the base lib_common_sources and also added
conditionally under HAVE_OPENAT/HAVE_DIRFD (line 88). In autotools,
procfs.c is only added conditionally. Remove it from the base list
to match autotools and avoid duplication.

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

lib/meson: remove duplicated source files from lib_common_sources

The files idcache.c, randutils.c, md5.c, sha1.c, strutils.c, and
strv.c were listed both in the base lib_common_sources string list
and then appended again via files() objects. Remove them from the
base list since the files() append is needed for the named variables
used by other meson.build files.

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

pg: Reset SIGCHLD to default

When pg runs a child process, i.e. evaluating an "!" command, it calls
wait to receive status for the child.

If SIGCHLD is ignored, this results in an endless loop. Fix this by
resetting SIGCHLD to default before the any fork() call is performed.

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

The incorrect order of fcntl() parameters resulted in the inability to clear O_NONBLOCK

Signed-off-by: Leefancy <lijian01@kylinos.cn>

eject: Reset SIGCHLD to default for wait

If SIGCHLD is ignored, the wait call which waits for the umount command
to finish could fail with -1 and ECHILD. Since its return value is not
checked, the uninitialized status variable would be read, leading to
undefined behavior.

While at it, removed the additionally printed newline from errx message.

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

eject: Fix OOB access with empty argument

The the supplied argument is an empty string, eject triggers an out of
boundary access due to strlen() - 1 calculation.

Verify that the string is not empty before checking its last character.

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

fincore: should add break here

Put file types in namei.1.adoc in a table structure

lib/pager: Translate error message

Since the error message is not printed by a signal handler anymore, it
can be translated.

Requested-by: Karel Zak <kzak@redhat.com>
Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

lib/pager: Always reset signals before exit

Just in case another exit handler relies on these signal handlers being
set to default, or to allow users to press ^C and let the program exit
as expected.

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

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

* 'mkfs_minix_alarm' of https://github.com/stoeckmann/util-linux:
  mkfs.minix: Simplify SIGALRM handler

Merge branch 'PR/portability-hurd' of https://github.com/karelzak/util-linux-work

* 'PR/portability-hurd' of https://github.com/karelzak/util-linux-work:
  agetty: simplify USE_SYSTEMD ifdef in output_special_char
  agetty: introduce USE_NETLINK to separate netlink from reload
  agetty: guard netlink-related code with AGETTY_RELOAD
  libblkid: guard loop device code with __linux__ in cache
  build-sys: move shells.c to common (non-Linux) sources
  build-sys: move netlink sources from libcommon to agetty

Merge branch 'PR/lsmem-sysroot-prefix' of https://github.com/karelzak/util-linux-work

* 'PR/lsmem-sysroot-prefix' of https://github.com/karelzak/util-linux-work:
  lsmem: apply --sysroot prefix to all sysfs paths

lsmem: apply --sysroot prefix to all sysfs paths

Tests fail on architectures where /sys/module/memory_hotplug/parameters/
memmap_on_memory does not exist (e.g., loong64), because lsmem reads
from the real /sys instead of the --sysroot directory.

 - Route memmap_on_memory read through ul_path API with a local handler
   in print_summary(), so --sysroot prefix is respected.

 - Apply --sysroot prefix to sysmemconfig (/sys/firmware/memory) before
   the memory0 existence check.

 - Fix sysmemconfig memory leak (missing ul_unref_path).

 - Update expected test output (remove "Memmap on memory parameter"
   lines, not available in test dumps).

Reported-by: Chris Hofstaedtler <zeha@debian.org>
Signed-off-by: Karel Zak <kzak@redhat.com>
Tested-by: Sumanth Korikkar <sumanthk@linux.ibm.com>

Convert code blocks into tables

lib/pager: Properly handle fork error return value

Most of the code checks for pager_process.pid != 0 to see if a pager is
running. If fork fails and returns -1, pager_process.pid is not 0, too.

This in turn leads to an issue if waitpid is eventually called with -1
as pid, which would wait for all children. Since there are no children,
the wait_for_pager function calls err with an EXIT_FAILURE value.

Properly handle a potential fork error.

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

mkfs.minix: Simplify SIGALRM handler

Calling printf is not signal safe and in general there are too many
branches in alarm_intr for what it's used for.

Check the signal flag outside of the handler and print messages when
needed. Also, disable the alarm and reset the signal handling when it's
not needed anymore.

While at it, use correct formatter in printf.

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

tools/git-grouped-log: add co-author support

Parse Co-authored-by and Co-Author trailers from commit bodies and
display co-author names alongside the primary author.

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

agetty: simplify USE_SYSTEMD ifdef in output_special_char

Remove unnecessary duplicate #ifdef USE_SYSTEMD around the else
branch for utmpx-based user counting. Use the same pattern as
USE_NETLINK where the else and opening brace are split across the
endif boundary.

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

agetty: introduce USE_NETLINK to separate netlink from reload

The netlink functionality (network address display in issue file)
and the reload mechanism (inotify-based) are separate features that
shared the same AGETTY_RELOAD ifdef guard. Additionally, the netlink
code in output_special_char() used a different guard
(#if defined(RTMGRP_IPV4_IFADDR) && defined(RTMGRP_IPV6_IFADDR))
creating inconsistency.

Introduce USE_NETLINK, defined when AGETTY_RELOAD is enabled and
the required RTMGRP constants are available. Use it consistently
for all netlink-specific code (nl field, helper functions, socket
handling, escape sequences \4, \6, \a, \A).

Keep AGETTY_RELOAD for the inotify/reload mechanism only.

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

agetty: guard netlink-related code with AGETTY_RELOAD

The struct issue nl field and related helper functions use types
and headers (netaddrq.h, linux/netlink.h) that are only available
when AGETTY_RELOAD is defined. Guard the .nl.fd initializations
in main() and show_issue(), and wrap the netlink helper functions
(print_iface_best, print_addrq_bestofall, dump_iface_good,
dump_iface_all) with #ifdef AGETTY_RELOAD.

This fixes compilation on systems without inotify/netlink support
(e.g., GNU/Hurd).

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

libblkid: guard loop device code with __linux__ in cache

The is_loopdev() and loopdev_has_backing_file() functions from
loopdev.h are Linux-specific. Guard the loop device cache cleanup
in blkid_gc_cache() with __linux__ to allow compilation on
non-Linux systems like GNU/Hurd.

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

build-sys: move shells.c to common (non-Linux) sources

shells.c uses POSIX setusershell()/getusershell()/endusershell()
which are available on non-Linux systems (e.g., GNU/Hurd). Move it
from the Linux-only section to common libcommon sources.

The meson build already had shells.c in common sources, so only
the autotools build needed fixing.

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

build-sys: move netlink sources from libcommon to agetty

The netlink and netaddrq sources use Linux-specific headers
(linux/netlink.h, linux/rtnetlink.h) and cannot be compiled on
non-Linux systems like GNU/Hurd.

Move them out of libcommon and link directly into agetty (the only
user), following the same pattern as plymouth-ctrl.c. Also move the
corresponding test programs to the Linux-only section.

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

Remove backticks from table in terminal-colors.d.5.adoc

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

* 'pager_git' of https://github.com/stoeckmann/util-linux:
  lib/pager: Drop less workaround
  lib/pager: Add LV support
  lib/pager: Do not overwrite LESS environment var

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

* 'libblkid/parent-offset' of https://github.com/t-8ch/util-linux:
  libblkid: (probe) Account for IO bias when retrieving buffer from parent

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

* 'hardlink_arithmetic' of https://github.com/stoeckmann/util-linux:
  lib/fileeq: Check arithmetic in ul_fileeq_set_size

libblkid: (probe) Account for IO bias when retrieving buffer from parent

The 'off' variable gets aligned to 'pr->io_size'. However when
delegating to the parent prober, the value for 'off' from the original
function parameter needs to be used.

Undo the bias before calling into the parent prober.

Fixes: 8a534253bc52 ("libblkid: (probe) read data in chunks")
Closes: https://github.com/util-linux/util-linux/issues/4078
Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

man: hide LIBSMARTCOLS_JSON in man pages that do not support --json

PR #4070 consolidated libsmartcols env vars into man-common/env-smartcols.adoc
and had all tools (including fdisk and cfdisk) include it. That documented
LIBSMARTCOLS_JSON in fdisk/cfdisk man pages although those tools do not
support --json output, which is misleading.

Use ifdef::smartcols-json[] in env-smartcols.adoc so the JSON env var
section is only rendered for man pages that set :smartcols-json: (column,
lsblk, findmnt, sfdisk). fdisk and cfdisk include the same file but do not
set the attribute, so they only show the debug-related variables.

Addresses: util-linux#4070

lib/pager: Drop less workaround

This was fixed in less back in 2007. Let's follow git and drop the
workaround.

Taken from https://github.com/git/git/commit/e8320f350f523c3219ff8ec639663193941af57d

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

lib/pager: Add LV support

Set LV environment variable to "-c" for lv support.

Taken from https://github.com/git/git/commit/e54c1f2d2533c5406abeb8e3e0cf78c68ca9c21e

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

lib/pager: Do not overwrite LESS environment var

If LESS is already set, do not overwrite it.

Taken from https://github.com/git/git/commit/25fc1786ab147daa9ffa43ebcbaf5c1cc6e50d4f

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

lib/fileeq: Check arithmetic in ul_fileeq_set_size

Make sure that arithmetics do not overflow data types. Such overflows
could occur with large hardlink options or on 32 bit systems with large
files (due to size_t usage).

If possible, reduce sizes so operations can continue successfully.

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

tests/lsfd: skip executable mode tests when not running as root

This test does not necessarily work without root.

Reported-by: Christian Hesse <mail@eworm.de>
Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

meson: define _GNU_SOURCE when testing for 'struct statx'

glibc (at least with 2.43) requires _GNU_SOURCE to be defined to expose
'struct statx' from 'sys/stat.h'.

Define _GNU_SOURCE in the feature test, so 'struct statx' is correctly
detected. The code is being build with _GNU_SOURCE anyways.

With autotools this happens automatically.

Reported-by: Christian Hesse <mail@eworm.de>
Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de>

build-sys: update release dates

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

po: merge changes

po-man: merge changes

tools/git-tp-sync: fix po/ update

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

tools/checkconfig: strip C comments before macro extraction

Strip C/C++ comments (// line comments, single-line /* */ and
multi-line /* */ blocks) before extracting HAVE_ and ENABLE_ macros.
This avoids false positives from macros mentioned in comments, e.g.
"when HAVE_PIDFD_* ..." in include/pidfd-utils.h.

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

include/path: fix HAVE_STRUCT_STATX use

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

lib/procfs: remove duplicate statfs_magic.h include

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

include/mountutils: remove duplicate inttypes.h include

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

tools: git-version-bump, accept X.Y-devel version on master branch

Allow "X.Y-devel" as a valid version format, restricted to the master
branch only. Release versions (X.Y, X.Y.Z, X.Y-rcN, X.Y.Z-rcN) still
require a stable/vX.Y branch.

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

po: update LINGUAS list

po: add ar.po (from translationproject.org)

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

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

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

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

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

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

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

po: update ko.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 cs.po (from translationproject.org)

po-man: merge changes

po-man: add sv.po (from translationproject.org)

po-man: add ar.po (from translationproject.org)

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

po-man: update sr.po (from translationproject.org)

po-man: update ro.po (from translationproject.org)

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

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

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

Merge remote-tracking branch 'github/dependabot/github_actions/actions/upload-artifact-6'

Merge remote-tracking branch 'github/dependabot/github_actions/actions/cache-5'