ctrlaltdel: minor changes to usage()

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

arch: minor changes to usage()

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

Merge branch 'sys-utils' of https://github.com/kerolasa/lelux-utiliteetit

* 'sys-utils' of https://github.com/kerolasa/lelux-utiliteetit: (29 commits)
  docs: add non-return function and if shorthand tips
  build-sys: fixes to USAGE_* macros
  ipcrm: check IPC syscalls
  ipcrm: add --verbose option
  ipcmk: allow high speed ipc creation
  ipcrm: add --all option
  docs: add long options to ipcs.1 man page
  docs: add long options to ipcrm.1 man page
  docs: add long options to ipcmk.1 man page
  docs: add --version to setarch.8
  docs: mention long options in ctrlaltdel.8
  ctrlaltdel: add version & help options
  docs: mention long options in pivot_root.8
  pivot_root: add version & help option
  ipcs: comment & white space clean up
  ipcs: include-what-you-use header check
  ipcs: add long options
  ipcrm: include-what-you-use header check
  ipcrm: refactor new and old main to share code
  ipcrm: exit if unknown error occurs
  ...

mount: use common libs

Get rid of the local xmalloc.[c/h] files by using the global xalloc and
strutils libraries.

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

lscpu: add column names to --help

 * use uppercase by default (only -p uses lowercase)
 * a little cleanup usage()

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

lscpu: gettextize two columns

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

agetty: test ECHO on c_lflag

Reported-by: xinglp <xinglp@gmail.com>
Addresses: http://bugzilla.redhat.com/show_bug.cgi?id=739522
Signed-off-by: Karel Zak <kzak@redhat.com>

docs: add non-return function and if shorthand tips

Non-return functions should not be combined with `else' clause.

The if shorthands `var = e ? t : f;' need to fit to single line,
and if that does not look good use normal "if else" syntax.

Both tips are mentioned in email bellow.

http://www.spinics.net/lists/util-linux-ng/msg05152.html

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

build-sys: fixes to USAGE_* macros

The USAGE_BEGIN_TAIL is removed as unnecessary.

In between command specific options and --help & --version
USAGE_SEPARATOR is inserted. For now the separator is empty line.

The USAGE_MAN_TAIL is changed to take an argument.

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

ipcrm: check IPC syscalls

It's not enough to check errno for errors as the variable is not
reset, we also need to check the last syscall return value to
verify a problem. This addresses bogus msgqueue errors when
deleting keys.

Signed-off-by: Davidlohr Bueso <dave@gnu.org>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcrm: add --verbose option

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

ipcmk: allow high speed ipc creation

This commit will allow user to create quickly plenty of ipc
resources. Earlier the line bellow gave no satisfaction.

for i in $(seq 0 42); do ipcmk -Q; done

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

ipcrm: add --all option

An --all option will remove all ipc entries. The option takes
optional resource argument, which limits the removal to be
applied only the given resource entries.

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

docs: add long options to ipcs.1 man page

Includes few style fixes from Documentation/howto-man-page.txt
and align information by what is being said by The Open Group.

http://pubs.opengroup.org/onlinepubs/009604599/utilities/ipcs.html

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

docs: add long options to ipcrm.1 man page

Includes few style fixes from Documentation/howto-man-page.txt

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

docs: add long options to ipcmk.1 man page

Includes few style fixes from Documentation/howto-man-page.txt

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

docs: add --version to setarch.8

Includes few style fixes from Documentation/howto-man-page.txt

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

docs: mention long options in ctrlaltdel.8

Includes other manual page maintenance fixes.

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

ctrlaltdel: add version & help options

Including other necessary changes to usage().

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

docs: mention long options in pivot_root.8

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

pivot_root: add version & help option

Including other necessary changes to usage().

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

ipcs: comment & white space clean up

Add to multiline comments to have left side stars, convert spaces
to tabs and indent preprocessor directives

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

ipcs: include-what-you-use header check

Two includes added & a sort to alphabetical order.

ipcs.c should add these lines:
 #include <features.h>                   // for __GLIBC__
 #include <stddef.h>                     // for size_t

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

ipcs: add long options

Includes necessary usage() changes.

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

ipcrm: include-what-you-use header check

Three removes & a sort to alphabetical order.

ipcrm.c should remove these lines:
- #include <ctype.h>  // lines 29-29
- #include <sys/types.h>  // lines 18-18
- #include <unistd.h>  // lines 27-27

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

ipcrm: refactor new and old main to share code

The code is now much more tidy, and as a bonus old main has
shares error printing with the `new' code.

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

ipcrm: exit if unknown error occurs

Previously for instance lack of memory space caused unknown
reason to be printed, and ipcrm tried to continue.

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

mount: check for target before source on remount

Addresses: http://bugzilla.redhat.com/show_bug.cgi?id=737091
Reported-by: Eric Paris <eparis@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

lib: do not attempt to close(0) in sysfs_deinit()

If the 'open' fails we 'goto err' which runs 'sysfs_deinit()' on a 'cxt'
which hasn't been fully initialised. The 'dir_fd' is still 0, so
sysfs_deinit calls "close(0)".

Addresses: https://bugzilla.novell.com/show_bug.cgi?id=714151
Reported-by: Diego Ercolani <diego.ercolani@gmail.com>
Analysed-by: Neil Brown <nfbrown@suse.com>
Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

chfn: remove non-ANSI definitions [smatch scan]

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

build-sys: add err and errx to smatch no_return_funcs

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

build-sys: add smatch data

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

ipcrm: add long options

With long options usage function had to be changed. The change
also takes libc error printing facilities to use, primarily to
get rid of execname & progname variables.

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

ipcmk: include-what-you-use header check

Three removes & a sort to alphabetical order.

ipcmk.c should remove these lines:
- #include <string.h>  // lines 24-24
- #include <sys/types.h>  // lines 30-30
- #include <unistd.h>  // lines 29-29

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

ipcmk: remove camel casing

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

ipcmk: validate numeric option arguments

Use strtoul_or_err() instead of atoi().

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

ipcmk: remove useless code

Checking same return value twice does not make command any
better. Secondly the program_invocation_short_name is known to
work, so global progname variable does not add anything extra.

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

ipcmk: add long options & fix usage()

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

script: non-ANSI definition [smatch scan]

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

whereis: non-ANSI definition [smatch scan]

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

logger: non-ANSI definition [smatch scan]

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

cal: non-ANSI definition [smatch scan]

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

blkid: potential null derefence [smatch scan]

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

libmount: previously assumed null pointer [smatch scan]

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

kill: potential null derefence [smatch scan]

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

libmount: potential null derefence [smatch scan]

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

libmount: variable dereferenced before check [smatch scan]

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

libmount: redundant null check on calling free() [smatch scan]

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

libmount: use unsigned int for bitfields [smatch scan]

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

libblkid: remove unnecessary position check from list_for_each_safe loops

The position pointer is dereferenced before the check in the
for() in the list_for_each_safe macro.

Note that struct list_head should not contain NULL pointers. See
INIT_LIST_HEAD() macro.

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

build-sys: don't use full paths for smatch

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

agetty: add --nohints

Reported-by: "Gabor Z. Papp" <gzp@papp.hu>
Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: use --enable-most-builds for distcheck

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

include,c.h: include fcntl.h before check for O_CLOEXEC [smatch scan]

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

build-sys: add checksmatch target

lscpu: remove unneeded file

As noted by Sami I accidentally added a garbage file.
Just remove it again.

Cc: Sami Kerola <kerolasa@iki.fi>
Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

fdformat: fix uninitialized variable

Spotted by EKOPath compiler.

Signed-off-by: Francesco Cosoleto <cosoleto@gmail.com>

Merge branch 'smatch' of https://github.com/kerolasa/lelux-utiliteetit

* 'smatch' of https://github.com/kerolasa/lelux-utiliteetit:
  kill: fix issue warned by smatch
  last: fix smatch warning
  libblkid: fix smatch warning
  column: fix error reported by smatch
  build-sys: ddate typo fixed
  build-sys: add --enable-most-builds

lscpu: s/Y/Yes/ s/N/No/

However I'd like to see one change if you don't object: printing just "N" or
"Y" instead of "No" and "Yes" in the human readable output looks a bit ugly to
me.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

setarch: add version printing

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

setarch: use program_invocation_short_name

Trust that the c.h is setting program name correctly.

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

setarch: move options struct to function scope

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

kill: fix issue warned by smatch

misc-utils/procs.c +82 get_pids(52) error: potential null derefence 'pids'.

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

last: fix smatch warning

login-utils/last.c +447 ttyconv(11) error: buffer overflow '(arg)' 3 <= 3

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

libblkid: fix smatch warning

libblkid/src/probe.c +798 blkid_probe_get_idmag(32) error: we previously assumed 'id' could be null.

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

column: fix error reported by smatch

column.c:201:2: error: 'for' loop initial declarations are only allowed in C99 mode

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

build-sys: ddate typo fixed

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

build-sys: add --enable-most-builds

The --enable-most-builds will build everything else but
experimental code.

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

lib,cpuset: fix compiler warning [-Wuninitialized]

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

chcpu,cpuset: reduce code duplication for cpu list parsing

Reduce code duplication and print better error message if an
unsupported cpu number was passed.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

lib,cpuset: enforce stricter parsing of cpu lists

The current cpulist_parse() function ignores extra non-parsable characters at
the end of the to be parsed cpu list string.  E.g. it would accept something
like "0bla" and just set bit 0 in the cpu set.  Since such a string is invalid
implement stricter parsing that makes sure that everything of the string has
been succesfully parsed.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

lib,cpuset: fix stride handling in cpulist_parse()

If cpulist_parse() gets passed a cpu list with a stride value of 0 it will be
stuck in an endless loop.  E.g. the following cpu list will cause an endless
loop: "0-2:0".  Fix this by causing a parse error if the stride value is 0.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

lib,cpuset: fix odd placed braces in cpulist_parse()

The opening and closing braces for two following if statements within
cpulist_parse() are placed in an odd manner.
Just fix this to prevent broken code in the future.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

lib,cpuset: fix comments for cpu mask/list functions

The comments for cpumask_parse() and cpulist_parse() each describe
the wrong function. Just exchange the comments.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

mount: remove superfluous "an" under ext4 mount options in mount.8

Signed-off-by: Frederik "Freso" S. Olesen <freso@freso.dk>

lscpu: add Hypervisor to output

Some vendors have several hypervisors. Therefore it makes sense to not only
print out the hypervisor vendor but also the name of the hypervisor.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

lscpu: add s390 test case

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

lscpu: add --online option

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

lscpu: add --all option

The parsable output includes only lines of online CPUs. To also include
lines for all offline CPUs the "--all" option can be specified.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

lscpu: add online state to output

lscpu only prints lines for online CPUs. At least for the human readable
list the offline CPUs are of interest as well. In order to distinguish
between online and offline CPUs introduce the "Online" column.
By default the human readable output now displays online and offline CPUs.
The parsable output is not changed. It will print only lines for online
CPUs as it used to do.

[kzak@redhat.com: - minor changes]

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

lscpu: add configured state to output

CPUs may be in a configured or deconfigured state depending if the CPU resource
may be used by the guest.  If a CPU is in configured state the guest may use it
(i.e. set it online). It it is in deconfigured state it cannot use it before
changing its state to configured.  Display this CPU attribute as well.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

lscpu: add human readable extended cpu table output

Based on patch from Heiko Carstens <heiko.carstens@de.ibm.com>:

lscpu currently only supports a parsable output which contains a row for
each cpu and its attributes. This output contains only comas as separators
and is hard to read for humans.

Therefore add a new option "-e | --extended" which outputs the rows in a
much more readable (and non-parsable) form. Just like for the -p option a
list of columns can be specified that shall be included in the output.

By default this option will print all columns that contain data.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

lscpu: use buffer for data and headers

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

lscpu: use function to search in cpu masks arrays

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

lscpu: cleanup output modes

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

lscpu: add --version option

Add a --version option like most other tools have it.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

lscpu: allow read_cache() to be called for offline cpus

First check path before accessing files to be sure they actually exist. This is
necessary when also informations for offline CPUs will be printed.  Since we do
not necessarily know if "cpu is offline" means the same as "path does not
exist" just check for it.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

lscpu: simplify cache column output function

Simplify the logic to "always print a ',' for each cache except if it is the
last one.  This is also a preparation patch for printing the cache column for
offline CPUs where it would print one colon too much because of the current
logic.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

lscpu: fix cache output for extended parsable output

The extended parsable output prints a colon instead of comma between each
item. The case where a CPU doesn't belong to any cache was not converted.
Just fix this.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

lscpu: remove comma operator

Fix typo where the comma operator has been introduced.
Use a semicolon instead so we end up with simple assignment expressions.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>

mount: fix compiler warning [-Wunused-parameter]

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

dmesg: add --file option

This new option is mostly for testing (to test raw logs from another
boxes/kernels).

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

script: libintl.h included in nls.h

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

libblkid: DRBD proxy data-log identification.

Signed-off-by: Philipp Marek <philipp.marek@linbit.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: new magic for 8.4 style (unclean) activity log.

Signed-off-by: Philipp Marek <philipp.marek@linbit.com>

libuuid: add limits.h include for OPEN_MAX

Needed as definition for usage in close_all_fds().
Noticed on klibc build.

Signed-off-by: maximilian attems <max@stro.at>

agetty: double equals sign typo in opentty()

Reported-by: Francesco Cosoleto <cosoleto@gmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

script: fix building with --disable-nls

We should only include the libintl.h when NLS is requested.

This fixes issue when building util-linux with uClibc.

Signed-off-by: Natanael Copa <ncopa@alpinelinux.org>

write: fix path for freopen()

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

dmesg: remove extra space only after time stamp

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