Revert "blockdev: Remove the --setbsz (set blocksize) option which has never worked."

This reverts commit b1555acc2f709ac4f3b1e6c686a11cadb7b04f72.

It seems that the option is used by kernel guys to test kernel, so
let's keep the option in the blockdev(8) although it's almost useless
in userspace. All we need is to improve docs to make things more
obvious to end users.

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

libmount: fix mnt_context_is_child() comment

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

libmount: Fix typo in comment

s/MNT_OMASK_/MNT_OMODE_/

Signed-off-by: Namhyung Kim <namhyung@gmail.com>

cal: improve the help text and man page

* misc-utils/cal.c: Add a little doc string, and mention that the
  current month is implicit if not specified.  Also remove mention
  of "current" from two option descriptions as a specific date may
  have been specified.
* misc-utils/cal.1: Likewise.

[kzak@redhat.com: - use fputs and USAGE_SEPARATOR]

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>
Signed-off-by: Pádraig Brady <P@draigBrady.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: add -t <type> to enable only specified type

In some cases (for example hybrid GPT) it's useful to force fdisk to
read only specified disk label and ignore everything else.

For example:

# fdisk -l /dev/sda

Disk /dev/sda: 149.1 GiB, 160041885696 bytes, 312581808 sectors
Units: sectors of 1 * 512 = 512 bytes
Sector size (logical/physical): 512 bytes / 512 bytes
I/O size (minimum/optimal): 512 bytes / 512 bytes
Disklabel type: gpt
Disk identifier: 3549290F-417C-4941-8503-F7835109B821

Device           Start          End   Size Type
/dev/sda1         2048      2050047  1000M EFI System
/dev/sda2      2050048      6146047     2G Microsoft basic data
/dev/sda3      6146048     26462207   9.7G Linux swap
/dev/sda4     26462208     98142207  34.2G Microsoft basic data
/dev/sda5     98142208    230662143  63.2G Microsoft basic data
/dev/sda6    230662144    312580095  39.1G Microsoft basic data

but when GPT is disabled we can access PMBR:

# fdisk -l -t dos /dev/sda

Disk /dev/sda: 149.1 GiB, 160041885696 bytes, 312581808 sectors
Units: sectors of 1 * 512 = 512 bytes
Sector size (logical/physical): 512 bytes / 512 bytes
I/O size (minimum/optimal): 512 bytes / 512 bytes
Disklabel type: dos
Disk identifier: 0x00000000

Device    Boot Start       End    Blocks  Id System
/dev/sda1          1 312581807 156290903+ ee GPT

Reported-by: Craig <util-linux_contact@magister.com.au>
Signed-off-by: Karel Zak <kzak@redhat.com>

libfdisk: add API to disable specified label

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

cytune: fix glush typo in cytune

Signed-off-by: Rafael Ferreira <rafael.f.f1@gmail.com>

losetup: cut the remaining little block of help text into slices too

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

libfdisk: (gpt) use consistent final period in status messages

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

libfdisk: (dos) improve the grammar of an error message

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

libfdisk: (dos) correct the grammar of the disk-identifier prompt

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

textual: standardize some "cannot read" and "seek failed" error messages

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

tests: use 'btrfs' rather than 'btrfsctl'

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

tests: update fdisk outputs

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

blockdev: Remove the --setbsz (set blocksize) option which has never worked.

This option has never worked.  If you try setting the block size on a
block device, and then read it using --getbsz, you will see that the
block size never changes.

The reason for this is because the block size is specific to the
current file descriptor opening the block device, so the change of
block size only persists for as long as blockdev has the device open,
and is lost once blockdev exits.

Also the block size is not really used anywhere.  Filesystems, for
example, have their own idea of block size and ignore this setting
completely.

(Thanks Masayoshi Mizuma for diagnosing the problem)

Signed-off-by: Richard W.M. Jones <rjones@redhat.com>

various: fix mixing declarations and code compiler warnings [smatch]

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

docs: improve grammar and wording of the release-schedule text

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

COPYING: fix grammar of referring phrase, and indicate location better

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

textual: make two messages somewhat easier to understand

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

textual: tag a translator comment and put it in proper place

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

textual: use consistent wording and punctuation in fdisk stuff

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

textual: change spelling of "disk label" to the more abundant "disklabel"

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

textual: use the standard angular brackets for non-literal arguments

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

libfdisk: (gpt) fix warning

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

libfdisk/gpt: partly pluralize one message

And don't mention the largest when there is only one segment.

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

wdctl: pluralize one thrice-repeated message

Also put in one hard space, to keep text and number separated when
the translated string should be too long.

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

wdctl: gettextize and pluralize one forgotten message

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

partx: correct three messages

Properly describing the partition type, using a verb instead of a noun,
putting the blank line after the command options, and using the correct
argument for --output.

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

partx: pluralize one message

Reported-by: Petr Písař <petr.pisar@atlas.cz>
Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

libfdisk/sgi: tweak and harmonize some message strings

Using consistent capitals and lowercase, adding missing periods and
a missing space, repositioning commas for clarity, improving grammar,
and aligning substrings for esthetics.

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

libfdisk/sgi: tweak and harmonize some comments

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

libfdisk/sgi: properly pluralize three messages

Reported-by: Petr Písař <petr.pisar@atlas.cz>
Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

mkfs.minix: properly pluralize four messages

Reported-by: Petr Písař <petr.pisar@atlas.cz>
Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

wipefs: pluralize one message

Reported-by: Petr Písař <petr.pisar@atlas.cz>
Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

su-common: pluralize one message

Reported-by: Petr Písař <petr.pisar@atlas.cz>
Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

mcookie: pluralize two messages

Reported-by: Petr Písař <petr.pisar@atlas.cz>
Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

textual: properly use uppercase in certain abbreviations

Reported-by: Petr Písař <petr.pisar@atlas.cz>
Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

hexsyntax: in help text, wrap argument of option -e in angular brackets

Reported-by: Petr Písař <petr.pisar@atlas.cz>
Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

textual: end both the singular and the plural message with a period

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

libmount: (docs) make it obvious how we call mount.<type> helpers

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

build-sys: use tinfo *or* ncurses for more(1), ul(1) and setterm(1)

 * it seems that we don't have to link the utils with ncurses, tinfo is enough.
   This change saves one unnecessary dependence.

 * libtinfo is also distributed with pkg-config files, so we can use
   PKG_CHECK_MODULES() as a primary source for LIBS and CFLAGS.

 * add TINFO_CFLAGS (although it's probably always empty)

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

libfdisk: cleanup warning messages

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

docs: tweak the grammar of an agetty man-page fragment

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

libfdisk: improve spelling of two comments

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

agetty: use standard angular brackets in usage message

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

build-sys: Fix check for __GNU_LIBRARY__

If we are not on glibc, __GNU_LIBRARY__ will not exist causing the check
to always fail and try to use syscalls directly.

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

build-sys: Check for type sighandler_t and use if present

__sighandler_t is libc implementation specific and should not be relied
upon. Instead, we fall back upon void (*)(int), as specified by POSIX.

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

include: Add missing includes

sys/types.h: For u_char typedef
sys/params.h: For MAXNAMLEN
sys/ttydefaults.h: For various tty definitions (also add configure check)

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

build-sys: Use _POSIX_VERSION to determine support for %m

%m is included in POSIX 2008, so we can check if the libc implements
that before failing.

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

testsL fix py<libname> check

Reported-by: Bruce Dubbs <bruce.dubbs@gmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

su: fix caught_signal logic and initialize oldact [coverity scan]

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

fdisk: fix possible memleak [coverity scan]

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

lscpu: cleanup DMI code [coverity scan]

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

docs: update TODO

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

lscpu: check strchr() result [coverity scan]

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

lscpu: don't pass null pointer to string functions [coverity scan]

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

libmount: remove dead code [coverity scan]

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

libblkid: use proper type in xfs log checker [coverity scan]

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

pylibmount: check for mnt_fs_get_tag() return value [coverity scan]

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

pylibmount: make tab parsers more robust [coverity scan]

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

utmpdump: make analyzers happy

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

last: check fstat() return [coverity scan]

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

libmount: make mnt_table_replace_file() more robust [coverity scan]

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

fsck: don't print warning if there is no /etc/fstab at all

Reported-by: Thomas Bächler <thomas@archlinux.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

losetup: don't require 512-byte aligned offsets

Kernel aligns the device size, but the offset where the device starts
is not required to be aligned.

 # losetup --offset 32 -f file.img

is just fine, the final size of the look device will be (in sectors)

  (backing_file_size - offset) >> 9

so we have to do the same in userspace when we check for successful
set capacity ioctl.

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

Merge branch 'master' of https://github.com/yurchor/util-linux

bash-completion: use '\n' as IFS when ask for filenames

The bash completion for more(1) treats the space-separated pieces of
filenames as different files.

$ touch foo\ bar
$ more foo<TAB>
bar foo

Reported-by: Ángel González <ingenit@zoho.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

textual: fix misspelled words in -rc1

Reported-by: Rafael Ferreira <rafael.f.f1@gmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: Save errno if mkostemp() failed

After mkostemp() failed, umask() and free() might alter the errno
to another value.  Not sure those calls really changes the errno
or not.  But let's be more conservative.

Signed-off-by: Namhyung Kim <namhyung@gmail.com>

build-sys: simplify python detection

This allows './configure --enable-most-builds' and 'make distcheck' to
work when both python2 and python3 are installed, and user has set python
version preference using symlink and PATH order.

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

clean up term lib handling

The ncurses package has been providing pkg-config files for a while now.
So let's start using them to get the proper linker & compiler flags.  It
can make a difference when ncurses is configured in a way that requires
extra link time flags but util-linux doesn't provide them, or when the
headers live in a weird place and util-linux can't find them.

Since the NCURSES_LIBS is always defined for the Makefile, there's no need
to gate on the HAVE_NCURSES conditional.  When it's disabled, the var will
simply be empty.

With a minor tweak to how tinfo is handled, we can do the same thing -- we
just always use TINFO_LIBS in the Makefile's.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>

setterm: fix term.h/ncurses.h include ordering

The ncurses term.h header has logic in it to detect if nucrses.h has
already been included and void defining things when it has.  But since
setterm includes term.h and the ncurses.h, it doesn't work and we can
get fun build-time warnings like:

  CC       term-utils/setterm-setterm.o
In file included from term-utils/setterm.c:109:0:
/usr/include/ncursesw/ncurses.h:827:12: warning: redundant redeclaration of 'tigetflag' [-Wredundant-decls]
 extern NCURSES_EXPORT(int) tigetflag (NCURSES_CONST char *);  /* implemented */
            ^
In file included from term-utils/setterm.c:106:0:
/usr/include/ncursesw/term.h:775:12: note: previous declaration of 'tigetflag' was here
 extern NCURSES_EXPORT(int) tigetflag (NCURSES_CONST char *);
            ^

Signed-off-by: Mike Frysinger <vapier@gentoo.org>

findmnt: always allocate memory for cell value

Since b3386c83fe77, findmnt's output uses TT_FL_FREEDATA, which causes
a crash here when string literal is returned instead of a heap address.

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

nologin: add new command

Currently it's maintained as distro specific (or people use impolite
/bin/false way).

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

Fix various typos

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

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

docs: update v2.24-ReleaseNotes

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

build-sys: one install dir for all pylibmount stuff

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

build-sys: fix lastb symlink

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

po: merge changes

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

tests: update build tests

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

build-sys: fix static builds

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

libblkid: update docs

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

libmount: update docs

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

umount: provide more helpful error message

non-root user on device that is not mounted:

 $ umount /mnt/flash
 umount: /mnt/flash: umount failed: Operation not permitted

this is true, because we cannot evaluate permissions as there is no
entry in mtab, but users expect something more helpful -- for example

 umount: /mnt/flash: not mounted

References: https://bugs.archlinux.org/task/31647
Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: small cleanup in umount code

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

docs: add v2.24-ReleaseNotes

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

mount: add notes about UUIDs to the man page

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

libmount: compose mount.<type> helpers paths more carefully

The [u]mount helper path is "/sbin/mount.<type>", the <type> should not
be possible to interpret like a path, for example:

  # mkdir /sbin/mount.fake
  # mount -t fake/../../../home/user/foo.sh

Anyway, this is *not mount(8) security issue*, because:

  - non-root users don't have permissions to specify -t on mount(8)
    command line
  - /sbin is not writable for regular users

This patch makes libmount more robust for situations when the library
is used by incompetent developers who don't sanitize user's input.

Reported-by: "Horsfall, Matthew" <mhorsfal@akamai.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: add pylibmount __init__.py EXTRA_DIST

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

build-sys: fix ${prefix} usage, be more verbose about py paths

 - incorrect ${prefix} and ${exec_prefix} usage in AC_CASE
   (bug introduced by commit 9f57e6e8)

 - disable python by default (you have to use --with-python)

 - add python paths to the finale ./configure info message

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

build-sys: install to the correct python dirs

It seems we have to call AM_PATH_PYTHON() monster to get
pyexec (shared libs) and python (scripts) directories.

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>

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

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

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

po: update util-linux.pot (from translationproject.org)

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

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