column.c: coding style fixes

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

column.c: add version printing

The patch makes return value to be non-zero when command line
short option is unknown.

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

column.c: replace emalloc with xcalloc

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

docs: agetty manual update

Tell users how to see what issue escape will print by using other
commands, such as uname.

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

agetty.c: move issue path to pathnames.h

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

agetty.c: further scrubbing

In reply to two requests.

http://permalink.gmane.org/gmane.linux.utilities.util-linux-ng/3990
http://permalink.gmane.org/gmane.linux.utilities.util-linux-ng/3999

The scubbing consists;

o Old school introduction back to the top of the source file.
o Indentation changes.
o Removal of void casting.
o Removal of braces from single statements.
o Unified format for comments.
o Comparison with W. Venema's code from 1993 to clean few comment
  mysteries.
o Web referrals added to further explanation to near the code or
  comments which some could find difficult to understand why.
o Clean up of comments.
o FIXME items added where attention is needed.
o Make functions static.
o Functions with same argument types use same argument order.
o Get rid of classic K&R style function declarations.

Please notice that this patch _should not_ affect how program
works e.g. this has no actual code changes.

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

findmnt: minor coding style changes

 * remove __<MACRONAME> from code
 * use 'static' for global variables

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

findmnt: filter filesystems and actions for --poll

The patch allows to filter by:

 * actions name:
$ findmnt --poll=umount

 * target, source, fstype, options:
$ findmnt --poll /mnt/test
$ findmnt --poll -O ro
$ findmnt --poll -t ext4

 * --first-only option to wait only for the first matching change:

$ findmnt --poll=umount --first-only /mnt/test

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

findmnt: print OLD-* columns only when necessary

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

findmnt: add --poll and --timeout to the man page

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

findmnt: add --timeout option

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

findmnt: add --poll option

The --poll functionality is based on poll(/proc/self/mountinfo) and
mnt_tabdiff_* functions from libmount.

 $ findmnt --poll
 ACTION     TARGET    SOURCE    FSTYPE OPTIONS
 mount      /mnt/test /dev/sda1 ext3   rw,relatime,errors=continue,user_xattr,acl,barrier=0,data=ordered

... after "mount /dev/sda1 /mnt/test".

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

libmount: minor fix to mnt_tabdiff_*

 - tabdiff has to support NULL fs->source
 - disable some debug messages

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

lib: [tt.c] support fixed width and multiple tt_print_table() calls

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

libmount: add mnt_tabdiff_* functions

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

libmount: add mnt_reset_table()

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

libmount: support NULL source path for mnt_table_find_* functions

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

build-sys: non-linux fixup

Signed-off-by: Samuel Thibault <samuel.thibault@ens-lyon.org>

libmount: add support for x-* mount comments

The current way how add any comment or extra information to /etc/fstab
is to use comment="" mount option. This is not too elegant and readable
if you want to add more than one comment.

It seems better to add any generic prefix that will be used for all
3rd-party userspace mount options.

Expected syntax is:

  x-<application>-<options>[=<data>]

For example:

  x-systemd-bar=managed

All x-* options (as well as comment=) are ignored by libmount and not
stored to the mtab file.

Requested-by: Lennart Poettering <lennart@poettering.net>
Signed-off-by: Karel Zak <kzak@redhat.com>

tests: don't run mount/regfile test on old kernel

The /sys/.../loop/backing_file is available since 2.6.37.

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

tests: add test for RLIMIT_FSIZE issue

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

tests: use libmount tab update tests for UID=0 only

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

getopt: fix gcc warning

getopt.c:80:13: warning: ‘set_initial_shell’ declared ‘static’ but
never defined

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

mesg: add \n to usage() output

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

mesg: man page clean wqup

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

mesg: new --verbose option

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

mesg: add long options to the man page

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

mesg: use long options, also --help and --version added

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

agetty: coding style - fix identation

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

agetty: coding style - fix comments, remove tailing whitespace

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

agetty: getopt case segment reordering

The case clause after getopts is in alphabetical order, just like
option string definition and manual page. The patch also convers
spaces to tabs in this code block.

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

docs: agetty long options

Long options to manual page with the new --version and --help.
The manual page option order is also set to be the same as it is
in code. For the persons who don't see the order it is short
option case in sensitive alphabetical order.

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

agetty: symbolic standard file descriptors

I find it easier to read code in which standard file descriptors
are symbolic strings rather than numbers.

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

agetty: #endif comments

The longer #ifdef ... #endif markups have now at the end section
comment where they begun, which should improve preprosessor
directive readability.

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

agetty: error message about required arguments

This patch also fixes only remaining magical number to EXIT_*
markup.

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

agetty: use long options

This patch will introducing --help and --version options, which
does not have short version. Reason to this is a that the -h was
already assigned.

[kzak@redhat.com: - fix one usage call,
                  - use %1$s in usage printf format
                  - remove "static const" from enum]

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

agetty: use program_invocation_short_name

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

agetty: use xalloc.h

Unifies and fixes malloc and strdup calls.

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

libmount: use libmnt_lock for utab flock

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

libmount: merge mtab and utab locking code

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

libmount: block signals when update utab

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

libmount: more robust mtab and utab update (CVE-2011-1676, CVE-2011-1677)

http://thread.gmane.org/gmane.comp.security.oss.general/4374

Changes:

 - always use temporary file

 - use fflush() for the temporary file

 - check fprintf() return value

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

mount: use lock from mnt_update_table()

The mnt_update_table() is able to create a mtab lock now.

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

umount: block signals when umounting and updating mtab (CVE-2011-1676, CVE-2011-1677)

http://thread.gmane.org/gmane.comp.security.oss.general/4374

Based on CVE-2011-1089.

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

mount: use fflush() and temporary file for mtab updates (CVE-2011-1089)

http://thread.gmane.org/gmane.comp.security.oss.general/4374

Changes:

  - force mount(8) to use /etc/mtab.tmp file every time. The original
    code used the tmp file for remount/move operations only.

  - call and check fflush() return code for the tmp file

Note mount(8) blocks all signals when writing to mtab, so it's not
affected by SIGXFSZ and the mtab lock file is always removed.

This patch does not fix the same issue in umount(8) and libmount.

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

libblkid: remove test for non-zero head count in FAT superblock probe.

This change enables the auto-detection of the internal storage on a VTech
Kidizoom camera, which has a zero head count value in the superblock.

Microsoft documentation, "FAT: General Overview of On-Disk Format", states
that the sectors and heads fields are only relevant for media that have
a geometry.

The Linux kernel also does not require a non-zero head count, and can
manually mount the filesystem.

Signed-off-by: Nick Holloway <Nick.Holloway@pyrites.org.uk>

script: optional timing output file argument added

And update to manual page accordingly.

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

docs: scriptreplay add note about new options

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

scriptreplay: allow arguments as command line switches

The command line switches override old style arguments

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

docs: scriptreplay mention basic long options

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

scriptreplay: support long options

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

scriptreplay: end printing with new line

Depending on opinion this is either modest bug fix or tiny
improvement.

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

libmount: avoid redundant declaration of mnt_context_do_umount

Signed-off-by: Jan Engelhardt <jengelh@medozas.de>

mkswap: add alignment check

This patch produces the program to warn if the given device is misaligned.

[kzak@redhat.com: - call the check for block devices only]

Signed-off-by: Davidlohr Bueso <dave@gnu.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

blkdev: add blkdev_is_misaligned()

This functions returns the status of the device's alignment. It will
be 0 when aligned, otherwise return the offset.

[kzak@redhat.com: - returns 0 if the ioctl failed]

Signed-off-by: Davidlohr Bueso <dave@gnu.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

hexdump: don't include err.h directly

script: don't include err.h directly

iosize: fix gcc warning

isosize.c: In function ‘main’:
isosize.c:168:8: warning: unused variable ‘p’

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

elvtune, isosize: print usage text in case of invalid option

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

isosize: use program_invocation_short_name

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

isosize: simplify some error messages

perror() and fprintf() are replaced with err() calls

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

elvtune, isosize: remove redundant message in case of invalid option

This leaves getopt() only to print a similar error message on invalid
options.

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

readprofile: don't stop parsing at __init_end

Some architectures like ARM place __init_end before .text section.
If any function in .init section is hit while profiling, readprofile
stops at __init_end.  That means if we enable profiling at boot time,
the profiler probably hits init functions and readprofile does not
work well unless we reset profiling buffer with -r option.

Signed-off-by: Po-Yu Chuang <ratbert@faraday-tech.com>

wall: support --timeout switch (vol 2.)

This is the rest of the commit

cae7485e59299c4b70ea7e1c7a4206df416701dc

the original patch was committed without man page and Makefile
changes. Sorry.

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

wall: cleanup mbufsize usage

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

wall: support --timeout switch

The switch controls message write time out to terminals.

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

wall: add long options and 79 char cut info to the man page

Mention that over 79 characters long lines are cut, while long
options is an obvious change.

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

ttymsg: fix compiler warnings and use EXIT_

[kzak@redhat.com: - remove 'register' storage class specifier]

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

wall: use long options

Also --version & --help options added, and rewrote help output.

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

wall: remove global variables and support TMPDIR

The TMPDIR environment is preferred over _PATH_TMP in which a
file is created that is ultimately sent to terminals. This patch
will also fix compiler warnings and removes magical numbers,
which has not explanation.

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

mount: add note about blank lines to fstab.5

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

umount: support non-canonical devnames in mtab

Addresses: https://bugs.launchpad.net/ubuntu/+source/util-linux/+bug/755193
Signed-off-by: Karel Zak <kzak@redhat.com>

docs: update TODO

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

setterm: use xmalloc()

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

setterm: fix variable types, and unused argument warnings

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

setterm: add note about long dump file paths to man page

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

setterm: function screendump coding style fixed

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

setterm: mention in man setterm.1 -version and -help switches

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

setterm: use libc error printing facilities

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

setterm: support -help and -version options

[kzak@redhat.com: - merge all options to the one string]

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

setterm: segfault with long dump file name

The patch will make too long path to be truncated to MAX_PATH
lenght. If user supplies too long file name that is caught at
fopen. Some users might find this unintuitive, so the altered
behavior will be mantioned in manual page.

Bug in earlier version was visible with a command bellow.

setterm -dump -file $(for i in $(seq 254); do printf x ; done)

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

column: use xalloc lib

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

rev: mention long options in man page

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

rev: option parsing bug fixed & long options added

The former getopts segment gave impression unknown options will
cause the program to exit with error and help is available with
-h. Neither work quite as designed, all unknown options made the
program to exit with success; and none of the options where
known.

The fix also has support for long options, and new --version
switch.

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

rev: memory leak fix

Moving the malloc out of loop will make leak to disappear, and
the command might run few jiffie quicker when there are 1+N
arguments.

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

rev: use xrealloc from xalloc.h

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

partx: trivial comment fix

Replace atgv for argv

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

getopt: add static qualifiers

Signed-off-by: Olivier Mengué <dolmen@cpan.org>

build-sys: remove check-news

I need to call "make distcheck" often during development phase to check
that all files are correctly added to Makefiles. The "check-new"
automake option is useless if $(VERSION) is generated from git.

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

build-sys: use git-version-gen to distinct git and release versions

The script is copied as is from gnulib.

[kzak@redhat.com: - generate .tarball-version and .version files in
                    top level Makefile.am
                  - delete autom4te.cache in autogen.sh]

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

libmount: fix parsing of mountinfo from 2.6.39

The /proc/self/mountinfo file uses " - " field as a separator between
optional fields and next fields in the file. The '-' char could be
used in the fields (for example in UUIDs), so it's necessary to check
for whole " - " string rather than for '-' char only.

Reported-by: "Aneesh Kumar K. V" <aneesh.kumar@linux.vnet.ibm.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: fix distcheck for term-utils

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

libmount: add mountpoint(1) implementation to samples/

umount: use helper= for all UIDs

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

libmount: support /run/mount rather than /dev/.mount

The /run directory should be preferred on distributions where this
directory exists.

http://thread.gmane.org/gmane.linux.redhat.fedora.devel/146976

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

tests: add dump from PPC64 to lscpu tests

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

lscpu: detect sun4{u,v} in /proc/cpuinfo for sparc64

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

tests: add dump from UltraSparc T1 to lscpu tests

Thanks to Jan Engelhardt.

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

tests: add dump from huge ia64 to lscpu tests

Thanks to Jan Engelhardt.

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

lscpu: fix op-mode for /{sys,proc} dumps

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