libmount: add mnt_table_{set,get}_userdata()

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

libmount: add mnt_table_with_comments()

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

pylibmount: cleanup *_HELP strings formatting

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

pylibmount: use mnt_fs_set_userdata() rather than ->userdata

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

pylibmount: fs.c - cleanup, remove libmount private stuff

 * remove unnecessary comments
 * cleanup up strings format
 * remove direct access to private libmount stuff

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

pylibmount: remove bindsrc from fs API

It's unnecessary and very low-level.

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

pylibmount: cleanup object names

Cxt -> Context
Tab -> Table

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

pylibmount: coding style changes in tab.c

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

pylibmount: coding style changes in fs.c

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

pylibmount: coding style changes in context.c

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

tests: consolidate Py tests initialization

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

build-sys: install pylibmount into

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

pylibmount: add regression tests

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

pylibmount: add __init__.py

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

pylibmount: basic code

[kzak@redhat.com: - split to more patches
                  - split to more .c files]

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

build-sys: add pylibmount

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

build-sys: remove deprecated elvtune command

The command is deprecated for years and useless with 2.6.x kernels.

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

build-sys: enable mesg(1) by default

It seems that our mesg(1) implementation is compatible with sysvinit,
let's use it by default in mainstream distros.

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

wall: make banner sysvinit compatible

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

wall: allow to specify <message> on command line

wall(1) from sysvinit supports

   # wall I love this company!

semantic, This patch add this functionally to the util-linux wall.

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

last: add link to util-linux to last.1

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

last: coding style clean up

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

last: use xalloc.h, err.h, nls.h and EXIT_ macros

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

last: consolidate btmp/wtmp file paths

 .. so share the paths with another utils.

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

last: add long options, improve usage()

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

build-sys: fix last stuff

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

build-sys: improbe UL_CONFLICT_BUILD macro

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

last: clean up the begin of the file

 .. and improve last(1) stuff in configure script.

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

last: merge sysvinit last/lastb

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

agetty: -L accepts optional argument

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

losetup: use loop-control to explicitly ask for device

Now we use LOOP_CTL_GET_FREE ioctl to ask for free device, for example

  losetup -f foo.img

Unfortunately, losetup(8) allows to ask for specified device

  losetup /dev/loop100 foo.img

and in this case we assume that the device already exists in the
system. This is incorrect, we should be able to use loop-control
LOOP_CTL_ADD ioctl to ask for the specified device.

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

blkdiscard: use fstat() rather than stat() [coverity scan]

 Calling function "open(char const *, int, ...)" that uses "path" after
 a check function. This can cause a time-of-check, time-of-use race
 condition.

.. well, in blkdiscard context it's mostly cosmetic change.

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

losetup: fix loop device name usage

The code is not paranoid enough, loopcxt_set_device() only set the
device name to loopdev struct, but it does not check if the device
really exists.

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

libmount: be robust when work with loopdev backing file paths

It's usually unnecessary as we compare devno and ino, but let's
use absolute paths for situations when it's necessary to compare
paths as strings.

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

losetup: make -j <path> more robust

It's usually unnecessary as we compare devno and ino, but let's
use absolute paths for situations when it's necessary to compare
paths as strings.

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

libmount: canonicalize for conversion from loopdev backing file

  # mount foo.img /mnt
  # umount foo.img
  umount: foo.img: not mounted

The loopdev code (and sysfs backing_file) uses absolute paths, but
libmount does not canonicalize the path before lookup for the backing file.

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

cal: set statically defined data read-only

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

more: make output redirection more efficient

Especially with large inputs the change improves performance
considerably.

old> time more /boot/vmlinuz >/dev/null
real    0m0.224s

new> more /boot/vmlinuz >/dev/null
real    0m0.009s

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

column: use variable lenght printf field width to wprint blanks

This makes program to run a little faster.  My test input show about 20%
speed improvement.

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

libmount: fix typos

Signed-off-by: Ondrej Oprala <ooprala@redhat.com>

more: guarantee space for multibyte

.. to make the code more robust.

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

more: check for buffer size when write multibyte char

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

mkswap: add note about swap header limit to the man page

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

rev: use string printing rather than character output

Fliping a string in memory, and printing it with multibyte output
function makes the command about 1/3 quicker.

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

build-sys: use backticks rather than $() for commands in configure

The configure had both backticks and $() in use.  These command
susbstitutions are mutually interchangeable, so one should pick only one
form.  In this case backticks were favored because they are known with
greater range of shells, making the backticks to be more portable.

References: https://www.gnu.org/software/autoconf/manual/autoconf-2.67/html_node/Shell-Substitutions.html#index-g_t_0024_0028_0040var_007bcommands_007d_0029-1632
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: prefer AS_IF rather than shell 'if'

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

build-sys: prefer AS_CASE rather than shell 'case'

References: http://www.gnu.org/savannah-checkouts/gnu/autoconf/manual/autoconf-2.69/html_node/Limitations-of-Builtins.html#case
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: use m4 quoting consistently

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

setarch: prefer preprocessor rather than autotools check

It seems to be pointless to spend time in ./configure phase when
preprocessor has to perform #ifndef check anyway.

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

lscpu: fix shadow declaration [smatch scan]

sys-utils/lscpu.c:572:22: warning: symbol 'fd' shadows an earlier one

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

more: fix buffer overflow

The bug has been probably introduced by commit
1ac300932deab8dea2c43050921bbbdb36d62ff1.

Reported-by: "Dr. David Alan Gilbert" <dave@treblig.org>
References: https://bugzilla.novell.com/show_bug.cgi?id=829720
Signed-off-by: Karel Zak <kzak@redhat.com>

setsid: add an option to wait child return value

This change proposed in util-linux maillist by Damien Wyart in behalf of
Daniel Kahn Gillmor.  Sami Kerola changed the patch to use option rather
than makign the default behavior of the command to change.

Reported-by: Damien Wyart <damien.wyart@gmail.com>
Reference: http://markmail.org/message/2z6qh5auk7wjie3l
Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=495881
CC: Daniel Kahn Gillmor <dkg@fifthhorseman.net>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

libmount: add missing declarations to libmount.h.in and documentation

Signed-off-by: Ondrej Oprala <ooprala@redhat.com>

libmount: change tailing to trailing in *_tailing_comments functions

Signed-off-by: Ondrej Oprala <ooprala@redhat.com>

dmesg: don't use pager for --follow

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

losetup: set errno for misaligned offsets

References: https://bugs.archlinux.org/task/36189
Signed-off-by: Karel Zak <kzak@redhat.com>

mount: update xfs docs

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

hwclock: remove references to adjtimex from man page

The adjtimex is optional on many distros.

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

sfdisk: don't use word 'disk'.

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

uuidd: add missing initializer [clang-analyzer]

Reported-by: Sami Kerola <kerolasa@iki.fi>
Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: fix memory leak [clang-analyzer]

Reported-by: Sami Kerola <kerolasa@iki.fi>
Signed-off-by: Karel Zak <kzak@redhat.com>

sulogin: fix compiler warning

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

more: use variable lenght printf field width to print blanks

This makes program to run a little faster.

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

column: dereference of null pointer [clang-analyzer]

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

ul: use string printing function

Use of for loop and printing a string character by character is slower
than to print whole string with single function.

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

ul: use correct types

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

agetty: fix memory leak [clang-analyzer]

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

sulogin: fix memory leak [clang-analyzer]

And make code a little bit more robust.

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

sfdisk: assigned value is never read [clang-analyzer]

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

chfn: assigned value is never read [clang-analyzer]

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

eject: assigned value is never read [clang-analyzer]

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

mkfs.cramfs: argument to free() is a constant address [clang-analyzer]

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

utmpdump: assigned values are never read [clang-analyzer]

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

more: assigned value is never read [clang-analyzer]

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

lscpu: fix memory leak [clang-analyzer]

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

ipcs: assigned values are never read [clang-analyzer]

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

lib/mbsalign: initializations values are never read [clang-analyzer]

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

lib/loopdev: assigned value is never read [clang-analyzer]

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

dmesg: inform user --show-delta and iso8601 time format does not mix

References: http://marc.info/?l=util-linux-ng&m=137267606929951&w=2
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: add missing options to sfdisk manual

And improve what is said about four of the dangerous options.

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

sfdisk: make unhiding as complete as possible

It is unlikely anyone will ever need this, but I like idea commands being
complete.  The partition id's that can be unhide after this change are;

Hidden HPFS/NTFS
Hidden W95 FAT32
Hidden W95 FAT32 (LBA)
Hidden W95 FAT16 (LBA)

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

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

* 'patch-1' of https://github.com/kernc/util-linux:
  Fixes "getty: unrecognized option '--loginpause'"

docs: make description of --activate clearer on man page of sfdisk

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

docs: fix wording and formatting of man page for dmesg

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

docs: improve formatting and wording of man pages for fdisk and cfdisk

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

unshare: add --mount-proc for pid namespaces

Based on patch from Mike Frysinger <vapier@gentoo.org>.

Mike Frysinger wrote:
 When it comes to pid namespaces, it's also useful for /proc to reflect
 the current namespace.  Again, this is easy to pull off, but annoying
 to force everyone to do it themselves.  So let's add a --mount-proc to
 do the magic for us.  The downside is that this also implies creating
 a mount namespace as mounting the new pid namespace /proc over top the
 system one will quickly break all other processes on the system.

Signed-off-by: Karel Zak <kzak@redhat.com>
Acked-by: Mike Frysinger <vapier@gentoo.or>

unshare: add --fork options for pid namespaces

The ability of unshare to launch a new pid namespace is a bit limited.
The first process in the namespace is expected to be the "init" for it.
When it's not, you get bad behavior.

For example, trying to launch a shell in a new pid namespace fails very
quickly:
$ sudo unshare -p dash
# uname -r
3.8.3
# uname -m
dash: 2: Cannot fork
# ls -ld /
dash: 3: Cannot fork
# echo $$
1324

For this to work smoothly, we need an init process to actively watch over
things.  But forcing people to re-use an existing init or write their own
mini init is a bit overkill.  So let's add a --fork option to unshare to
do this common bit of book keeping.  Now we can do:
$ sudo unshare -p --fork dash
# uname -r
3.8.3
# uname -m
x86_64
# ls -ld /
drwxr-xr-x 22 root root 4096 May  4 14:01 /
# echo $$
1

Thanks to Michael Kerrisk for his namespace articles on lwn.net

[kzak@redhat.com: - fix "forkif logic, remove --mount-proc]

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

include/xalloc: add warn_unused_result to allocation functions

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

lsblk: use devno to check if the filesystem is mounted

The device (for example LVM logical volume) could be renamed and then
the device name from /proc/self/mountinfo does not match with reality.
So, we also need to check devno. Unfortunately we cannot completely
rely on devno, because for example btrfs uses psudo device numbers.

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

lsblk: use devno to get mountpoint

The device (for example LVM logical volume) could be renamed and then
the device name from /proc/self/mountinfo does not match with reality.
So, we also need to check devno. Unfortunately we cannot completely
rely on devno, because for example btrfs uses psudo device numbers.

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

Fixes "getty: unrecognized option '--loginpause'"

Usage incorrectly stated --loginpause is a valid option, where instead --login-pause is.

Also --no-hostname --> --nohostname.

findmnt: avoid unused parameter warning

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

tests: update libmount fstab full parse test

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

libmount: add functions to safely replace a fs table

Signed-off-by: Ondrej Oprala <ooprala@redhat.com>

libmount: add functions to handle comments in fs tables

Co-Author: Karel Zak <kzak@redhat.com>
Signed-off-by: Ondrej Oprala <ooprala@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: add a generic append_string() function

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

libmount: be more restrictive about valid tag names

 # mount DUMMY=filename.img /mnt

The 'DUMMY=filename.img' is a filename and should not be
interpreted as tag name. The valid tag names are LABEL, UUID,
PARTLABEL and PARTUUID only.

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

lib/canonical: use realpath() from libc

.. after many many years it seems that we can follow libc code rather
than duplicate realpath(3) implementation.

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

findmnt: make udev usage more robust

 $ findmnt -rn --target /home/kzak/.bashrc -o UUID

uses libudev to ask for UUID, unfortunately /home/kzak is DM device
and libudev expects sysfs-like names (e.g. dm-0). We have to convert
the name to kernel devname.

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

dmesg: make time stamps to be printed consistently

Earlier uptime determination, which was done with sysinfo(2), had one
second resolution, which made time stamps to be rounded unstable way
depending on when a dmesg command was executed.  In practical terms; the
command below was supposed not to differ but it did.

$ diff -q <(dmesg --ctime) <(sleep 0.5 ; dmesg --ctime)

[kzak@redhat.com: - add -lrt to Makemodule.am,
                  - fallback to sysinfo() based boot time
                  - use #ifdef]

CC: Kay Sievers <kay@vrfy.org>
References: https://lkml.org/lkml/2013/6/30/37
Buglink: https://github.com/karelzak/util-linux/issues/24
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
Signed-off-by: Karel Zak <kzak@redhat.com>