tools: add err_oom() to smatch no_return_funcs

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

lib/blkdev: use 64-bit to count size in bytes

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

lib/timeutils: fix indention [smatch scan]

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

tailf: fix open() return value check [coverity scan]

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

setterm: remove dead code [coverity scan]

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

agetty: cleanup plymouth usage [coverity scan]

* use macros for paths
* check open() return value

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

rtcwake: improve open() usage [coverity scan]

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

sfdisk: clarification for sfdisk man page

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

lsipc: cleanup usernames

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

lsipc: check scols_line_refer_data() return code

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

lsipc: don't duplicate already allocated data

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

build-sys: add --disable-assert

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

libmount: cleanup assert() usage

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

libfdisk: cleanup assert() usage

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

libblkid: remove dead code [coverity scan]

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

mkfs.minix: add hint for scanners [coverity scan]

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

mkfs.minix: use xalloc

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

mkfs.cramfs: remove dead code [coverity scan]

There two possible ways, print error and exit on too long names or
truncate the filename -- but it's impossible to do both in the same
code :-) It seems that code already assumes warning on long names, so
let's remove errx() and keep the behavior in dependence on -E.

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

fsck.minix: be more paranoid with block buffers [coverity scan]

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

cfdisk: check return value [coverity scan]

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

lsipc: ifdef SHM_* macros

Reported-by: Andreas Henriksson <andreas@fatal.se>
Signed-off-by: Karel Zak <kzak@redhat.com>

tests: fix minix tests filenames

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

tests: try mkfs & mount all supported minix versions

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

bash-completion: update few options changed since v2.26

Attempt to find and update all changes to command line options that has
not been reflected in bash-completion files in between versions v2.26 to
v2.27-rc1.

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

tests: fix minix test

- run the test with UID=0 only. The minix FS inode contains UID and
  GID, so the final image checksum depends on user...

- always use subtest name for the images

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

tests: add fsck.minix checks

The debian bug was fixed in commit 730ae9c89971, and the rest are about
having test coverity for various file name lenghts and minix fs versions.

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

tests: add way to control mkfs.minix time stamps

Needed in order to create reproducable file systems image files, so that
out come of mkfs.minix can be checksum'ed.

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

mkfs.minix: re-fix block count maths

Error, that Joshua Hudson already pointed out, creapped back to commit
da41ff5 when changes were applied from mailbox rather than git remote, as
the corrected change had included some accidental rubbish.

Reference: http://www.spinics.net/lists/util-linux-ng/msg11764.html
Reference: http://www.spinics.net/lists/util-linux-ng/msg11848.html
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

unshare: remove angular brackets from literal argument in usage text

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

docs: fix some grammar and punctuation in the 2.27 release notes

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

mount: remove duplications of --help and --version from usage text

And slice up the affected portions.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

lslocks: slice up the recently modified usage text

For ease of translation when it changes.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

textual: adjust grammar and punctuation of some messages

Also equalize three messages to one other one,
and fix a typo in USE_COLORS_BY_DEFAULT.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

build-sys: mark script(1) as Linux only due to signalfd()

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

build-sys: release++ (v2.27-rc1)

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

docs: add v2.27-ReleaseNotes

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

docs: update AUTHORS file

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

po: merge changes

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

lsblk: Display mountpoint even for top-level device

If a filesystem is mounted on top-level block device
with existing partitions, the mountpoint is not displayed
in the lsblk output.

This situation can happen by a configuration mistake
and lsblk could be used to detect such a mistake.

This patch allows searching for a mountpoint for all displayed
devices, not only for leaf nodes.
(It should be pretty cheap operation, mtab is parsed only once.)

For example: lsblk /dev/loop1
NAME      MAJ:MIN RM  SIZE RO TYPE MOUNTPOINT
loop1       7:1    0  128M  0 loop /mnt/tst
└─loop1p1 259:0    0  127M  0 loop

Signed-off-by: Milan Broz <gmazyland@gmail.com>

sfdisk: don't print extra linebreaks on --quiet

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

libfdisk: allow to change partition type for extended partitions

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

hexdump: fix man page (-x vs. default output)

Reported-by: Reuti <reuti@staff.uni-marburg.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

docs: make fsck.minix(8) more pretty

Use real table to device list, add missing new paragraph markers, remove
braces that cause reader to wonder if test is meant or not, squeeze error
code list, move references to bottom of the manual, and use mail or web
address macro when possible.

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

fsck.minix: introduce long options to the command

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

fsck.minix: add minix v3 support

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

fsck.minix: rename device file descriptor variable

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

mkfs.minix: refactor root block content creation

This does not belong to main() function.

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

mkfs.minix: check requested blocks will not exceed available on device

Earlier user could define more blocks than device, or backing file for
loopback file system, had available.  That lead to a system crash with
following commands;

fallocate --length 64KiB test-file
mkfs.minix -3 -i 842160 test-file 104882174
mkdir test-file.d
mount test-file test-file.d
cp /etc/service test-file.d
Killed
sudo umount test-file.d

The minix driver should probably not hang the whole kernel, but the least
that mkfs.minix ought to do is not to let users to get that condition
quite as easily.

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

mkfs.minix: check user input carefully

File name lenght and version input needs to be checked against each
other, which will determine what version of file system is in question.

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

mkfs.minix: add fs_control structure, and remove most global variables

This allows better code structure in future.

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

mkfs.minix: remove unuseful code

Checks about inodes vs block sizes does not add much robustness.  Both
values are derived at compilation time from struct minix_inode size, and
they form full definition circle.

Bad block check for none-block devices should not be supressed, user
requested it so let him have it.

Check for st_rdev == 0x0300 || st_rdev == 0x0340 was unreachable.

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

mkfs.minix: use is_mounted() from libcommon

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

mkfs.minix: introduce long options to the command

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

mkfs.minix: increase maximum minix v2 and v3 file system sizes

mkfs.minix misbehaves when attempting to create a large v2 or v3
filesystem.  I finally traced it down to attempting to create too many
inodes so that the first zone is past 65535 blocks in.  This obviously
doesn't work as the on-disk superblock says this is a 16 bit integer.

I wrote a patch that catches this, clamps to the absolute v2/v3 limit
(like it already does for v1), and sets the blocks per inode to a more
reasonable ratio when exceeding half a gigabyte.  Having a half-gig
filesystem with most files being smaller than 3k isn't really reasonable.

I suppose if you don't want to adjust inode sizes automatically you could
take that part out, and it will just crab sooner.

Given the non-attention in the code, I suspect nobody ever had cause to
try such a big minix filesystem.  Well I have my reasons involving some
deeply embedded work where ext2 would place too much strain on the
hardware.

Reviewed-by: Sami Kerola <kerolasa@iki.fi>
Signed-off-by: Joshua Hudson <joshudson@gmail.com>

libmount: make mnt_get_filesystems() more robust [clang analyze]

The mnt_free_filesystems() relies on NULL as the last item in the
filesystems array. It's necessary to keep NULL there after failed
strdup() too, because we call mnt_free_filesystems() to deallocate
after error.

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

lsipc: cleanup {user,group}name usage

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

mcookie: fix -f usage [clang analyze]

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

libsmartcols: make child-parent reference more robust [clang analyze]

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

hwclock: make static analyzes more happy

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

libfdisk: result of operation is garbage or undefined [clang analyze]

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

libfdisk: undefined or garbage value returned to caller [clang analyze]

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

fdisk: value is never read [clang analyze]

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

fdisk: value is never read [clang analyze]

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

lsipc: value is never read [clang analyze]

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

fdisk: value is never read [clang analyze]

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

po: update fr.po (from translationproject.org)

po: update da.po (from translationproject.org)

unshare: remove duplicate [make checkincludes]

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

lslogins: remove duplicate [make checkincludes]

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

libmount: sort pseudo filesystems by LC_COLLATE=C

Addresses: https://github.com/karelzak/util-linux/issues/212
Signed-off-by: Karel Zak <kzak@redhat.com>

tasts: add logger octet counting test

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

logger: Add support to logger for RFC6587 octet counting

This patch adds support to logger for RFC6587 octet counting.
RFC6587 provides support for two sorts of framing:

1. Octet counting (at RFC6587 s3.4.1)

   In essence each frame is preceded by a decimal length and a
   space.

2. Non-transparent framing (at RFC6587 s3.4.2), also called
   'octet stuffing'

   In essence each frame is terminated by a `\n`

Prior to this patch, logger used option 2 (non-transparent framing)
on TCP, and used no framing on UDP. After this patch, the default
behaviour is unchanged, but if the '--octet-count' option is supplied,
option 1 is used for both TCP and UDP. Arguably octet count framing
makes little sense on UDP, but some servers provide it and this
allows testing of those servers.

Signed-off-by: Alex Bligh <alex@alex.org.uk>

sfdisk: add --list-free

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

fdisk: add 'F' command to list free unpartitioned space

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

Merge branch 'test-fixes' of https://github.com/rudimeier/util-linux

Revert "tests: start script/race test only when --force specified"

This reverts commit 8ba3f35e07f736a0165669ac787b016b4311eb29.

This test is not too slow anymore and BTW since last script refactoring
it does work now even on slow/heavy-loaded systems.

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

script: improve SIG{TERM,QUIT,KILL} sensitivity

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

cfdisk: don't print obsolete CHS addresses

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

cfdisk: make sure that output fits to terminal width

Reported-by: Benno Schulenberg <bensberg@justemail.net>
Signed-off-by: Karel Zak <kzak@redhat.com>

libsmartcols: reduce relative columns more aggressively

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

fdisk: init libsmartcols debug

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

cfdisk: remove extra line between menu and info line

Reported-by: Benno Schulenberg <bensberg@justemail.net>
Signed-off-by: Karel Zak <kzak@redhat.com>

cfdisk: spell "label" in lower case

To be less shouty.  It is not an abbreviation like "UUID".

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

cfdisk: allow an uppercase X to toggle the extra info

To make true the help text line that says that all commands
can be entered with either upper or lower case.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

build-sys: add --disable-cal

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

build-sys: fix test_colors build

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

tests: update build-sys tests

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

libsmartcols: don't link with tinfo

Let's move color names to sequence translation to separate file to
make it usable without all the stuff in lib/colors.c.

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

tests: add udevadm settle

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

libfdisk: add comment to the header file

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

lsipc: make --id usable with all output formats and with -o

This patch allows to use things like:

 # lsipc -m --id 47611910 -o COMMAND,SIZE,KEY --json

or

 # lsipc -m --id 47611910 -o SIZE --bytes --noheadings --raw

to get just one value for the resource.

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

lsipc: the default --global is mutually exclusive with -c --id and -t

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

lsipc: remove --print0 and -colon-separate

These options have been originally designed for lslogins for
compatibility another passwd-like tools. It does not make sense for
IPC.

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

lsipc: remove duplicate "Usage:"

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

cfdisk: make sure that extra info is refreshed/removed

For example when you move from a Partition to the FreeSapce then we
need to remove old extra info and draw nothing.

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

cfdisk: properly toggle extra info window, make it more robust

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

lsipc: tiny change about --global to the man page

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

ipcmk: support <size>{K,M,G,...}

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

lsipc: default to --global

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

lsipc: add USE% to --global

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