docs: align vipw.8 with howto-man-page.txt

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

docs: fix return value table in fsck.8

Commint also has few alignments with howto-man-page.txt

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

docs: chcpu and readprofile are sysadmin utils

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

tests: add loopdev libmount test

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

libmount: support {offset,sizelimit}= foor loopdevs

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

libmount: add function to parse offsets/sizes

ipcrm: fix wrong subject in error messages for -m, -q, -s id options

This makes error messages print 'id' instead of 'key' using id options.

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

ipcrm: use ternary operator in "(key|id) failed" message

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

ipcrm: add static keyword to some functions

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

misc: use %m in format string instead of %s and strerror(errno)

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

docs: update TODO

TODO items already done by Sami Kerola

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

sfdisk: warn if partition exceeds partition table limits

Warn the user if the partition to be created exceeds limits
imposed by the DOS partition table, which is:
* partition has to start on sector < 2^32
* partition size has to be < 2^32 sectors

For 512-byte logical sector size, these limits are ~2.2 TiB.

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

sfdisk: use unsigned long long to internally represent sector number

Unsigned long is only 4 bytes long on i586, which is not enough to
represent sector number on todays large disks. Use unsigned long long,
which is 8 bytes long, to store the sector numbers internally, so that
we could later make some sanity checks and warn the user if the sector
numbers exceed the limits imposed by DOS partition table format.

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

sfdisk: introduce get_ull() function

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

sfdisk: get_disksize() returns long long

unsigned long is 4 bytes long on i586, which is not enough for big
HDD's with 512B sectors. Use unsigned long long, which is 8 bytes.

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

sfdisk: make the cylinder boundary check less fatal

If the specified format is not cylinders, make the cylinder boundary
check only print a warning and proceed anyways.

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

sfdisk: fix weird indentation in msdos_partition()

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

sfdisk: file descriptor is of int type

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

blkid: fix unfinished sentence for -l option in blkid.8

Signed-off-by: Dave Reisner <dreisner@archlinux.org>

uuidd: Add missing break to option case statement

Specifying the "-n" option to uuidd would incorrectly
fall through to the "-p" case, and assign that number to
the pidfile_path.

Signed-off-by: Eric Sandeen <sandeen@redhat.com>

libmount: fix use of logical '&&' with constant operand

Found by clang.

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

lib,loopdev: fix use of logical '&&' with constant operand

Found by clang.

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

flock: make flock(1) work on NFSv4

To pleace an exclusive lock on a file, NFSv4 requires
the file to be opened RW because of the emulation of flock()
by fcntl():

http://www.spinics.net/lists/linux-nfs/msg18502.html

So instead of O_RDONLY, open the file in O_RDWR if access()
indicates it is possible (unless shared lock is requested).

From: Michal Kubecek <mkubecek@suse.cz>
Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

build-sys: add path.h to Makefile.am

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

lscpu: stricter command line parsing

Disallow superfluous commands for lscpu like e.g. "lscpu bla" and let it
fail print the help text instead.

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

lscpu: add --offline option

Implement "--offline" option which only prints offline cpus. As a side effect
we can get rid of the internal "allcpus" flag, since if we want to print
informations for online and offline cpus we simply set both flags.

When reading sysfs attributes of cpus this is now done for all cpus, since
e.g. the topology informations of the online cpus may influence the
topology informations of the offline cpus. This mainly because online cpus
may contain masks which include offline cpus while offline cpus have a
missing topology directory.

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

lscpu: fix -e output

The modifier mod->allcpus must be set earlier and also must be used
earlier. The current code only reads sysfs attributes from online
cpus but skips offline cpus.
So initialize mod->allcpus earlier.

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

chcpu: provide better user feedback

Instead of printing error messages like "I/O resource busy" which are
supplied by strerror, give better feedback if the reason of failure
is known.
E.g. taking the last cpu offline cannot succeed, therefore print a
message that gives this "hint".

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

lib,path: use write_all()

From: Heiko Carstens <heiko.carstens@de.ibm.com>

Since write() doesn't necessarily write the complete buffer with
one call we better use write_all() which takes care of this.

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

chcpu: convert to use lib/path.c

Use the common path access functions. In order to simplify chcpu also implement
and use path_writestr() which writes a string to the path specified.

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

lib,path: move path access functions from lscpu into lib/path.c

A couple of these functions already have been copied to chcpu.c,
so it makes sense to move these functions into an own file.

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

setarch: minor changes to usage()

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

pivot_root: minor changes to usage()

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

ipcs: minor changes to usage()

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

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>