lsblk: add TRANsport to --scsi output

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

lsblk: don't ignore disk device

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

lsblk: Add --scsi switch

The --scsi switch prints similar information to lsscsi command.

Signed-off-by: Milan Broz <mbroz@redhat.com>

lsblk: Add vendor column

Signed-off-by: Milan Broz <mbroz@redhat.com>

lsblk: Add revision column

Signed-off-by: Milan Broz <mbroz@redhat.com>

agetty: fix autodetection for TERM

open_tty() autodetects whether to set TERM to "linux" or "vt102", but it
can apply the result only if op->term is still NULL.

gpt: create empty disklabels

This patch enables creating a new, empty, GPT disklabel from either
an empty disk or one that already has a disklabel. For this
purpose, a 'g' option is added to the main menu and is visible to all
labels. Here's an example for a scsi_debug device (/dev/sdb):

...
Device does not contain a recognized partition table
Building a new DOS disklabel with disk identifier 0x20a614c8.
3696: fdisk:  CONTEXT: zeroize in-memory first sector buffer

Command (m for help): g
3696: fdisk:    LABEL: changing to gpt label

3696: fdisk:  CONTEXT: zeroize in-memory first sector buffer
3696: fdisk:    LABEL: created new empty GPT disklabel (GUID: D4EA0706-F011-46DC-B7DE-6A72C7090AF8)

Command (m for help): w
The partition table has been altered!
...

Acked-by: Petr Uzel <petr.uzel@suse.cz>
Signed-off-by: Davidlohr Bueso <dave@gnu.org>

fdisk: gpt: write entire first sector

We were writing only 1 byte in LBA0, and GPT requires dealing with an entire
sector. This bug wasn't affecting when dealing with already existing devices
with GPT as we weren't writing an important part of the first sector, thus
leaving it unchanged. Also use write_all() wrapper from all-io.h

Acked-by: Petr Uzel <petr.uzel@suse.cz>
Signed-off-by: Davidlohr Bueso <dave@gnu.org>

build-sys: cleanup cpu_set_t usage

 - make taskset depend on cpu_set_t
 - make cpuset.c optional for libcommon and lib/path.c

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

sfdisk: fix formatting typo in man page

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

su: fix stupid typo

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

build-sys: move runuser(1) to sbin dir

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

runuser: add -u to not execute shell

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

mount: supports paths canonicalization for alternative fstabs

 # mount --fstab /path/my.fstab

has to support standard operations with paths.

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

libmount: support loopdevs in mnt_table_is_fs_mounted()

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

libmount: tiny refactoring in mnt_table_is_fs_mounted()

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

fdisk: fix compiler warning [-Wpointer-arith] and floating point exception

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

misc: make readlink() usage more robust

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

lsblk: use new sysfs functions, clean up get_transport()

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

lib/sysfs: add SCSI link functions

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

lib/sysfs: add functions for SCSI host attributes

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

include/sysfs: add SCSI host:channel:target:lun support

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

lib/tt: don't ignore "extreme" columns if an free space is available

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

lsblk: add SCSI transport type attribute

Patch adds "SCSI transport layer" similar attribute to "lsscsi -t".

This is useful for script where you want to distinguish e.g. FC, iSCSI
or USB devices from local disks.

Detection logic is quite simplified in comparison to lsscsi but it
should provide the same output (except detailed transport attributes).

Signed-off-by: Milan Broz <mbroz@redhat.com>

lsblk: add SCSI H:C:T:L attribute

For block devices it is sometimes useful to print SCSI device ID"
Host:Channel:Target:LUN.

Patch adds column name HCTL which can be used in lsblk.

Signed-off-by: Milan Broz <mbroz@redhat.com>

lsblk: add parent kernel name column

For raw output used in scripts it is sometimes necessary
to reconstruct tree of devices.

Parent kernel name (PKNAME) provides needed information
to do that easily.

Signed-off-by: Milan Broz <mbroz@redhat.com>

login: fix compiler warning [-Wunused-result]

It's probably unnecessary paranoia, but let's check if we're able to
restore the original IDs after ~/.hushlogin file check.

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

include/c: move usleep() fallback to c.h

To make it available everywhere in code.

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

build-sys: remove gethostbyname() check

Removed as unnecessary, there are no references to gethostbyname() in
code.

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

agetty: replace gethostbyname() with getaddrinfo()

The gethostbyname() is legacy function which may be withdrawn in a
future.

Reference: http://pubs.opengroup.org/onlinepubs/009695399/functions/gethostbyname.html
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

logger: replace gethostbyname() with getaddrinfo()

The gethostbyname() is legacy function which may be withdrawn in a
future.

Reference: http://pubs.opengroup.org/onlinepubs/009695399/functions/gethostbyname.html
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

libmount, eject: replace index() and rindex() with strrch() or strrchr()

Both index() and rindex() are legacy functions which may be withdrawn in
a future.

Reference: http://pubs.opengroup.org/onlinepubs/009695399/functions/index.html
Reference: http://pubs.opengroup.org/onlinepubs/009695399/functions/rindex.html
Acked-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

fsck.cramfs: compile with -DINCLUDE_FS_TESTS for make check

 make check
 sudo su -
 cd tests
 ./run.sh cramfs

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

docs: add line breaks to whereis.1

Fix to tools script checkmans.sh warning.

testing: /home/src/util-linux/misc-utils/whereis.1
:120: warning [p 1, 10.8i, div `an-div', 0.0i]: can't break line

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

include/c: remove MAXHOSTNAMELEN macro

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

last: stop usig MAXHOSTNAMELEN

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

write: stop using MAXHOSTNAMELEN

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

agetty: stop using MAXHOSTNAMELEN

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

login: stop using MAXHOSTNAMELEN

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

include/xalloc: add xgethostname()

The new function allocates memory by xalloc() for hostname and
fill in the buffer by gethostname().

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

include/env: unify indentation

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

include/c: add get_hostname_max() inline function

Using sysconf(_SC_HOST_NAME_MAX) does not work everywhere.  At such even
header values for hostname length should be preferred, and if nothing
else works guess the maxium being 64 bytes.

Based-on-code-by: Karel Zak <kzak@redhat.com>
CC: Mike Frysinger <vapier@gentoo.org>
References: http://www.spinics.net/lists/util-linux-ng/msg07050.html
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: gpt: use swap_efi_guid for new partitions

Signed-off-by: Davidlohr Bueso <dave@gnu.org>

fdisk: fix segfault when printing gpt raw data

Signed-off-by: Davidlohr Bueso <dave@gnu.org>

fdisk: api: improve some comments

Signed-off-by: Davidlohr Bueso <dave@gnu.org>

fdisk: api: propagate partition deletion to users

The generic fdisk_delete_partition() function returns 0 when a partition
is correctly deleted, otherwise it's corresponding error (negative values).
This, however, does not include problems that can occur in actual label
specific contexts, so we need to propagate the corresponding return code,
back to the user visible api.

Signed-off-by: Davidlohr Bueso <dave@gnu.org>

docs: update deprecated file

The goal is to consolidate the very basic linux commands and minimize
dependence on another packages (e.g. shadow-utils). It seems better to
keep newgrp, vipw and vigr as non-deprecated for now. Maybe we will
found a way how to improve the code. We will see... :-)

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

lib/loopdev: check for /sys

The current loopdev code prefers /sys to get information about
loop devices. The old methods like scan /dev are fallback solution
only. Unfortunately, the code does not check if /sys is mounted.

Addresses: http://blog.flameeyes.eu/2012/10/sophistication-can-be-bad
Signed-off-by: Karel Zak <kzak@redhat.com>

lib/loopdev: improve debug messages

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

dmesg: Add a --color option to colorize error and panic messages

[kzak@redhat.com: - update man page
                  - colorize critical messages too]

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

lib/color: add module for work with terminal colors

[kzak@redhat.com: - split from dmesg patch
                  - add more colors]

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

mount: add long options for -L and -U to man page

The -L and -U flags both have long options too (--label and --uuid).
The usage() function will show them but the man page didn't list them.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>

mount: add -c abbreviation for --no-canonicalize to man page

The --no-canonicalize option can also be set using -c.  The --help for
mount shows this option but the man page did not.  Add -c to the man
page.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>

docs: remove hwclock --compare from TODO

commit 83765871ef0d6c554c46027577648b77c7351a78 added the --compare
option to hwclock so it can be removed from the TODO.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>

hwclock: add the --compare option to the man page

The compare option for hwclock was not listed in the man page.  This
adds a simple decription of it.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>

docs: add su(1) btmp logging

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

tests: add mising "export" keyword

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

tests: skip ipcs limits overflow when bc is not available

Found only this one while trying as-root tests

Signed-off-by: Gilles Espinasse <g.esp@free.fr>

tests: fix ipcs basic limit when bc is not available

Skip basic limit test if bc is not available.
That happen to be the case when building from sources, following LFS book

Signed-off-by: Gilles Espinasse <g.esp@free.fr>

libmount: fixed comment about param

Signed-off-by: Raul Gutierrez Segales <rgs@itevenworks.net>

libblkid: fix compiler warning [-Wstrict-aliasing]

libblkid/src/superblocks/befs.c: In function 'get_uuid':
libblkid/src/superblocks/befs.c:353:6: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]

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

swapon: remove loop declaration [smatch scan]

sys-utils/swapon.c:677:2: error: 'for' loop initial declarations are
only allowed in C99 mode

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

more: remove few memory leaks

The remaining memory leaks are related to ncurses internals, or the one
open file descriptor when user users 'q' to exit.

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

more: align void in functions with prototypes, and remove void casts

If there is void in argument list at function prototype it is reasonable
to expect to see it also where the function is wrote.  This change also
removes unnecessary return value void casting.

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

more: fix coding style

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

pg: fix coding style

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

pg: do not turn off warnigns artificially

Compiler warnings often mean something, fiddling with them is not good
practise.  Besides the 'proglem' removed macro tried to 'fix' does not
even occur when compiling with modern gcc.

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

pg: use unistd.h STDOUT_FILENO

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

pg: add noreturn function attributes

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

ddate: remove from util-linux

See RedHat bug for reasons why the ddate is cleaned up.  The reference is
where to get the command in future.

Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=823156
References: https://github.com/bo0ts/ddate
Acked-by: Petr Uzel <petr.uzel@suse.cz>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
Signed-off-by: Karel Zak <kzak@redhat.com>

pg: add const qualifiers where suitable

Includes a fix also to one assignment warning (see below).

text-utils/pg.c:1477:24: warning: assignment discards 'const' qualifier \
from pointer target type [enabled by default]

[kzak@redhat.com: - use const char rather than xstrdup() for static /bin/sh string]

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

pg: use libc error printing facilities

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

pg: refactor argument handing

Add function parse_arguments(), which has the same code block that was in
over long, and too deeply intended, main().

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

docs: fix column.1 groff syntax error

Quote backslash which user is expected to see.

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

column: add --output-separator option

The --output-separator option will allow user to define table column
separator.  This will allow for example to write back same delimeter as
which was used as input separator, for example

column -t -s : -o : /etc/passwd

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

mkfs.minix: check numeric user inputs

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

pg: check numeric user inputs

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

swapon: clarify swapon.8 priority option

Value -1 is valid argument for -p, even though it might not make much
sense as the -1 is in use when priority is not specified at all.

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

swapon: check numeric user inputs

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

docs: align blkdiscard.8 with how-to-man-page.txt

* Add spaces after dots (English spacing).
* Remove use of inline emphasis where possible.
* Use mail and url macros.

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

wipefs: use O_EXCL

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

libmount: don't use umount optimization for -l or -f

The options -l (lazy) and -f (force) means that the mountpoint may be
unreadable (for example because NFS server is unreadable). So we
should not try to be smart in this case and we should try to minimize
number of situations when stat() or readlink() is used for the
mountpoint.

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

build-sys: clean update-potfiles script

 - use only sed(1)
 - remove config.h from the result
 - remove samples/, tests/, util-linu-*/ ("make ditcheck" product)
   directories from the result

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

build-sys: remove --enable-gtk-doc from make check

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

sfdisk: fix aliasing warnings

Compiles down to the same code.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>

md5: fix strict aliasing warnings

This is the same fix as was merged in gcc/binutils where this code
appears to originate from.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>

tests: fix mount move test to work with shared mount

If parent mount for test directory is mounted with shared flag,
move mount test fails because kernel rejects mount move operation.

Use another directory level and explicitly make parent mount private.

(All "modern" systems using system have mounts shared for some reason.)

Signed-off-by: Milan Broz <mbroz@redhat.com>

tests: ignore test if kernel doesn't support minix fs

Minix mount test returs failure if kernel have no minix support,
 minix: mkfs ... FAILED (minix/mkfs)
ignore test result instead in this case.

Signed-off-by: Milan Broz <mbroz@redhat.com>

tests: fix timezone for cramfs

cramfs test need explicitly set timezone otherwise
ls diff fails with
   cramfs: mkfs checksums ... FAILED (cramfs/mkfs)

Signed-off-by: Milan Broz <mbroz@redhat.com>

fix bold style of man page references

Signed-off-by: Mike Frysinger <vapier@gentoo.org>

fdisk: add AArch64 support

Attached patch fixed build problem when util-linux is built for AArch64
architecture.

mount: add verbose messages

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

libmount: fix umount file.img

The umount optimization (commit 9cc03553f7b8494c7193024eec6e13b903eb2eec)
has to be disabled if the umount argument is not a directory.

Reported-by: Milan Broz <mbroz@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

login: reuse DEAD_PROCESS utmp entries

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

hwclock: fix in usage()

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

dmesg: fix console-level off-by-one

  commit f06ec64f dmesg; support level names (e.g. --console-level=alert)

introduced an off-by-one error.  The kernel will print messages with
a *higher* level than the console-level. The bug made it impossible to
set the level for debugging, like it is documented in e.g
Documentation/networking/netconsole.txt :

 nemi:/tmp# dmesg -n 8
 dmesg: unknown level '8'

And attempting to set the "emerg" level would result in an invalid 0 value:

 nemi:/tmp# dmesg -n emerg
 dmesg: klogctl failed: Invalid argument

Restoring the old behaviour for numeric levels, and mapping the level
names so that "dmesg -n debug" behaves as expected: logging everything
at level "debug" and higher.

[kzak@redhat.com: - add comment to parse_level()]

Signed-off-by: Bjørn Mork <bjorn@mork.no>
Signed-off-by: Karel Zak <kzak@redhat.com>

column: describe change of separator behavior in man page bugs section

Add to manual page how to achieve old behavior, just in case someone
relies on buggy behavior of the command.

[kzak@redhat.com: - remove unnecessary info from the man page :-)]

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

su: fixed a typo in pam error message

Signed-off-by: Vitezslav Cizek <vcizek@suse.cz>

column: clarify the --separator option

Indicate that the string is a set,
which also clarifies that it's only
significant when parsing the input.

column: make defined separator to be non-greedy

This patch changes interpretation of subsequent delimeter interpretation.
Earlier version merged columns that had null string as content together,
which lead to output as visualized below.

$ printf "a:b:c\n1::3\n" | column  -t -s ':'
a  b  c
1  3

The number 3 has wrong column, which this patch takes care of, and alters
the output following way.

$ printf "a:b:c\n1::3\n" | column  -t -s ':'
a  b  c
1     3

This patch does not alter the default case, e.g., subsequent white spaces
are understood as separator of the same field, and the beginning of line
white spaces are being ignored together.

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