script: may be hangs

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

blkdiscard: fix underflow when offset is greater than device size

If offset (range[0]) is greater than device size (blksize), the variable 'end'
will be greater than blksize, and range[1] (length) will be recalculated.
The underflow happens when subtracting range[0] (offset) from blksize, thus
range[1] will be the result of an underflow. The bug leads to unwanted behavior
from the program, where range[1] is likely to be a high number and then will
discard a considerable amount of blocks from the device. The fix consists of
exitting the program with an error message when the condition stated above is
true. Spotted while auditing the code.

Signed-off-by: Raphael S. Carvalho <raphaelsc@cloudius-systems.com>

build-sys: check for libtoolize rather than libtool

The system libtool program has architecture dependent behaviour. It is
therefore unavailable in cross build environments. The only place it was
used in util-linux is autogen.sh to determine the availability of
libtool. All other places correctly use libtoolize or
$(top_builddir)/libtool.

Signed-off-by: Helmut Grohne <helmut@subdivi.de>

textual: grammarize an informative message, and tweak some comments

Also add two clarifying comments for translators.

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

libmount: provide version as int for better compiletime test

Signed-off-by: Søren Holm <sgh@sgh.dk>

dmesg: improve reltime colors

[  +0.004034] sd 113:0:0:0: [sdb] Attached SCSI disk
[Oct10 13:04]  sdb: unknown partition table
[  +0.034011]  sdb: sdb1 sdb2 sdb3 sdb4 < sdb5 sdb6 >

.. the "Oct10 13:04" is possible to colorize by "timebreak"
scheme (default is bold green).

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

tests: add PMBR sfdisk test

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

tests: add sfdisk GPT tests

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

libfdisk: fix script next_token()

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

sfdisk: leave on error

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

libfdisk: make script header parser more pedantic

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

tests: test cazy sizes for sfdisk

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

libfdisk: (dos) reset logical partition stuff after error

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

tests: update sfdisk tests

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

libfdisk: cleanup script dump

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

sfdisk: ask on EOF

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

libfdisk: (dos) follow explicit sizes

* this patch add to dos driver support for fdisk_partition->size_explicit
  to avoid unexpected last sector alignment for sizes explicitly specified
  in sectors

* add support for small "first LBA", the current default is to use
  1MiB offset for the first partition and for each EBR. This is not
  backwardly compatible and it makes impossible to apply sfdisk
  scripts/dumps from old systems, because original offset can be
  smaller than 2048 sectors (old sfdisk default is 1 sector).

  The solution is on the fly to detect this situation and change
  fdisk_context->first_lba to 1 sector. Nasty.

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

libfdisk: (gpt) follow explicit size, fiz size+1 buf

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

libfdisk: follow explicitly specified partition size

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

libfdisk: add functions to play nasty games with first/last LBA

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

fdisk: reset cached output fields when change disk label

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

libfdisk: allow to specify partition size by explicit number of sectors

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

libfdisk: use skip_blank() in scripts

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

sfdisk: cleanup isatty() usage

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

libfdisk: (dos) don't inform about logical partition

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

libfdisk: consolidate script parser code

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

libfdisk: fix script parser

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

libfdisk: (dos) fix booable flag usage

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

sfdisk: add --label-nested for hybrid GPT

regular disk label:
# sfdisk --list /dev/sdb
..
Disklabel type: gpt
Disk identifier: 9DF9A9F1-0654-4E7A-9A5E-36E66D60FD79

Device     Start    End Sectors Size Type
/dev/sdb1   2048  22527   20480  10M Linux filesystem
/dev/sdb2  22528  43007   20480  10M Linux swap
/dev/sdb3  43008 204766  161759  79M Linux filesystem

nested (PMBR):
# sfdisk --list --label-nested dos /dev/sdb
...
Disklabel type: dos
Disk identifier: 0x00000000

Device     Boot Start    End Sectors  Size Id Type
/dev/sdb1           1 204799  204799  100M ee GPT

and for example:
        # sfdisk --label-nested dos /dev/sdb <<EOF
        1, 2047, ee
        , 10M, L
        , 10M, S
        ,,

creates hybrid GPT (PMBR partitions point to the same location as GPT)

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

libfdisk: propagate assign-device operation to parent

Let's make it possible to use fdisk_assign_device() for nested
contexts.

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

libfdisk: fix sun and sgi to be usable from sfdisk

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

fdisk: fix typo

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

sfdisk: add --output <list> for print command(s)

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

fdisk: add --output <list> for print command(s)

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

libfdisk: add fdisk_label_get_field_by_name() and const for labels

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

libfdisk: (gpt) remove 'Cylinders'

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

sfdisk: add --part-attrs

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

libfdisk: (gpt) parse attr bits

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

build-sys: add hint about 'make'

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

libfdisk: (dos) use script label-id

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

sfdisk: cleanup option names

Rename options to --part-{label,uuid,type} to be compatible with
another tools (like lsblk, parts, etc).

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

sfdisk: add --name

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

sfdisk: add --uuid command

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

sfdisk: more update to the man page

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

sfdisk: update man page

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

sfdisk: add --append

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

sfdisk: be sensitive to PT limits

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

sfdisk: add --backup and --backup-file

The option --backup force sfdisk to store *all* fragments of the
partition table (including MBR partition tables store in the
extended partitions) to

$HOME/sfdisk-<devname>-<offset>.bak

The options -O, -backup-file <path> allows to override the default
path, but sfdisk still appends <devname>-<offset>.bak to the <path>.
The backup files always contain only raw data from the device, so it's
possible to use dd(1) to restore original data on the device.

The original sfdisk also supported -O <file>, but semantic was little
bit different:

   - all was based on 512-byte sectors
   - all sectors was stored to the one file in format
  <offset>|<sector>|<offset>|...

this original concept makes the backup files specific to sfdisk and with
dependence on sector size.

The new concept is the same we already use for wipefs(8) backup files.

Example (disk with GPT):

   # sfdisk /dev/sda --backup

   Welcome to sfdisk (util-linux 2.25.202-f4deb-dirty).
   Changes will remain in memory only, until you decide to write them.
   Be careful before using the write command.

   Backup files:
          PMBR (offset     0, size   512): /root/sfdisk-sda-0x00000000.bak
    GPT Header (offset   512, size   512): /root/sfdisk-sda-0x00000200.bak
   GPT Entries (offset  1024, size 16384): /root/sfdisk-sda-0x00000400.bak

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

sfdisk: add --no-reread and --force

and also check if the device is in use.

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

sfdisk: add --no-act

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

sfdisk: add deprecated --Linux

The sfdisk does not care about compatibility with classic DOS
partitioning, and it does not warn about incompatibility with DOS at
all. It means that --Linux is default and it's unnecessary to use
this option.

It's the same situation like with "--unit S", these options are very
probably often used in scripts, and these all is default now. So for
backward compatibility new sfdisk accepts these options on command
line, but prints "option is deprecated" warning message.

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

sfdisk: add --quiet

Note that original sfdisk implementation suppressed warning
messages for --quiet.

Now we keep warning and error messages visible, but suppress
extra info messages only (for example to make it more usable in
scripts). IMHO suppress warnings is bad idea.

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

sfdisk: add --unit and --show-geometry, update deprecated.txt

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

libfdisk: remove old label drives' API for parttypes

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

libfdisk: (sgi) add set_part() API

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

libfdisk: (bsd) add set_part() API

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

libfdisk: use new set_part API in fdisk_set_partition_type()

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

libfdisk: (gpt) check string_to_guid()

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

sfdisk: cleanup sync() usage, fix --activate

 * use sync() only if we change disk layout
 * don't use sync() when we change details (like partition type)

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

linfdisk: cleanup fsync usage

 * always use fsync() if the device open read-write
 * use sync() on demand

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

sfdisk: use fdisk_set_partition_type()

This is more simple than the generic fdisk_set_partition() API.

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

sfdisk: cleanup usage() and long options

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

sfdisk: add --parttype

The patch also makes --{id,change-id,print-id} deprecated in favour
of --parttype. The original --id is too generic option name and the
--print-id and --change-id are unnecessary and inconsistent with
another sfdisk options (e.g. we don't have --change-bootable)

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

sfdisk: add --verify

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

sfdisk: add --list-types

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

sfdisk: add missing stuff to usage()

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

sfdisk: make dump format usable on stdin

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

libfdisk: (gpt) use generic API to change UUID and name

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

libfdisk: (gpt) implement fdisk_set_partition() backend

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

libfdisk: (dos) implement fdisk_set_partition() backend

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

libfdisk: add fdisk_set_partition()

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

libfdisk: cleanup internal drivers' API

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

sfdisk: support -N for primary partitions

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

libfdisk: add fdisk_copy_parttype()

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

sfdisk: require confirmation before write to the device

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

libfdisk: make yes/no dialogs more robust

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

sfdisk: improve UI

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

libfdisk: cleanup script size= code

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

libfdisk: return partno when add new partition

 * improve the way how sfdisk report results
 * the API change simplify applications

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

sfdisk: implement command_fdisk()

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

libfdisk: fix script partitions size parsing

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

libfdisk: (dos) more carefully check for primary partitions

  * check if there is space for primary partitions, if no and extended
    exists then create logical partitions

  * create extended if @pa->type is extended partition

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

sfdisk: cleanup comments and command_activate()

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

sfdisk: add --activate

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

libfdisk: add fdisk_partition_is_bootable()

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

sfdisk: add comments, remove unnecessary function

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

sfdisk: add --show-size

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

fdisk: refactor /proc/partitions usage

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

sfdisk: update usage()

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

sfdisk: add --dump functionality

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

sfdisk: --list just one device

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

sfdisk: implement --list

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

fdisk: improve --list output

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

fdisk: move --list functionality to separated file

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

sfdisk: basic main() and friends

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

tests: add sfdisk test

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

libmount: fix mnt_is_readonly() #ifdef

This issue affects util-linux portability to GNU/HURD

Reported-by: Pino Toscano (from Red Hat)
Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: zeroize errno on blkid_probe_get_buffer() success

Since 37f4060225df0591ab8e1dd676dbc8115d900d4f prober functions are
sensitive to errno, it seems more robust to set errno=0 with in
blkid_probe_get_buffer() on success than set the zero on all places
where we call blkid_probe_get_buffer().

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

flock: zero timeout is valid

This patch reverts Sami's "timeout cannot be zero", introduced
in commit 605325b23b36238c8f3ae165e37cab9064553cf7.

The --timeout 0 has been originally interpreted as --nonblock. The
patch also add hint about this behavior to the man page.

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

losetup: improve man page -D docs

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