fdisk: (dos) kill 'partitions' global variable

 ... and use size_t for partition numbers

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

libfdisk: add part_get_status operation

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

libfdisk: cleanup fdisk_get_partition_type() returns

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

fdisk: (dos) move dos specific code

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

fdisk: (dos) move 'toggle active' code to fdiskdoslabe.c

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

fdisk: (dos) move dos specific code

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

fdisk: remove dead code

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

fdisk: (dos) move dos_list_table_expert() to fdiskdoslabe.c

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

fdisk: (dos) move 'move begin' code to fdiskdoslabe.c

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

fdisk: (dos) move fix order code to fdiskdoslabe.c

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

fdisk: (dos) move list table code to fdiskdoslabe.c

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

fdisk: use libfdisk label->changed

The patch is huge because it's necessary to add a pointer context to
all read_<foo> functions.

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

libfdisk: add label->changed flag

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

fdisk: (sun) remove nasty sunlabel macro

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

fdisk: (sun) remove global variable 'partitions'

 * use label counters
 * add assert()
 * use struct fdisk_label if possible

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

fdisk: (bsd) add partitions counters

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

fdisk: (aix) remove unuused variable 'partitions'

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

libfdisk: (gpt) introduce driver independent partitions counters

 label->nparts_{max,cur} to later replace global variable partitions

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

fdisk: (gpt) remove global variables, cleanup header verification

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

fdisk: differentiate between get and read operation

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

libfdisk: move label identifier to label struct

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

libfdisk: add fdisk_label to driver API

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

libfdisk: improve label deinitialization

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

libfdisk: kill dos_compatible_flag global variable

... move the variable to label private struct.

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

libfdisk: add fdisk_new_context(), reuse context

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

libfdisk: split label and label operations

 Changes:

 - fdisk_label is a private label driver struct
 - generic header of the fdisk_label points to fdisk_label_operations
 - the private fdisk_label stuff is always allocated for all drivers
   during fdisk_context initialization

 - context->labels[] contains pointers to all supported labels (drivers)
 - context->label is a pointer to the currently used label

 This change allows to:

 - store label specific global settings (e.g. dos compatible mode,
   display units, ...) independently on the current label

 - add label specific function to the API

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

libfdisk: cleanup the rest of fdisks/utils.c stuff

 - remove obsolete code
 - move fdisk_{set,get}_partition_type() to label.c (this is label
   driver operation)

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

libfdisk: add context functions

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

libfdisk: add probing function

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

libfdisk: add fdisk_reset_alignment()

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

libfdisk: add topology and geometry functions

 - rename __discovery_* to fdisk_discovery_*
 - rename fdisk_context_force_sector_size() to fdisk_override_sector_size()
 - rename fdisk_context_set_user_geometry() to fdisk_override_geometry()

 - remove non-default sector size warning

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

libfdisk: add firstsector utils

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

fdisk: remove duplicate code (already in libfdisk)

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

libfdisk: add generic label code

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

libfdisk: add alignment code

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

libfdisk: add parttype code

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

libfdisk: add basic structs

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

fdisk: use debug stuff from libfdisk

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

libfdisk: add basic library files

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

tests: add fdisk GPT test

Note that the test does not check on-disk data (by checksum)
because the current fdisk does not allow to modify automatically
generated partition and disk UUIDs.

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

tests: add fdisk BSD test

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

tests: update fdisk mbr tests

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

tests: use checksum to detect rev(1) consistency

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

tests: make utmpdump TZ independent

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

ionice: keep output backwardly compatible

 ...bug introduced by 8c219bf463dbbae412767d269bff1cdd4e4bf4a5

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

tests: update ipcs (reorder default output)

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

tests: update libblkid PT tests

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

tests: update UDF test

 .. blkid now returns infor from ISO header too.

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

tests: update ISO tests

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

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

docs: improve wording, grammar and formatting of lsblk man page

Also sort the options alphabetically.

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

docs: improve grammar and formatting of explanation of "-o +list"

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

build-sys: add exec_shell.h to distribution

Signed-off-by: Bernhard Voelker <mail@bernhard-voelker.de>

unshare: tweak style

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

unshare(1): delete empty NOTES section

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

login: remove file descriptor leak [cppcheck]

[login-utils/login.c:510]: (error) Resource leak: fd

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

wall: use xgethostname()

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

readprofile: fix --reest (--reset) typo

Reported-by: Branislav Blaskovic <notifications@github.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: use correct block size for UDF probe

In UDF, Volume Structure Descriptors are always 2048 bytes long (ECMA-167,
3rd ed., §2/9.1), while filesystem sectors are the same size as the sectors
of the underlying media (e.g., UDF 2.01 §1.3.2). Before this commit, the
block size was estimated from VSD offsets, which gives incorrect answers for
non-optical media. Now, the UDF probe uses the device block size by default,
falling back to 2048-byte sectors for the case of optical-disk images.

Signed-off-by: Zachary Catlin <z@zc.is>

libmount: ignore name=value if only 'name' is expected

For example

mount /srv/www /mnt -o rw,group=woven,dev,suid

the group= should not be interpreted as userspace mount option, because
umount(8) expects 'group' (without =<value>).

Reported-by: Jan Engelhardt <jengelh@inai.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

umount: add note about namespaces too --all-targets

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

docs: update deprecated.txt

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

tests: add umount --all-targets tests

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

umount: add --all-targets

The same source (whole filesystem or filesystems subvolume) could be
mounted on more targets (mountpoints). For example bind mounts or
btrfs sub-volumes.

If you want to detach the source device from the system then you need
to unmount all the mountpoints.

 umount --all-targets <device|mountpoint>

provides this functionality. This option could be used together with
--recursive. Then all nested mountpoints within the filesystem are
unmounted too. For example:

 umount --recursive --all-targets /dev/sdb1

should be a way how to make the device unmounted at all.

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

libmount: export umount mountpoint look up code

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

libmount: don't prepare update for empty mtab/utab on umount

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

libmount: optimize tab files parsing

 - ignore empty files
 - ignore empty tables

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

umount: improve --recursive docs

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

Merge branch 'patch-1' of https://github.com/dtrebbien/util-linux

* 'patch-1' of https://github.com/dtrebbien/util-linux:
  Implement mempcpy() in terms of memcpy() if mempcpy() is unavailable

add missing rpmatch.h everywhere it needs to be

Signed-off-by: William Pitcock <nenolod@dereferenced.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: add debug messages for "idmag" stuff and btrfs

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

libblkid: make backup superblock visible for wipefs(8) only

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

lib/tty: don't truncate output on non-terminals

 for example:

$ findmnt | cat
$ lslocks > foo

 the output should not be truncated.

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

libblkid: add blkid_probe_step_back to documentation

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

libblkid: return disk ID/UUID by PT binary interface

 Add blkid_parttable_get_id() function.

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

findmnt: allow to read (PART)UUID/LABEL from udev DB

Currently findmnt uses libmount to get filesystem properties (and
libmount uses libblkid). This works as expected, but root permissions
are required to read the tags from the devices.

It seems better to read the information from udev db and use libblkid
as fallback solution only.

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

lsblk: resuse udev handler

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

docs: add missing information about -o +<list>

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

lslocks: support -o +<list> convention

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

lslocks: add BLOCKER column

 BLOCKER -  PID of the process blocking the lock

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

libmount: fix a mem leak

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

fsck.cramfs: add dummy -a -y for compatibility

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

lslocks: detect blocked locks, fix /proc/locks parser

$ flock foo -c "sleep 100" & flock foo -c "sleep 100"

old version:
  $ lslocks
  lslocks: failed to parse pid: 'WRITE'

new version:

COMMAND           PID  TYPE  SIZE MODE   M      START        END PATH
[...]
flock            1318 FLOCK    0B WRITE* 0          0          0 /home/projects/
flock            1319 FLOCK    0B WRITE  0          0          0 /home/projects/

 The asterisk (e.g. WRITE*) is used for blocked processes.

Reported-by: Mantas Mikulenas <grawity@gmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

lslocks: use libmount to parse mountinfo

old version:
$ strace -e open lslocks 2>&1 | grep mountinfo | wc -l
118

new version:
strace -e open ./lslocks 2>&1 | grep mountinfo | wc -l
1

Reported-by: Bernhard Voelker <mail@bernhard-voelker.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

mountpoint: don't hardcode paths

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

lscpu: don't read cpuid on non-live systems

Reported-by: Luis Aranguren <pizzaman@hotmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

nsenter: fix errors in manpage

- spell abbreviations with capital letters
- fix the names of a few options and files

Signed-off-by: Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>

unshare,nsenter: spawn shell by default

The behaviour mimics chroot.

Possibly it would have been nicer to to query the password database in
the new namepace and run the shell of the user there, but it's hard to
do correctly. getpwuid() might need to load nss plugins, and the arch
in the new namespace might be different (in case of NEWNS mounts), or
the hostname might be different, etc. So in general it's not possible
to do it reliably.

Signed-off-by: Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>

libblkid: read whole buffer for iso_volume_descriptor

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

libblkid: Probe UDF volumes for ISO9660 info as well

I have seen blkid failing to report any meaningful data on some of UDF
volumes I have (Windows installer media):

$ ./blkid -p -o udev en_windows_7_professional_with_sp1_x64_dvd_u_676939.iso
ID_FS_LABEL=UDF_Volume
ID_FS_LABEL_ENC=UDF\x20Volume
ID_FS_TYPE=udf

Also, once my patches to expose more of ISO9660 info get merged, we'd
also need the same info to be exposed for UDF volumes.

This patch adds ISO9660 probing to UDF volumes and here is the result I
see on my example UDF volume (together with my pending patch to expose
more ISO9660 information):

$ ./blkid -p -o udev en_windows_7_professional_with_sp1_x64_dvd_u_676939.iso
ID_FS_PUBLISHER_ID=MICROSOFT CORPORATION
ID_FS_APPLICATION_ID=CDIMAGE 2.54 (01/01/2005 TM)
ID_FS_UUID=2011-04-12-02-38-58-00
ID_FS_UUID_ENC=2011-04-12-02-38-58-00
ID_FS_BOOT_SYSTEM_ID=EL TORITO SPECIFICATION
ID_FS_LABEL=GSP1RMCPRXFRER_EN_DVD
ID_FS_LABEL_ENC=GSP1RMCPRXFRER_EN_DVD
ID_FS_TYPE=iso9660

$ ./blkid -p -o udev en_windows_xp_professional_with_service_pack_3_x86_cd_x14-80428.iso
ID_FS_PUBLISHER_ID=MICROSOFT CORPORATION
ID_FS_APPLICATION_ID=CDIMAGE 2.52 (03/09/2004 TM)
ID_FS_UUID=2008-04-14-12-00-00-00
ID_FS_UUID_ENC=2008-04-14-12-00-00-00
ID_FS_BOOT_SYSTEM_ID=EL TORITO SPECIFICATION
ID_FS_LABEL=GRTMPFPP_EN
ID_FS_LABEL_ENC=GRTMPFPP_EN
ID_FS_TYPE=iso9660

Before anyone asks, Yes! The UDF media I have is all legal and downloaded
directly from MSDN. :)

[kzak@redhat.com: - read LABEL from UDF only if not found anything
                    useful in ISO9660]

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

libblkid: Expose more ISO9660 headers

In order to kill libosinfo's infamous udev rule[1], we need to make blkid
report the following information as udev properties on IS09660 media so
that libosinfo can make use of that for detection:

1. Volume ID (already exposed as label).
2. System ID.
3. Publisher ID.
4. Application ID.
5. Boot record's boot system ID, (almost always 'EL TORITO
   SPECIFICATION' if boot record is present).

Example use:

$ blkid -p -o udev Fedora-17-x86_64-DVD.iso
ID_FS_SYSTEM_ID=LINUX
ID_FS_APPLICATION_ID=GENISOIMAGE ISO 9660/HFS FILESYSTEM CREATOR (C)
1993 E.YOUNGDALE
ID_FS_UUID=2012-05-22-20-55-32-00
ID_FS_UUID_ENC=2012-05-22-20-55-32-00
ID_FS_BOOT_SYSTEM_ID=EL TORITO SPECIFICATION
ID_FS_LABEL=Fedora_17_x86_64
ID_FS_LABEL_ENC=Fedora\x2017\x20x86_64
ID_FS_TYPE=iso9660
ID_PART_TABLE_TYPE=dos

[1] https://fedorahosted.org/libosinfo/ticket/1

[kzak@redhat.com: - add blkid_ltrim_whitespace() and use it together
                    with blkid_rtrim_whitespace() to trim white spaces
                  - enlarge blkid values to 128 bytes
                  - add generic blkid_probe_set_id_label() function
                  - always terminate all _ID with \0
                  - don't export the _IDs to blkid cache]

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

libblkid: Don't keep reading same sector of ISO

We were reading the same sector over and over again when parsing ISO9660.
This patches fixes it.

build-sys: add package url to AC_INIT

See last line from './configure --help'.

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

build-sys: use AC_USE_SYSTEM_EXTENSIONS

The AC_GNU_SOURCE is obsolete.

Addresses: http://www.gnu.org/savannah-checkouts/gnu/autoconf/manual/autoconf-2.69/html_node/Obsolete-Macros.html#index-AC_005fGNU_005fSOURCE-2058
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: use AC_COMPILE_IFELSE

The AC_TRY_COMPILE is obsolete.

Addresses: http://www.gnu.org/savannah-checkouts/gnu/autoconf/manual/autoconf-2.69/html_node/Obsolete-Macros.html#index-AC_005fTRY_005fCOMPILE-2203
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

libmount: add missing MNT_ERR_AMBIFS

Sorry, the last commit into libmount has been incomplete.

Reported-by: Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Signed-off-by: Karel Zak <kzak@redhat.com>

switch_root: Add a sanity check

As switch_root basically does rm -Rf / we should make sure
that / is really an initramfs.

Signed-off-by: Richard Weinberger <richard@nod.at>

build-sys: add config-gen.d stuff for chsh-chfn

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

chsh-chfn: fix bugs, improve compilation

 * rename --disable-require-password to --disable-chsh-chfn-password

 * is_local() is really unnecessary when linked with libuser

 * fix set_value_libuser() returns codes

 * fix chfn.c, there is no 'pw', but oldf.pw

 * don't link with PAM when--disable-chsh-chfn-password

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

libmount: correctly propagate ambivalent blkid probing results

libmount ignores "ambivalent probing result" from libblkid and tries
filesystems /etc/filesystems. This is incorrect behavior.

Reported-by: Lukas Czerner <lczerner@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>