umount: fix memory leak [coverity scan]

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

fstrim: fix memory leak [coverity scan]

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

findmnt: fix memory leak [coverity scan]

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

sulogin: don't use uninitialized struct sigaction [coverity scan]

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

sulogin: check result of the open() [coverity scan]

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

last: fix is_phantom() logic [coverity scan]

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

libmount: add missing assert()

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

libblkid: comparing an array to null is not useful [coverity scan]

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

lib/procutils: check fgets() return value [coverity scan]

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

lib/loopdev: comparing an array to null is not useful [coverity scan]

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

docs: bring four more man pages closer to standard formatting

In the IPC man pages strictly adhere to the memory, queue, semaphore
order, to reduce disorientation.  And for 'ipcmk', the arguments for
-M and -S and -p are not optional.  Add --version for 'mountpoint'.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

fstrim: avoid TOCTOU race

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

fallocate: avoid unnecessary computation

Where POSIX_FADV_SEQUENTIAL and HAVE_POSIX_FADVISE are not available it
is waste of resources to have variables that are meaningful only for
posix_fadvise().  Also initialize the variables immediately to correct
values, and since cachesz is never changed mark it read only.

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

lib/colors: remove memory leak

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

build-sys: harmonize some indentation

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

docs: improve the wording and conventions in the man-page howto

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

docs: improve the formatting and wording of two man pages

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

cfdisk: make code more readable for analyzers [coverity scan]

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

cfdisk: fix possible memory leak [coverity scan]

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

docs: bring some more man pages closer to standard formatting

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

docs: bring a few more man pages closer to standard formatting

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

docs: bring several man pages closer to standard formatting

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

lslogins: make --notruncate more robust

 - keep coldescs[] read-only
 - don't expect that only COL_GECOS uses truncate flag

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

lslogins: add --noheadings

That's fatal mistake that this option which is required for script
is missing.

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

lslogins: use bit-array, cleanup SELinux code

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

cfdisk: restore the license that was lost in January

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>
Signed-off-by: Karel Zak <kzak@redhat.com>

docs: mount -o nouser is not useful, be clearer about that

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

docs: improve wording and formatting of the man page of cfdisk

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

tests: add kill --queue check

Correction.  The final failure to catch signal exit should not use
exiter() function.

--->8----
From: Sami Kerola <kerolasa@iki.fi>
Date: Mon, 14 Jul 2014 18:35:56 +0100
Subject: [PATCH] tests: add kill --queue check

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

cfdisk: fix warning usage, improve 's' help

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

cfdisk: accept 's' as well as 'S' for sectors

Reported-by: Benno Schulenberg <bensberg@justemail.net>
Signed-off-by: Karel Zak <kzak@redhat.com>

cfdisk: make menus sensitive to ESC

Reported-by: Benno Schulenberg <bensberg@justemail.net>
Signed-off-by: Karel Zak <kzak@redhat.com>

cfdisk: use \n for messages on stdout and stderr

Reported-by: Benno Schulenberg <bensberg@justemail.net>
Signed-off-by: Karel Zak <kzak@redhat.com>

libfdisk: make disk sync() optional

 ... this allows to avoid unnecessary sync() from cfdisk.

Reported-by: Benno Schulenberg <bensberg@justemail.net>
Signed-off-by: Karel Zak <kzak@redhat.com>

docs: fix name and URL of 2.25-rc1

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

libmount: adjust the ordering of a #define and tweak some comments

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

docs: many tiny formatting and wording improvements to man page of mount

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

docs: improve formatting and wording of a few man pages

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

build-sys: exclude Documentation/ from looking for translatable strings

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

textual: fix the usage message of kill

Using angular brackets around each individual argument, indenting a
continuation line, not using a space before =, improving some wordings,
and the argument of --queue is a value (a piece of data), not a signal.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

Merge branch 'minor-fixes' of git://github.com/kerolasa/lelux-utiliteetit

* 'minor-fixes' of git://github.com/kerolasa/lelux-utiliteetit:
  lscpu: avoid double free
  libsmartcols: avoid variable dereference after release
  libfdisk: do not do the same thing twice in single if statement
  whereis: avoid printing uninitialized string
  setpriv: avoid alloca() use xmalloc() instead
  dmesg: avoid unnecessary variable assignment
  fdisk: avoid code duplication
  mkfs.cramfs: use defined failure name rather than magic value
  fdformat: match variable and print format types
  cfdisk: add braces to ensure operation order
  lscpu: avoid use of bzero() in favor of memset()
  setterm: remove unnecessary variable
  cal: remove unnused structure and definition
  textual: fix some typos

uuidd.rc: Requires remote_fs

The Debian package checking tool lintian thinks that the script is
missing Requires on $remote_fs because it pokes at stuff under /usr
and it's probably right.

Signed-off-by: Andreas Henriksson <andreas@fatal.se>

uuidd.rc: drop on-demand mode from script

The on-demand mode was dropped from libuuid in
commit ea4f8845f0241c7 "libuuid: don't exec uuidd"

You now need systemd (socket activation) to use
uuidd on demand.

Signed-off-by: Andreas Henriksson <andreas@fatal.se>

kill: use --queue option argument as sigval integer value

The sigqueue(3) takes two values, signal and sigval.  Contents of the
signal can be altered with --signal option argument, so the --queue
argument should be reserved to affect sigval_int.

This is regression fix introduced by commit
9e8dffd5cd29f03029b1ac99eecb129532ca5c0f.

Reference: http://man7.org/linux/man-pages/man3/sigqueue.3.html
Reported-by: Benno Schulenberg <bensberg@justemail.net>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: improve the informative message when not building some util

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

docs: fix many wording and some formatting issues in the man page of kill

Also, add the most-used form of '-signal' to the synopsis.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

whereis: fix missing newline on empty results

Running "whereis foo.bar.quux" previously resulted in
printing the "foo.bar:" pattern prefix without any
newline following it.

Signed-off-by: Andreas Henriksson <andreas@fatal.se>

libfdisk: make first sector buffer usage more robust

$ ~/util-linux/fdisk -b 4096 xxx
..
Segmentation fault

The buffer has to be reinitialized always when we apply a new
device properties (e.g. sector size).

Reported-by: Bernhard Voelker <mail@bernhard-voelker.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

lscpu: avoid double free

The double free was possible for architectures other than x86_64 or i386.

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

libsmartcols: avoid variable dereference after release

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

libfdisk: do not do the same thing twice in single if statement

The second argument was very likely meant to be 'be->offset'.
Maintainers review & sign-off will further verify this change being
correct.

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

whereis: avoid printing uninitialized string

The patbuf[] needs to be filled before print out.

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

setpriv: avoid alloca() use xmalloc() instead

The getgroups() can return up to NGROUPS_MAX supplementary groups, that
is (since kernel 2.6.3) 65536 in total.  The git_t is 4 bytes, so maximum
request is 256 kilobytes.  When a system happen to have memory preasure
alloca() may not be able to allocate enough memory, making debugging
unnecessarily difficult.  IMHO 64 pages is significant enough amount of
memory to be properly error checked at a time of allocation.

Reference: http://www.gnu.org/software/libc/manual/html_node/Disadvantages-of-Alloca.html
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

dmesg: avoid unnecessary variable assignment

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

fdisk: avoid code duplication

To me having call to close() twice is less readable than one new
variable.

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

mkfs.cramfs: use defined failure name rather than magic value

From manual page; 'On error, the value MAP_FAILED is returned, and errno
is set appropriately'.

Reference: http://man7.org/linux/man-pages/man2/mmap.2.html
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

fdformat: match variable and print format types

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

cfdisk: add braces to ensure operation order

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

lscpu: avoid use of bzero() in favor of memset()

The bzero() is marked as LEGACY in POSIX.1-2001.

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

setterm: remove unnecessary variable

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

cal: remove unnused structure and definition

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

textual: fix some typos

Found with misspell-check version 2.0d.

Reference: https://github.com/lyda/misspell-check
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

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

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

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

lib/ismounted: more robust buffer usage [clang-analyze]

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

fsck: uninitialized argument value [clang-analyze]

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

libfdisk: fix possible memory leak [clang-analyze]

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

cfdisk: fix division by zero [clang-analyze]

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

libblkid: fix probe_ddf() [clang-analyze]

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

findmnt: simplify cache_set_targets()

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

libmount: always use mnt_resolve_target() in mnt_fs_match_target()

The requested path is also target mountpoint, so let's optimize
realpath() usage as well.

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

findmnt: use mnt_cache_set_targets() for non-kernel table

findmnt compares the user-supplied path <target> with each entry in the
parsed table. To do this comparison, libmount attempts to canonicalize
the target path of each table entry, when the entry does not originate
from the kernel (kernel supplied target paths are already
canonicalized). However, if one of these entries is an active mount
point, stat(2) or readlink(2) on the mount target path can hang (e.g.
unreachable NFS server).

If the main table is not a kernel table, we parse /proc/self/mountinfo
into a secondary table and call mnt_cache_set_targets(). This allows
libmount to check that the target path of each entry in the main table
is not an active mount point, so it can avoid canonicalizing it.

Signed-off-by: Eric Rannaud <e@nanocritical.com>

libmount: mnt_resolve_target: tiptoe around active mount points

Current code in mnt_fs_match_target() and mnt_table_find_target()
already does not canonicalize active mount points (when read from
mountinfo), because they are already canonicalized by the kernel.
Calling realpath(fs->target) on a mount point can hang -- e.g. if the
NFS server is unreachable.

This patch optionally extends this strategy to the general case, that is
when @fs does not directly come from the kernel through mountinfo (for
instance, it may have been parsed from /etc/fstab).

Given @mtab parsed from mountinfo, and if mnt_cache_set_targets(cache,
mtab) is used, then mnt_fs_match_target() and mnt_table_find_target()
check whether @fs->target is a known mount point in the cached
mountinfo, before attempting to canonicalize @fs->target, no matter
where @fs itself comes from. If found in the cached mountinfo,
@fs->target is not canonicalized.

[kzak@redhat.com: - don't allocate libmnt_iter,
                  - add docs for mnt_cache_set_targets(),
                  - fallback to mnt_resolve_path() if no cache->mtab specified,
                  - use streq_except_trailing_slash() to compare paths]

Signed-off-by: Eric Rannaud <e@nanocritical.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: mnt_resolve_path: don't canonicalize fs->target for swap

This is how mnt_table_find_target() does it. It makes sense because
@fs->target is "none" for swap and is never a sensible match for a
user-specified target.

Signed-off-by: Eric Rannaud <e@nanocritical.com>

libmount: mnt_resolve_path: use strcmp() only if both are canonical

Signed-off-by: Eric Rannaud <e@nanocritical.com>

fdisk: don't use --geom-* prefix for CHS options

We already use --cylinders, --heads and --sectors for sfdisk, let's
make new fdisk option compatible.

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

libsmartcols: add note about deallocations

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

swapon: mark --sumarry as deprecated

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

swapon: align --summary fields

This commit makes  partitions and files to have remaining three columns
aligned.  Below print out demonstrates earlier misalignment.

$ swapon -s
Filename                                Type            Size    Used    Priority
/dev/sda2                               partition       4194300 0       3
/home/src/util-linux/newswap            file    496     0       -1

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

umount: slightly improve the man page

Remove some stray words from option arguments.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

findmnt: minimally improve wording of usage text

Also sort the options better and tweak whitespace.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

findmnt: slightly improve wording and formatting in man page

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

lslogins: improve the columns information

Some fixes for typos, whitespace and wordings.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

lslogins: improve wording and formatting of the man page

Some typos, spacing, line breaks, and missing optional arguments.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

setterm: improve formatting and wording of usage text

Mainly: showing that the word color is a placeholder by writing
<color>, showing some numbers as optional, and message levels
and colors as not optional.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

setterm: improve wording and formatting of the man page

The main changes are: showing the arguments of boolean options
as optional, improving the alphabetization, and restoring the
--half-bright option that went missing in 2011.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

fallocate: use O_CREAT only for the default behavior

Reported-by: Bernhard Voelker <mail@bernhard-voelker.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

fallocate: fix check of number of arguments

Signed-off-by: Bernhard Voelker <mail@bernhard-voelker.de>

fallocate: clarify usage()

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

fallocate: use err_exclusive_options(), cleanup getopt_long() stuff

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

fallocate: make man page readable for humans

Well, let's copy & past from fallocate(2) syscall man page rather than
try to be creative with another description for the flags.

Reported-by: Bernhard Voelker <mail@bernhard-voelker.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

fallocate: fix FALLOC_FL_ZERO_RANGE flag check

Reported-by: Bernhard Voelker <mail@bernhard-voelker.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

docs: add note about lsblk

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

bash-completion: update fallocate, fstrim and lsblk

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

bash-completion: update cfdisk

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

bash-completion: update fdisk

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

fdisk: add long options, cleanup man page and usage()

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

docs: fix minor things in man-page howto

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