tests: an attempt to get around https://github.com/karelzak/util-linux/issues/1110

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

scriptreplay: enable special character handling

Calling cfmakeraw disables terminal special character handling. For
example, Ctrl-C does not send SIGINT to scriptreplay. The following
fixes this.

 tattr.c_lflag |= ISIG;

where tattr is the struct termios with which we are working.

libmount (verity): let crypt_deactivate_by_name handle its own data structure

It's not necessary to initialize a crypt_device, the function will do it
on its own if NULL is passed. Removes a few extra library calls.

Signed-off-by: Luca Boccassi <luca.boccassi@microsoft.com>

libmount (verity): plug libcryptsetup logger into our logging system

Allows logs from the library to be printed by our system:

731814: libmount:   VERITY: crypsetup: Device libmnt_img.raw already exists.

If the mount context is in verbose mode, enable all debug logs from
libcrypsetup as well.

Signed-off-by: Luca Boccassi <luca.boccassi@microsoft.com>

Make scriptreplay set terminal to raw mode

bash-completion: add irqtop/lsirq --softirq

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

irqtop/lsirq: add softirq for man page

Signed-off-by: zhenwei pi <pizhenwei@bytedance.com>

irqtop/lsirq: add additional desc for softirq

Suggested by Karel, add additional description to make softirq more
friendly to end-user. Discuss about this:
    https://github.com/karelzak/util-linux/pull/1079

Note that, we should keep softirqs table align to kernel source code.

Signed-off-by: zhenwei pi <pizhenwei@bytedance.com>

irqtop/lsirq: support softirq

Add '-S' or '--softirq' for irqtop/lsirq, instead of interrupts, show
softirqs infomation. Because there is no more description of softirq,
do not show 'NAME' column by default.

Signed-off-by: zhenwei pi <pizhenwei@bytedance.com>

blkzone: add capacity field to zone report

NVMe ZNS specification defines zone capacity. The report zone interface
of Linux kernel supports it. Expose it in report zone by blkzone command.

Signed-off-by: Hans Holmberg <hans.holmberg@wdc.com>
Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>

hexdump: automatically use -C when called as hd

When invoking hexdump as hd enable the "Canonical" format to by
default, implying the -C option.

This is historic behaviour on Debian and apparently also on FreeBSD.
Some Debian users have asked for this to be restored, after Debian
switched to util-linux' hexdump and hd.

Signed-off-by: Chris Hofstaedtler <zeha@debian.org>

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

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

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

tools: remove changes merged by accident

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

docs: update v2.36-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 pt.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 cs.po (from translationproject.org)

docs: add blkdiscard to ReleaseNotes

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

libsmartcols: (docs) fix reference

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

libfdisk: (docs) add missing comment

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

libmount: (docs) fix typo, remove unused reference

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

libsmartcols: (docs) add missing references

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

libmount: (docs) add missing references

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

libfdisk: (docs) add missing references

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

libblkid: (docs) add missing references

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

Merge branch 'documentation_typo' of https://github.com/8vasu/util-linux

* 'documentation_typo' of https://github.com/8vasu/util-linux:
  fix typo: yourbranch -> yourgit

Manual pages: scriptreplay.1: Miscellaneous wording, grammar, and formatting fixes

Nothing too contentious here, I think, so I'm rolling all
of the edits into one patch.

Signed-off-by: Michael Kerrisk (man-pages) <mtk.manpages@gmail.com>

Manual pages: scriptlive.1: Miscellaneous wording, grammar, and formatting fixes

Nothing too contentious here, I think, so I'm rolling all
of the edits into one patch.

Signed-off-by: Michael Kerrisk (man-pages) <mtk.manpages@gmail.com>

Manual pages: script.1: Miscellaneous wording, grammar, and formatting fixes

Nothing too contentious here, I think, so I'm rolling all
of the edits into one patch.

Signed-off-by: Michael Kerrisk (man-pages) <mtk.manpages@gmail.com>

libmount: fix condition for mountinfo filter

The filter has to be used for valid entries.

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

fix typo: yourbranch -> yourgit

rename: tests: add more symlink checks

Add a few checks to the symlink test, including for the fix
in commit 477239ce0d60 ("rename: fix regression for symlink
with non-existing target").

Tested with './tests/run.sh rename/symlink' before/after that
commit with FAILED/OK results (and '--show-diff') as expected:

After:
       rename: symlink check                  ... OK

Before:
       rename: symlink check                  ... FAILED (rename/symlink)

With --show-diff:

    diff-{{{
    --- /home/mfo/git/util-linux/tests/expected/rename/symlink 2020-07-14 15:21:06.412792160 -0300
    +++ /home/mfo/git/util-linux/tests/output/rename/symlink 2020-07-14 15:45:10.980927233 -0300
    @@ -3,5 +3,3 @@
     rename_slink.3: `old' -> `new'
     `rename_slink.1' -> `rename_symlink.1'
     target
    -`rename_slink.2' -> `rename_symlink.2'
    -target
    }}}-diff

    diff-{{{
    --- /home/mfo/git/util-linux/tests/expected/rename/symlink.err 2020-07-14 15:37:42.466207786 -0300
    +++ /home/mfo/git/util-linux/tests/output/rename/symlink.err 2020-07-14 15:45:10.984927251 -0300
    @@ -1 +1,2 @@
    +rename: rename_slink.2: not accessible: No such file or directory
     rename: rename_slink.3: No such file or directory
    }}}-diff

Signed-off-by: Mauricio Faria de Oliveira <mfo@canonical.com>

Manual pages: runuser.1, su.1: miscellaneous wording and formatting fixes

Various wording and formatting fixes. Nothing too contentnious, I think,
so I rolled these changes into one patch.

Since there is much common text in su.1 and runuser.1, I've combined
the changes to both pages into one patch, and, as far as possible,
ensured that changes to the common pieces of text match across the
two pages.

Signed-off-by: Michael Kerrisk (man-pages) <mtk.manpages@gmail.com>

Manual pages: reword su.1 description

Reported-by: "Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

docs: reword others "allow to"

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

Manual pages: various: reword "allow(s) to"

The wording "allow(s) to" is not grammatical English. Reword various
pages to use a more correct form such "can be use to" or "allows
the [noun] of".

Aklong the way, fix a few nearby wording errors in some pages.

Signed-off-by: Michael Kerrisk (man-pages) <mtk.manpages@gmail.com>

Manual pages: login.1: various minor wording fixes

Signed-off-by: Michael Kerrisk (man-pages) <mtk.manpages@gmail.com>

Manual pages: login.1: formatting fixes

Bold for command names; italic for pathnames, etc.

Signed-off-by: Michael Kerrisk (man-pages) <mtk.manpages@gmail.com>

Manual pages: login.1: SEE ALSO: add utmp(5), lastlog(8)

utmp and lastlog are referred to in the manual page, and thus deserve a
cross references in SEE ALSO.

Signed-off-by: Michael Kerrisk (man-pages) <mtk.manpages@gmail.com>

manual pages: adjtime_config.5: format pathname with .I

Signed-off-by: Michael Kerrisk (man-pages) <mtk.manpages@gmail.com>

build-sys: add --enable-werror to devel build scenarios

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

libsmartcols: (sample) remove unused variable

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

libsmartcols: (sample) remove unnecessary check [coverity scan]

@str cannot be NULL ...

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

libsmartcols: (sample) check scols_line_refer_data() return code [coverity scan]

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

libuuid: (test) make sure UUID is terminated [coverity scan]

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

libsmartcols: (smaple) check scols_line_set_data() return code [coverity scan]

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

libuuid: (test) cleanup unused memory [coverity scan]

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

libuuid: (test) close fd [coverity scan]

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

test_tiocsti: check ioctl() return code [coverity scan]

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

libsmartcols: (sample) check scols_line_set_data() return code [coverity scan]

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

lsblk: Add SERIAL column to the SCSI output mode.

If there are several identical disks, disk serial number can help
to distinguish exact drive.

This could be helpful in debugging RAID failures and similar problems.

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

docs: misc-utils: change "allows to <verb>" to "allows <verb>ing"

  The web site "lintian.debian.org" shows some examples of "allows to",
which are changed to "allows one to".

  I chose here to use gerund.

Signed-off-by: Bjarni Ingi Gislason <bjarniig@rhi.hi.is>

Subject: docs: disk-utils: change "allows to <verb>" to "allows <verb>ing"

  The web site "lintian.debian.org" shows some examples of "allows to",
which are changed to "allows one to".

  I chose here to use gerund.

Signed-off-by: Bjarni Ingi Gislason <bjarniig@rhi.hi.is>

Subject: docs: remove unnecessary paragraph macros

  Remove a second paragraph macro (.TP, .PP) as it does not change the
output (.SS/.PP) or it adds an extra empty line (.TP/.TP)

  Warning from "mandoc -Tlint":

mandoc: ./sys-utils/hwclock.8.in:299:2: WARNING: line scope broken: TP breaks TP
mandoc: ./sys-utils/hwclock.8.in:459:2: WARNING: skipping paragraph macro: PP after SS
mandoc: ./sys-utils/hwclock.8.in:543:2: WARNING: skipping paragraph macro: PP after SS
mandoc: ./sys-utils/hwclock.8.in:574:2: WARNING: skipping paragraph macro: PP after SS
mandoc: ./sys-utils/hwclock.8.in:673:2: WARNING: skipping paragraph macro: PP after SS
mandoc: ./sys-utils/hwclock.8.in:721:2: WARNING: skipping paragraph macro: PP after SS

Signed-off-by: Bjarni Ingi Gislason <bjarniig@rhi.hi.is>

Subject: docs: change from nofill to fill mode

  Change request nofill mode to fill mode.

  Warning from "mandoc -Tlint":

mandoc: ./misc-utils/uuidd.8.in:81:2: STYLE: fill mode already disabled, skipping: nf

Signed-off-by: Bjarni Ingi Gislason <bjarniig@rhi.hi.is>

docs: remove trailing space in strings

  Trim trailing space in strings.

  There is no change in the output of "nroff" and "groff".

Signed-off-by: Bjarni Ingi Gislason <bjarniig@rhi.hi.is>

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

* 'coverity' of https://github.com/evverx/util-linux:
  docs: mention Coverity Scan and the Fossies codespell report
  travis: integrate util-linux with Coverity Scan

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

docs: mention Coverity Scan and the Fossies codespell report

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

travis: integrate util-linux with Coverity Scan

For this to work, a daily cron job running on the master branch
should be added: https://docs.travis-ci.com/user/cron-jobs/

The report can found at https://scan.coverity.com/projects/karelzak-util-linux

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

More typo fixes by fossies

travis: switch to a newer version of macOS

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

Fix minor typos

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

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

docs: update v2.36-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>

docs: update ReleaseNotes

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

last: fix use of non-terminated utmp->ut_line

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

docs: add terminal hyperlinks to TODO

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

travis: don't ask for Ubuntu release on XOS

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

libmount: (verity) remove unnecessary empty lines

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

Merge branch 'lp1886300' of https://github.com/mfoliveira/util-linux

* 'lp1886300' of https://github.com/mfoliveira/util-linux:
  rename: fix regression for symlink with non-existing target

Merge branch 'cryptsetup_dlopen' of https://github.com/bluca/util-linux

* 'cryptsetup_dlopen' of https://github.com/bluca/util-linux:
  cryptsetup: add option to use via dlopen in libmount

lsmem: report inaccessible /sys/devices/system/memory

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

lib/path: add ul_path_is_accessible()

This function allow to check that path_cxt is usable. Note that
ul_new_path() does not open the path.

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

lsmem: make it without leaks for non-error output

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

rename: fix regression for symlink with non-existing target

Since commit 5454df9c3110 ("rename: check source file access early")
rename fails early for symlinks with non-existing target (regression),
because access() dereferences the link.

From access(2):

  "access() checks whether the calling process can access the file pathname.
   If pathname is a symbolic link, it is dereferenced."

Thus replace access() with faccessat() and lstat() as fallback,
(as in do_symlink()), that is equivalent for symlink and files.

From fsaccess(2) and stat(2):

  "The faccessat() system call operates in exactly the same way as access(),
   except for the differences described here.
   [...]
   If pathname is relative and dirfd is the special value AT_FDCWD, then pathname
   is interpreted relative to the current working directory of the calling process
   (like access()).
   [...]
   AT_SYMLINK_NOFOLLOW
     If pathname is a symbolic link, do not dereference it:
     instead return information about the link itself."

  "lstat() is identical to stat(), except that if pathname is a symbolic link, then
   it returns information about  the  link  itself, not the file that it refers to."

Testing
-------

  1) symlink with existing target
  2) symlink with non-existing target
  3) non-existing symlink
  4) existing file
  5) non-existing file

Before:

  $ touch file-found
  $ ln -s file-found symlink-1
  $ ./rename sym symbolic- symlink-1 # XPASS.
  $ echo $?
  0

  $ ln -s file-not-found symlink-2
  $ ./rename sym symbolic- symlink-2 # FAIL! REGRESSION.
  rename: symlink-2: not accessible: No such file or directory
  $ echo $?
  1

  $ ./rename sym symbolic- symlink-3 # XFAIL.
  rename: symlink-3: not accessible: No such file or directory
  $ echo $?
  1

  $ touch file-found
  $ ./rename found existing file-found # XPASS.
  $ echo $?
  0

  $ ./rename found existing file-not-found # XFAIL.
  rename: file-not-found: not accessible: No such file or directory
  $ echo $?
  1

After:

  $ touch file-found
  $ ln -s file-found symlink-1
  $ ./rename sym symbolic- symlink-1 # XPASS.
  $ echo $?
  0

  $ ln -s file-not-found symlink-2
  $ ./rename sym symbolic- symlink-2 # PASS! REGRESSION FIXED.
  $ echo $?
  0

  $ ./rename sym symbolic- symlink-3 # XFAIL.
  rename: symlink-3: not accessible: No such file or directory
  $ echo $?
  1

  $ touch file-found
  $ ./rename found existing file-found # XPASS.
  $ echo $?
  0

  $ ./rename found existing file-not-found # XFAIL.
  rename: file-not-found: not accessible: No such file or directory
  $ echo $?
  1

And to test/simulate faccessat()'s EINVAL for AT_SYMLINK_NOFOLLOW
for Mac OS X, per commit 826538bf64c5 ("rename: skip faccessat()
failure if AT_SYMLINK_NOFOLLOW is not a valid flag"), forced 'if'
to evaluate to false so that lstat() is taken.

It still fails early; the error messages are slightly different
('not accessible' vs. 'stat of ... failed') but still tell same
'No such file or directory'; exit code is the same as well.

  $ ./rename sym symbolic- symlink-3 # XFAIL. DIFF MSG/SAME RC.
  rename: stat of symlink-3 failed: No such file or directory
  $ echo $?
  1

  $ ./rename found existing file-not-found # XFAIL. DIFF MSG/SAME RC.
  rename: stat of file-not-found failed: No such file or directory
  $ echo $?
  1

Tested on commit 2b41c409e ("Merge branch 'blkd-err' of ...")

Signed-off-by: Mauricio Faria de Oliveira <mfo@canonical.com>

cryptsetup: add option to use via dlopen in libmount

Enabling libcrypsetup in libmount had several unintended side
effects.
First of all, it increases the Debian minimal image size by
~2.5% (5.6MB worth of new libraries).
Then, due to libcryptsetup linkage to OpenSSL and libjson-c,
it causes incompatibilities with external programs linking
against both libmount and a private, static, old version of
OpenSSL, or external programs linking against libjansson or
json-glib, which have one symbol in common with libjson-c.

If ./configure is ran with --with-crypsetup=dlopen,
instead of linking to libcrypsetup, use dlopen to resolve
the symbols at runtime only when the verity feature is
used, thus avoiding clashes and keeping images size down.

Fixes #1081

Signed-off-by: Luca Boccassi <luca.boccassi@microsoft.com>

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

* 'blkd-err' of https://github.com/evverx/util-linux:
  tests: turn off detect_leaks on s390x, use more asan options
  tests: skip "blkid/dm-err" when `mknod` doesn't work

tests: turn off detect_leaks on s390x, use more asan options

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

tests: skip "blkid/dm-err" when `mknod` doesn't work

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

cal: use size_t to calculate width [lgtm scan]

We use size_t for width. This patch makes it consistent for
years_width too.

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

unshare: (man) add note about signals on --fork

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

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

libfdisk: (gpt) fix compiler warning [-Wmaybe-uninitialized]

Make sure that variables used in  gpt_calculate_* are always initialized.

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

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

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

* 'travis-arch' of https://github.com/evverx/util-linux:
  travis: build util-linux on arm64, ppc64le and s390x

travis: turn on CIFuzz

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