libmount: Ensure utab.lock mode 644

If utab.lock is created by a process with a restricted umask, utab.lock is
created with restricted permissions. It breaks userspace monitor.

Ensure that the mode is always 644.

Signed-off-by: Stanislav Brabec <sbrabec@suse.cz>

bash-completion: update fincore

Add RES column to --output completions.

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

bash-completion: update hwclock

Add --udate-drift [1], remove --badyear [2], and use the same argument
for --date as in manual page[3].

Commit [1]: f276d71a3ef6a65299deb5e14a188e0a786ecebd
Commit [2]: f6374e1fb33d5ca726b72212b787e11fea78c0ca
Commit [3]: cc5ec6936ea3c218b3e8c46880659db5ee01329f
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

Merge branch '170409' of github.com:jwpi/util-linux

* '170409' of github.com:jwpi/util-linux:
  hwclock: remove residual 'compare' from man-page
  hwclock: fix whitespace in hwclock-rtc.c
  docs: parse-date.txt fix whitespace
  parse-date: remove extraneous form feed
  hwclock: use errtryhelp()

script: don't call strftime() if not necessary

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

Merge branch 'script_quiet' of https://github.com/renyuneyun/util-linux

* 'script_quiet' of https://github.com/renyuneyun/util-linux:
  script: fix start message showing in output file when -q is on

libsmartcols: fix typo

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

lsblk: fix --inverse --list

Let's follow --inverse dependencies although --list output requested.

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

libsmartcols: add scols_sort_table_by_tree()

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

lsblk: move scols_flags to lsblk control struct

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

hwclock: remove residual 'compare' from man-page

Signed-off-by: J William Piggott <elseifthen@gmx.com>

hwclock: fix whitespace in hwclock-rtc.c

Signed-off-by: J William Piggott <elseifthen@gmx.com>

docs: parse-date.txt fix whitespace

Signed-off-by: J William Piggott <elseifthen@gmx.com>

parse-date: remove extraneous form feed

* lib/parse-date.y: Remove extraneous form feed (FF, 0x0C, ^L).

Signed-off-by: J William Piggott <elseifthen@gmx.com>

hwclock: use errtryhelp()

* sys-utils/hwclock.c: use errtryhelp() instead of usage().

Signed-off-by: J William Piggott <elseifthen@gmx.com>

agetty: various man-page fixes

Signed-off-by: J William Piggott <elseifthen@gmx.com>

chmem: don't use scanf format for printf

The compiler had not complained here because both macros are
probably the same.

But gettext issued a funny warning:
sys-utils/chmem.c:67: warning: Although being used in a format string position, the msgid is not a valid C format string. Reason: The string ends in the middle of a directive.

CC: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

misc: fix some printf format strings

Fix compiler warnings seen on Linux/i586 and OSX/travis.

  #type            #format   #cast
   unsigned long    %lu       -
   uint64_t         PRIu64    -
   fdisk_sector_t   %ju       (uintmax_t)

Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

libmount: fix file descriptor leak

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

fincore: fix file descriptor leak

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

libsmartcols: fix relative width interpretation

The non-defined width hint (=0) should not be interpreted as relative width.

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

libsmartcols: column width reduction refactoring

The current implementation is unreadable... Let's write it again with
more obvious semantic.

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

libsmartcols: add scols_line_move_cells()

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

libsmartcols: add scols_table_move_column()

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

libfdisk: (gpt) fix ents zeroize

The gpt->ents is "unsigned char" buffer now. We need to use bytes, no
number of entries...

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

libfdisk: (gpt) update nparts_max

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

fincore: use size_t everywhere

It seems better to avoid "int" at all from sizes calculations to make
the code more robust for future changes.

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

tests: fincore/count, skip unsupported FS

Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

tests: handle dd errors in fincore/count

On dd error the test will fail earlier and the test diff will be
more useful.

Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

fincore: fix multiplication overflow

This failed on ppc64 with page size 64k.

Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

tests: fix fincore/count for PAGE_SIZE != 4096

Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

tests: remove fincore SMALLER_THAN_PAGESIZE tests

These tests failed for example on ext4 with a non-default
blocksize 1024. In this case it seems that the kernel only
caches a full page if we read at least PAGE_SIZE -BLOCK_SIZE +1.

Both tests also failed on NFS allthough block size seems to be
1M there.

We still keep the test for PAGE_SIZE-1 which seems to work more
reliable.

Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

libfdisk: (gpt) care about SSIZE_MAX for read(2)

read(2) behavior is undefined if you want to read more  than SSIZE_MAX
bytes. Let's be paranoid and check for this...

Reported-by: Ruediger Meier <sweet_f_a@gmx.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

script: fix start message showing in output file when -q is on

Signed-off-by: Rui Zhao (renyuneyun) <renyuneyun@gmail.com>

libfdisk: (gpt) make entries array size calculation more robust

* use the same function everywhere
* keep calculation based on size_t

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

libfdisk: (gpt) use size_t and gpt_get_nentries()

If possible use size_t for number of entries (partitions). It
makes code more readable and without unnecessary casts.

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

libfdisk: (gpt) don't access entries array directly

We blindly assume that our sizeof(struct gpt_entry) is the same
on-disk stored header->sizeof_partition_entry.

It seems more correct would be to use sizeof specified by header to
access the entries array items. This patch introduces gpt_get_entry()
to avoid direct access to gpt->ents[].

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

agetty: fix login-options desc in man page

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

docs: update Documentation/deprecated.txt

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

Merge branch 'hwclock-date7-3' of github.com:jwpi/util-linux

* 'hwclock-date7-3' of github.com:jwpi/util-linux:
  hwclock: remove unused 'silent' arg
  hwclock: remove dead cmos code
  hwclock: improve cmos message strings
  hwclock: remove unused atomic arg in cmos
  hwclock: remove unused cmos ctl structs
  hwclock: remove alpha cmos

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

* 'tailf-removal' of https://github.com/kerolasa/lelux-utiliteetit:
  tailf: remove deprecated utility

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

* '2017wk13' of https://github.com/kerolasa/lelux-utiliteetit:
  docs: improve agetty.8 manual page
  agetty: make --remote to forward --nohostname as -H to login
  lib/cpuset: stop changing variable that is not read
  agetty: remove variable that is set but not read

Merge branch 'fix-ipcs-shmall-overflow' of https://github.com/rudimeier/util-linux

* 'fix-ipcs-shmall-overflow' of https://github.com/rudimeier/util-linux:
  ipcs: make shmall overflow a bit less worse

losetup.8: Reorder options documentation

Reorder options to match onboard help and group functionally.
Include some missing optional arguments.

[kzak@redhat.com: - don't include available columns in man page,
                  - add comments to the usage()
  - fix order in usage()
  - fix typo in usage()]

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

zramctl: be more specific about default output

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

docs: improve agetty.8 manual page

Reviewed-by: J William Piggott <elseifthen@gmx.com>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

agetty: make --remote to forward --nohostname as -H to login

Without this change an attempt to remove hostname printing required following
rather clumsy agetty invocation.

  /sbin/agetty --nohostname --login-options '/bin/login -H -- \u'

After the change --nohostname behaves similar way with --host option, that
is when combined with --remote the effect is passed to login(1).

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

lib/strutils: fix typos in huge size suffixes

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

ipcs: make shmall overflow a bit less worse

Still no large integer support but on overflow we print now
the largest possible value, maybe even the largest one which
makes sense at all.

So on x86_64 systems we'll see now:
$ echo "4503599627370496" > /proc/sys/kernel/shmall
$ ipcs -m -l | grep "max total"
max total shared memory (kbytes) = 18014398509481980

rather than this:
$ ipcs -m -l | grep "max total"
max total shared memory (kbytes) = 0

Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

tailf: remove deprecated utility

March 2017 is gone, it is time to remove this utility as scheduled in
earlier commit, and promised in manual page.

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

lib/cpuset: stop changing variable that is not read

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

agetty: remove variable that is set but not read

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

hwclock: remove unused 'silent' arg

The 'silent' argument for get_epoch_rtc() was used
to silence error messages when querying the rtc
driver for an alpha epoch while using cmos direct
access. Alpha cmos has since been removed so
'silent' is no longer used.

* sys-utils/hwclock.h: remove 'silent' argument
* sys-utils/hwclock.c: same
* sys-utils/hwclock-rtc.c: same

Signed-off-by: J William Piggott <elseifthen@gmx.com>

hwclock: remove dead cmos code

* hwclock-cmos.c: remove unused 'century' code.

Signed-off-by: J William Piggott <elseifthen@gmx.com>

hwclock: improve cmos message strings

* sys-utils/hwclock-cmos.c: improve message strings

Signed-off-by: J William Piggott <elseifthen@gmx.com>

hwclock: remove unused atomic arg in cmos

* sys-utils/hwclock-cmos.c: remove unused arg for atomic()

Signed-off-by: J William Piggott <elseifthen@gmx.com>

hwclock: remove unused cmos ctl structs

The hwclock-cmos.c ctl structs were only used for
the since removed alpha code.

* sys-utils/hwclock-cmos.c: remove unused ctl structs

Signed-off-by: J William Piggott <elseifthen@gmx.com>

hwclock: remove alpha cmos

Remove alpha direct I/O access, use RTC instead:
http://marc.info/?l=util-linux-ng&m=141682406902804

Resolves the alpha 2020 issue for util-linux:
http://marc.info/?l=util-linux-ng&m=148387021519787

Now it is only the kernel's RTC problem.

* sys-utils/hwclock.c: remove alpha cmos
* sys-utils/hwclock-cmos.c: same
* sys-utils/hwclock.h: same
* sys-utils/hwclock.8.in: same

Signed-off-by: J William Piggott <elseifthen@gmx.com>

tests: add --mountpoint to findmnt calls

... if necessary to make things more robust.

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

utmpdump: remove unnecessary variable

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

hwclock: remove 1994 Award BIOS workaround

Remove the 1994 Award BIOS bug workaround as
previously discussed more than two years ago:
http://marc.info/?l=util-linux-ng&m=141682406902804&w=2

* sys-utils/hwclock.c: remove badyear option
* sys-utils/hwclock.h: same
* sys-utils/hwclock.8.in: same

Signed-off-by: J William Piggott <elseifthen@gmx.com>

test: fix mountpoint device check in libmount/loop

This test could fail if the build directory is somehow exoticly
mounted (shared bind mounts, symlinks).

CC: Stanislav Brabec <sbrabec@suse.cz>
Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

lscpu: fix mem leak [coverity scan]

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

lscpu: check return code [coverity scan]

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

zramctl: make mm_stat parser more robust

Let's fallback to attribute files if mm_stat file is incomplete. It
should not happen, but I have seen RHEL7 kernel where is no
num_migrated/pages_compacted attribute...

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

agetty: fix typo

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

tests: define columns for fincore test

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

agetty: fix characters reorder in login prompt

The current agetty uses TIOCSTI ioctl to return already read chars
from login name back to the terminal (without read() before
tcsetattr() we will lost data already written by user). The ioctl
based solution is fragile due to race -- we can return chars when
terminal already contains another new chars. The result is reordered
chars in login name.

The solution is to use extra buffer for already read data.

Reported-by: Michael Tretter <m.tretter@pengutronix.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

unshare: add note about persisten PID namespaces

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

fincore: add column RES

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

tests: remove status=none dd(1) from fincore

seems dd(1) sucks on travis more than expected:

 dd: invalid status flag: `none'
 Try `dd --help' for more information.

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

tests: redirect unwanted dd(1) advices to /dev/null

It seems that dd(1) on travis writes unwanted messages to stdout

  dd: you probably want conv=notrunc with oflag=append

and we do not want conv=

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

blkid: fix compiler warning [-Wdiscarded-qualifiers]

Let's also hide the show[] array in has_item().

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

Merge branch '2017wk11' of git://github.com/kerolasa/lelux-utiliteetit

* '2017wk11' of git://github.com/kerolasa/lelux-utiliteetit:
  blkid: add control struct
  blkid: simplify version option handling
  tests: add static keyword where needed [smatch scan]
  tests: do not use plain 0 as NULL [smatch scan]
  libsmartcols: fix test variable shadowing

bash-completion: update fincore to have latest options

Also include --output smartcols fields.

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

Merge branch 'hwclock-date7-1' of github.com:jwpi/util-linux

* 'hwclock-date7-1' of github.com:jwpi/util-linux:
  hwclock-rtc.c: try the 'new' rtc class first

libblkid: fix typo

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

hwclock-rtc.c: try the 'new' rtc class first

* sys-utils/hwclock-rtc.c: try to open the 'new' rtc class driver first.
* sys-utils/hwclock.8.in: document this.

Signed-off-by: J William Piggott <elseifthen@gmx.com>

build-sys: add missing header file

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

libblkid: prevent OOB access while probing HFS+

It is possible to perform out of boundary read accesses due to
insufficient boundary checks in probe_hfsplus.

The first issue occurs if the leaf count in a B-node is too
small. The second happens while parsing a unicode description which
is longer than 255 UTF-8 characters. The length is stored in a 16 bit
integer, but the array in the struct is limited to 255 * 2, which is
in sync with Apple's Open Source HFS+ implementation (HFSUniStr255).

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

tests: refresh fincore test

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

fincore: fix error on open()

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

fincore: add --raw and --json

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

fincore: add --output

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

fincore: add --bytes and --noheadings

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

fincore: use libsmartcols

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

fincore: cleanup return codes

Don't use EXIT_* macros as functions return code.

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

bash-completion: add a function for fincore command

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

tests: add cases for testing fincore command

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

man: add a page for fincore command

Signed-off-by: Masatake YAMATO <yamato@redhat.com>

fincore: new command for counting pages of file contents in core

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

Merge remote-tracking branch 'jwpi/hwclock-date7'

* jwpi/hwclock-date7:
  lib: add parse-date documentation
  hwclock: use parse_date function
  build-sys: add parse-date.y
  lib: add parse-date.y

Fix menu spelling in cfdisk debug messages

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

blkid: add control struct

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

blkid: simplify version option handling

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

tests: add static keyword where needed [smatch scan]

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

tests: do not use plain 0 as NULL [smatch scan]

Likewise commit 87918040658f2fa9b1bf78f1f8f4f5c065a2e3a3.

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

libsmartcols: fix test variable shadowing

libsmartcols/samples/fromfile.c:57:16: warning: declaration of 'flags'
shadows a global declaration [-Wshadow]
libsmartcols/samples/fromfile.c:29:33: note: shadowed declaration is here
libsmartcols/samples/fromfile.c:101:8: warning: declaration of 'flags'
shadows a global declaration [-Wshadow]

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

libfdisk: fix compiler warning [-Wsign-compare]

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

tests: refresh lscpu tests

.. due to libsmartcols use for the default output. The output does not
contain extra unnecessary blank space anymore.

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