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)

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

last: fix man page groff warning

tbl:<standard input>:166: column separation specified for last column

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

mount: fix repeating words in the man page

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

fdisk: use xalloc stuff

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

libfdisk: (bds) cleanup includes

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

build-sys: fix include Makemodule.am

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

tests: improve cal --color test

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

libblkid: Get rid of an unnecessary check

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

libmount: make mnt_fs_{ap,pre}pend_options() more robust

We should not use the results from mnt_split_optstr() if the function
failed.

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

libmount: Remove stale comment on mnt_context_mount()

The commit f9906424 ("libmount: add post-mount checks to detect ro/rw")
added necessary check so the comment is not valid anymore.

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

libmount: Set each optstr's to NULL if failed

When mnt_split_optstr() failed in the middle, vfs, fs, user optstr's
are freed but not reset.  It can lead to double frees at the end of
mnt_fs_{ap,pre}pend_options().

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

libmount: Free splitted optstr's when error occurred

When strdup() failed, u, v and f optstr's should be freed.

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

libmount: Get rid of an unnecessary check

libmount_debug_mask was OR'ed to MNT_DEBUG_INIT so it should be non-null.
Thus the check is pointless.

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

tests: update odd output fdisk test

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

ORIG_HEAD
tests: update sun label test

 * add fdisk -l outputs (to make the test more usable)
 * use more usable geometry
 * don't expect that 'create sun label' oprations asks for C/H/S (we
   follows fdisk_context defaults)

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

fdisk: print geometry for dos-compatible mode too

libfdisk: (sun) make it more obvious why the first sector moved

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

libfdisk: improve fdisk_save_user_geometry()

The functions should not overwrite old setting if the new value (C/H/S) is
zero.

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

libfdisk: use context geometry in SGI and SUN

 * don't call HDIO_GETGEO ioctls in the drivers
 * use context geometry (so user can overwrite it by -C/H/S options)
 * use default geometry is ioctl does not return anything

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

tests: remove remaining set -x

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

tests: update lscpu tests

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

libfdisk: fix possible floating point exception

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

libfdisk: gpt: loosen check fot pmbr size in lba

While most disk partitioning tools out there create a pMBR's size in lba
to be the lesser of the whole disk or 2Tib, Microsoft apparently does not[1].
It always sets the entry to the maximum 32-bit limitation - even though a
drive may be smaller than that.

Loosen this check and only verify that the size is either the whole disk
or 0xFFFFFFFF.  No tool in its right mind would set it to any value
other than these.

[1] http://thestarman.pcministry.com/asm/mbr/GPT.htm#GPTPT

Signed-off-by: Davidlohr Bueso <davidlohr@hp.com>

libfdisk: gpt: avoid unnecessary pmbr check

We only jump to the 'check_hybrid' label when a valid pmbr
is detected, so we need not recheck again. Move the label's
logic so it doesn't include the check.

Signed-off-by: Davidlohr Bueso <davidlohr@hp.com>

lscpu: report cpu min mhz

The lscpu tool only shows the current and max CPU frequencies, however,
in many scenarios this is not enough. If there are energy saving situations
(like some CPUs being idle, or not fully used) the cpugov can lower this value.

Signed-off-by: Davidlohr Bueso <davidlohr@hp.com>

pylibmount: rewrite to be py2/py3 universal

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

build-sys: support --with-python[={2,3}]

 * we use pkg-config to get CGLAGS and LIBS, use package specific
   config (e.g. python-config) is non-sense.

 * default is to follow distribution and use pkg-config module name
   "python". This is probably symlink to python2.pc or python3.pc.

 * --with-python=2 forces to pkg-module "python2 >= 2"

 * --with-python=3 forces to pkg-module "python3 >= 3"

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

libmount: free username after check

Leak reported by valgrind:

==14226== 7 bytes in 1 blocks are definitely lost in loss record 1 of 1
==14226==    at 0x4C2757B: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==14226==    by 0x5534839: strdup (in /usr/lib/libc-2.18.so)
==14226==    by 0x4E53FE0: mnt_get_username (utils.c:560)
==14226==    by 0x4E456A5: mnt_context_prepare_umount (context_umount.c:413)
==14226==    by 0x4E464F7: mnt_context_umount (context_umount.c:851)
==14226==    by 0x403476: umount_one (umount.c:299)
==14226==    by 0x402B34: main (umount.c:629)

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

libmount: use memmove when overlap possible

When unmounting some mountpoints as an unprivileged user (via the
'user' option in fstab), the umount fails. Debug output of 'umount
/opt' reveals:

17760: libmount: CXT: [0x22890e0]: do umount
17760: libmount: UTILS: moving to /opt parent
17760: libmount: CXT: current directory moved to / [last_component='opt']
17760: libmount: CXT: [0x22890e0]: umount(2) [target='pt', flags=0x00000000]

valgrind shows the problem:

==23544== Source and destination overlap in memcpy(0x58d1370, 0x58d1371, 4)
==23544==    at 0x4C2BBC3: memcpy@@GLIBC_2.14 (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==23544==    by 0x4E537C3: mnt_chdir_to_parent (utils.c:168)
==23544==    by 0x4E45E4C: mnt_context_do_umount (context_umount.c:601)
==23544==    by 0x4E46513: mnt_context_umount (context_umount.c:855)
==23544==    by 0x403476: umount_one (umount.c:299)
==23544==    by 0x402B34: main (umount.c:629)
==23544==

ref: https://bugs.archlinux.org/task/36968
Signed-off-by: Dave Reisner <dreisner@archlinux.org>

bash-completion: fix typos in mkfs.bfs logic

There is no "bfs" command.  Change this file to match the other
mkfs files where we have a "mkfs." prefix.

Reported-by: Michał Górny <mgorny@gentoo.org>
Signed-off-by: Mike Frysinger <vapier@gentoo.org>

tests: update fdisk outputs

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

tests: update sun label test

Note that we use a little different label setting, so the checksums
are also different. For more details see commit
4170ae9cbd3ad54c9c7427212eaae6fdb030b57a.

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

tests: refresh fdisk tests

libfdisk: fix cylinders and sector buffer usage

 - sector buffer has to be correctly deallocated
 - round to cylinders when use cylinder units in dos driver

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

libfdisk: fix sectorbuffer usage in dos driver

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

fdisk: fix 'r' in BSD menu

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

libfdisk: update test_ranges() code

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

fdisk: remove unnecessary test

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

fdisk: bloody revolution in the man page

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

libfdisk: use fdisk_warnx() in fdisk_ask_partnum()

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

fdisk: separate disks by \n in -l output

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

sfdisk: fix y/n mismatch

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

fdisk: add debug message, use colors for -l

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

fdisk: fix -l

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

libfdisk: remove "gpt support is new" warning

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

fdisk: remove unused code, coding style changes

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

libfdisk: make default geom usage more robust

.. to avoid SIGFPE

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

fdisk: rewrite dump first sector add dump disklabel command

Expert command (m for help): D

PMBR: offset = 0, size = 512 bytes.
00000000  00 00 00 00 00 00 00 00  00 00 00 00 00 00 00 00
*
000001c0  01 00 ee fe ff ff 01 00  00 00 ff 9f 0f 00 00 00
000001d0  00 00 00 00 00 00 00 00  00 00 00 00 00 00 00 00
*
000001f0  00 00 00 00 00 00 00 00  00 00 00 00 00 00 55 aa

GPT Header: offset = 512, size = 512 bytes.
00000200  45 46 49 20 50 41 52 54  00 00 01 00 00 02 00 00
00000210  ae 36 81 28 00 00 00 00  01 00 00 00 00 00 00 00
00000220  ff 9f 0f 00 00 00 00 00  00 08 00 00 00 00 00 00
00000230  de 9f 0f 00 00 00 00 00  e0 9d d8 d5 d6 da 1a 44
00000240  98 57 e4 11 64 88 ce 3b  02 00 00 00 00 00 00 00
00000250  80 00 00 00 80 00 00 00  4b c7 c9 54 00 00 00 00
00000260  00 00 00 00 00 00 00 00  00 00 00 00 00 00 00 00
*

GPT Entries: offset = 1024, size = 16384 bytes.
00000400  af 3d c6 0f 83 84 72 47  8e 79 3d 69 d8 47 7d e4
00000410  bc ae 48 5e 22 e5 ca 4c  aa 98 14 6d c9 1d 72 f4
00000420  00 08 00 00 00 00 00 00  de 9f 0f 00 00 00 00 00
00000430  00 00 00 00 00 00 00 00  00 00 00 00 00 00 00 00
*

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

libfdisk: (gpt) support for disklabel dump

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

libfdisk: add fdisk_locate_disklabel()

The function returns offset and size of disklabel elements.

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

libfdisk: (gpt) use generic 'created partition' message

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

libfdisk: use fdisk_colon() and fdisk_sinfo()

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

libfdisk: extend fdisk_info() functionality

 - add per message flags
 - add fdisk_colon() for "foo bar: bla bla..." messages
 - add fdisk_sinfo() as smart version of fdisk_info()

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

fdisk: use fdisk_{info,warn} on more places

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

libfdisk: (gpt) more robust PMBR partition size check

According UEFI specification at least one (does not mean that the
first) partition is 0xFF, it makes sense to check the size of the
partition rather than check the first partition.

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

fdisk: add -L to colorize output

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