libparted: optimize _disk_sync_part_table

In _disk_sync_part_table, if the largest partition number (lpn) is 0,
it does not make sense to go through the loops below.  Return
immediately instead.
* libparted/arch/linux.c (_disk_sync_part_table): Return immediately
if lpn == 0.

build: ignore more generated files

parted: remove obsolete comments

is_start_exact and is_end_exact were removed from the source.
* parted/parted.c (constraint_from_start_end): Remove obsolete
documentation of the function.

libparted: check return value from ped_geometry_init

If ped_geometry_init() failed in ped_disk_set_partition_geom(),
then new_geom remains uninitialized, but still later used, which
leads to unexpected results.
* libparted/disk.c (ped_disk_set_partition_geom): Check return
value from ped_geometry_init().

tests: add test for partitionable loop devices

* tests/t8001-loop-blkpg.sh: New file.
* tests/Makefile.am: Add test.

libparted: improve support for partitions on loopback devices

Since linux-2.6.26, the kernel allows partitions on loopback devices.
Implement support for this feature in parted.
* libparted/arch/linux.c (_sysfs_int_entry_from_dev): New function.
(_loop_get_partition_range): New function.
(_device_get_partition_range): Add special handling for loop devices.
* NEWS: Mention this change.

libparted: differentiate between plain files and loop devices

Stop using PED_DEVICE_FILE for loopback devices;
loopback are significantly different from plain files.
* include/parted/device.h (PedDeviceType): Add PED_DEVICE_LOOP.
* libparted/arch/linux.c (_device_probe_type): Detect loopback device.
* parted/parted.c (do_print): Add "loopback" to list of transports.

dvh: factor out read_sector and use ptt_read_sector

* libparted/labels/dvh.c (read_sector): Factor out this function.
(dvh_probe): Use ptt_read_sector instead of read_sector.
(dvh_read): Likewise.

build: update gnulib to latest; adapt use of _GL_ATTRIBUTE_FORMAT

* libparted/arch/linux.c (_GL_ATTRIBUTE_FORMAT): Define here,
now that it's no longer defined by gnulib.

build: avoid new syntax-check failure

* cfg.mk: Exempt libparted/arch/beos.c from the PATH_MAX check.
I suspect that this is ok, because parted compiles only on systems
that define PATH_MAX to something reasonable.

doc: mention the origin of the other two bugs

* NEWS: Mention that the other two bugs were introduced long ago.

tests: test for the nilfs2 bug

* tests/t4300-nilfs2-tiny.sh: New test.
* tests/Makefile.am (TESTS): Add it.

libparted: fix a bug in the nilfs2 probe function

* libparted/fs/nilfs2/nilfs2.c (nilfs2_probe): Reject this partition
if we get a negative sb2 offset.  Passing a negative offset to
ped_geometry_read_alloc would evoke a failed assertion.
Bug introduced by 2010-07-09 commit d463e7de.
* NEWS: (Bug fixes): Mention it.
Reported by Daniel Fandrich in
http://thread.gmane.org/gmane.comp.gnu.parted.bugs/10466/focus=10472

gpt: don't abort for a truncated GPT-formatted device

This fixes the problem two ways.  The first fix (via gpt_alloc)
rejects any device that is too small, but it is insufficient.
Choose a slightly larger truncated device with an otherwise intact
primary GPT header and you can still trigger the failed assertion.
To fix it in general, we make _header_is_valid detect the problem.
* libparted/labels/gpt.c (gpt_alloc): Reject a device that is so
small that there is no room for a single partition.
(_header_is_valid): Validate LastUsableLBA here, as well, so that
we now reject as invalid any GPT header that specifies a
LastUsableLBA larger than the device size.
Leave the assertion in _parse_header.
* tests/t0203-gpt-tiny-device-abort.sh: Test for this.
* tests/Makefile.am (TESTS): Add it.
* NEWS: (Bug fixes): Mention it.
Reported by Daniel Fandrich in
http://thread.gmane.org/gmane.comp.gnu.parted.bugs/10466

libparted: avoid invalid diagnostic in interactive mode

* libparted/arch/linux.c (linux_read): In interactive mode,
a truncated device with a preexisting GPT could evoke this:
  Error: end of file while reading Success
it should have been like this:
  Error: end of file while reading /full/name/of/device
Use %0.0s to discard the strerror argument.

tests: add test for value less than 1

* tests/t9023-value-lt-one.sh: New file.
* tests/Makefile.am (TESTS): Add it.

tests: add test for radius divide by 2 fix

* tests/t9022-one-unit-snap.sh: New file.
* tests/Makefile.am (TESTS): Add it.

libparted: don't allow values less than 1 (#665496)

When a value < 1 is used there is the possibility that the range can
overlap sector 0. The user should use smaller units instead. 0 is a
special case and is still allowed.

* libparted/unit.c (ped_unit_parse_custom): Throw error if a value
between 0 and 1 is used.

libparted: fix snap radius so that it is using half (#665496)

The snap radius didn't match the documentation, it has been using +/-
unit size instead +/- 0.5 * unit (eg. 500KB for a MB unit). This caused
problems when specifying 1MB, 1GB, etc. as a partition start or end
resulting in partitions being created that were nowhere near the
specified size.

* libparted/unit.c (ped_unit_parse_custom): divide radius by 2
This addresses http://bugzilla.redhat.com/665496

maint: prefer AC_PROG_LIBTOOL over slightly deprecated AM_PROG_LIBTOOL

* configure.ac: Use AC_PROG_LIBTOOL rather than slightly deprecated
AM_PROG_LIBTOOL.

libparted: simplify kernel version-parsing logic

* libparted/arch/linux.c: (_get_linux_version): Simplify and add
an assertion.

libparted: accommodate two-component linux version number like 3.0

* libparted/arch/linux.c (_get_linux_version): Also accept 2-component
version numbers.
* NEWS: (Bug fixes): Mention it.
Reported by Richard W.M. Jones.

post-release administrivia

* NEWS: Add header line for next release.
* .prev-version: Record previous version.
* cfg.mk (old_NEWS_hash): Auto-update.

version 3.0

* NEWS: Record release date.

maint: placate coverity: don't ignore some return values

* libparted/cs/constraint.c: Include <assert.h>.
(ped_constraint_exact): Don't ignore ped_alignment_init's return
value or that from ped_geometry_init.

loop: avoid NULL dereference upon failure

* libparted/labels/loop.c (loop_partition_duplicate): Don't dereference
NULL upon failure.

shared library version: increment major: 1.0.0

* libparted/Makefile.am (CURRENT, REVISION): Update from 0.2.0 to 1.0.0,
due to removed interfaces.

build: update gnulib submodule to latest

doc: update NEWS

* NEWS (Changes in behavior): Describe the change.
(Bug fixes): Mention the leak fixes.

doc: reflect removal of FS-related commands

* README (WARNING): Say that all FS-manipulation code is now gone.
* doc/parted.texi (Command explanations): Remove descriptions of and
any examples or references to the following commands: check, cp,
mkfs, mkpartfs, move, resize.

build: remove configure-time options: --disable-fs, --enable-Werror

* configure.ac: Remove support for --enable-fs and --disable-fs.
Remove support for --enable-Werror and --disable-Werror.
* README: Remove paragraph about --disable-fs and
--enable-discovery-only.
* libparted/libparted.c: Now that --disable-fs is gone (i.e.,
ENABLE_FS would be always 1), remove its #ifdefs.

tests: make t0220 more useful when it fails

maint: update po/POTFILES.in

* po/POTFILES.in: Remove file names, to reflect file deletions.

doc: remove FS-related documentation and other stale bits

* doc/parted.texi: Remove FS-related documentation and other
stale bits.

linux_swap: remove all-but-probe FS-related code

xfs: remove all-but-probe FS-related code

jfs: remove all-but-probe FS-related code

ufs: remove all-but-probe FS-related code

reiserfs: remove all-but-probe FS-related code

hfs: remove all-but-probe FS-related code

ntfs: remove all-but-probe FS-related code

amiga: build with warning and error options

* libparted/fs/amiga/Makefile.am (AM_CFLAGS): Add $(WARN_CFLAGS)
and $(WERROR_CFLAGS).

ext2: remove all-but-probe FS-related code

fat: remove all-but-probe FS-related code

remove all top-level FS-op code and the clearfat program

UI: remove all file-system-aware sub-commands

Note that we are removing the resize command, even though parted
appears to be the only free tool that provides the ability to
resize FAT16 and FAT32 file systems.
* parted/parted.c (do_mkfs, do_cp, do_mkpartfs): Remove functions.
(do_move, do_check, do_resize): Likewise.
(_init_commands): Remove each command-definition clause.
* parted/parted.c (_partition_warn_loss): Remove now-unused function.
* include/parted/filesys.h: Remove declarations of now-removed
FS-munging functions

tests: t1100: show that adding a partition on a busy device succeeds

* tests/t1100-busy-label.sh: Demonstrate that adding a partition
succeeds, even when another on the same device is mounted (active).

amiga: avoid NULL-deref-on-OOM and an error-path leak

* libparted/labels/rdb.c (amiga_read): Don't dereference NULL on OOM.
Don't leak a constraint when failing to add a partition.

mac: avoid NULL-deref-on-OOM and an error-path leak

* libparted/labels/mac.c (mac_read): Don't dereference NULL on OOM.
Don't leak a constraint when failing to add a partition.

sun: avoid NULL-deref-on-OOM and an error-path leak

* libparted/labels/sun.c (sun_read): Don't dereference NULL on OOM.
Don't leak a constraint when failing to add a partition.

UI: avoid leak when interactive sector selection matches default

* parted/ui.c (command_line_get_fs_type): Don't leak an input "word".

UI: plug a leak

* parted/ui.c (command_line_get_fs_type): Don't leak an FS type name
string.

bsd: avoid NULL-deref-on-OOM and an error-path leak

* libparted/labels/bsd.c (bsd_read): Don't dereference NULL on OOM.
Don't leak a constraint when failing to add a partition.

linux: don't leak a FILE* upon /proc read failure

* libparted/arch/linux.c (_probe_proc_partitions): Call fclose
upon failure.

linux: plug a file descriptor leak

* libparted/arch/linux.c (read_device_sysfs_file): Don't leak a
file descriptor.

tests: t1100-busy-label: don't use mkpartfs; rework to use scsi_debug

* tests/t1100-busy-label.sh: Remove one last use of mkpartfs.
Use scsi_debug rather than requiring an explicitly-provided device.

maint: correct coverity-reported "suspicious sizeof"

* parted/table.c (table_add_row): Use sizeof(*VAR), rather than
erroneous sizeof(explicit_type) that just happened to be ok.

maint: remove unreachable code

* libparted/labels/sun.c (sun_partition_new): Remove unreachable stmt.
* libparted/labels/pc98.c (pc98_partition_new): Likewise.

build: update gnulib submodule to latest

post-release administrivia

* NEWS: Add header line for next release.
* .prev-version: Record previous version.
* cfg.mk (old_NEWS_hash): Auto-update.

version 2.4

* NEWS: Record release date.

shared library version: increment REVISION

* libparted/Makefile.am (REVISION): Increment, since there have
been changes to the library, but no interface addition or removal.
* README-release: Mention the shared-library version update procedure.

build: update gnulib submodule to latest

tests: avoid nasty FP-causing race condition

* tests/t9040-many-partitions.sh: Remove all partitions before
exiting -- otherwise, the rmmod-prompted removal of those many
partitions would continue long after rmmod completion and the
release of the scsi_debug lock, and thus partition-device removals
provoked by this test would cause failure in the subsequent test,
e.g., when the devices for partitions 14,15,16 have just been
successfully created (verified that they exist), yet the next
step -- to create a file system on one of them -- fails or is
erroneously skipped because the device has been removed asynchronously.

tests: add double quotes around another use of $VERBOSE

* tests/t-local.sh (scsi_debug_setup_): Quote another $VERBOSE.

build: list bc as a build-requirement

* bootstrap.conf (buildreq): Also require "bc", since it's used
in some of 'make check's tests.

tests: remove tests that are too FS-centric

* tests/t2100-mkswap.sh: Likewise.
* tests/t3000-resize-fs.sh: Remove file, now that all FS-ops are gone.
* tests/Makefile.am (TESTS): Remove them here, too.

linux: fix diagnostic about operating on very small file or device

* libparted/arch/linux.c (init_file): Correct diagnostic.
Parted would claim that a small-but-non-empty file has zero length.

bfs: remove unused files

tests: add double quotes around a use of $VERBOSE

* tests/t-local.sh (scsi_debug_cleanup_): Add missing double quotes.

tests: avoid new, spurious test failure

* tests/t9050-partition-table-types.sh: Use a "device size"
that is large enough to accommodate 1KiB and 4KiB sectors.
The original size, 3MiB was too small for a dvh partition
with PARTED_SECTOR_SIZE=1024.  Increase it to 30MiB.

tests: adapt t1101-busy-partition not to use mkpartfs

* tests/t1101-busy-partition.sh: Use mkfs.vfat instead.

tests: write all pairs of partition table types

* tests/t9050-partition-table-types.sh: New file.
* tests/Makefile.am (TESTS): Add it.

tests: add a NEWS entry and a test to exercise today's bug fix

* tests/t9041-undetected-in-use-16th-partition: New file.
* tests/Makefile.am (TESTS): Add it.
* NEWS (Bug fixes): Describe it.

linux: also detect "in-use" dmraid and scsi-Nth (N>=16) partitions

Prior to this change, Parted would fail to detect any in-use
partition on a dmraid disk, as well as any in-use partition
numbered 16 or greater on a regular scsi disk.
This was because assumptions were being made about the device node
major/minor numbers and how they relate to the partition number.
These assumptions are invalid for dmraid devices, and for normal
disks with >= 16 partitions.
* libparted/arch/linux.c (_partition_get_part_dev): Remove function.
(_partition_is_mounted): Rewrite to use _partition_is_mounted_by_path
instead of _by_dev.

build: update gnulib submodule to latest

tests: avoid subtle shell semantics bug

This bug was causing some root-only tests to be mistakenly skipped.
This bash command will always exit successfully,
i.e., the "return 1" will never happen:
  bash -c 'f(){ local f=$(false)||return 1; return 0; }; f'
That's because the assignment always succeeds, and that success
is what determines the return value, not the $(...) command.
This is very counter-intuitive, sigh.
The work-around is to separate the declaration and assignment, as in
local f; f=$(...) || ...
* tests/t-local.sh (new_sdX_): Do that.
(gpt_corrupt_primary_table_): Likewise.

tests: minor fix; add/adjust comments

* tests/t1101-busy-partition.sh: Don't ignore failed comparison.
Add a comment.

s390/dasd: avoid buffer overrun upon API mismatch diagnostic

* libparted/labels/fdasd.c (fdasd_check_api_version): The static
buffer for the error string was too small.  Double its length.
This resolves http://bugzilla.redhat.com/693852

linux: fix typo in comment: /path%d => /part%d

* libparted/arch/linux.c (_device_get_part_path): Fix typo in comment.

linux: don't free invalid pointer upon asprintf failure

* libparted/arch/linux.c (_device_get_part_path): When asprintf
fails, it leaves its first argument in an undefined state, and
hence that pointer must not be freed.  However, here, in two
places we could potentially free an invalid pointer.  Use
zasprintf; then the pointer is either NULL or allocated,
and hence always freeable.

linux: clean up device naming code (no semantic change)

* libparted/arch/linux.c (zasprintf): New function.
(_device_get_part_path): Clean up:
Use size_t, not "int" for strlen-returned value.
Combine mostly duplicate snprintf uses.
Use zasprintf instead of malloc+snprintf.

don't reference before start of buffer for short device name

* libparted/arch/linux.c (_device_get_part_path): Avoid invalid
reference to memory before dev->path when its length is 4 or less.

maint: s/can not/cannot/ in a comment and documentation

Without this, a new "make syntax-check" rule would fail.
* libparted/filesys.c: s/can not/cannot/
* doc/parted-pt_BR.texi (BIOSes and Firmware): Likewise.

tests: remove now-unused Makefile rule

* libparted/tests/Makefile.am (old-init.sh): Remove obsolete rule.

tests: improve t1101-busy-partition.sh

* tests/t1101-busy-partition.sh: Fail hard upon timeout while
waiting for device to appear.

tests: make the tests in libparted/tests work properly once again

Since switching to init.sh, the root-only tests did not clean up.
* libparted/tests/Makefile.am (prereq): Always create a symlink
to tests/init.sh.
* tests/init.cfg: Source $abs_top_srcdir/tests-relative names,
not $srcdir-relative names.  This way, it works also when run
from libparted/tests/.
* tests/t-local.sh: Likewise.

tests: avoid a race condition in the scsi_debug-using tests

* tests/t-local.sh (new_sdX_): New function.
(scsi_debug_setup_): Use new_sdX_ to find the just-created device name,
rather relying on the list of file names in /sys/block/sd* changing.
Sometimes the list would not change, which would lead to tests failing
with I/O errors when run in parallel.  Thanks to Brian Lane for the
tip that /sys/block/sd*/device/model contains "scsi_debug" if there
is a device created by our "modprobe scsi_debug" command.

make parted "do what I say" with IEC start and end values like 5GiB

In a command like this,
  parted -s -- $dev mklabel gpt mkpart P-NAME 4MB -34s
parted interprets the "4MB" as merely a suggestion for the starting
sector number.  It uses half of the MB-units value as a "radius"
about the sector containing byte 4,000,000, and it may choose some
other more appropriate sector, based on partition table or alignment
constraints within the range 3,500,000B..4,500,000B.
Before, parted handled IEC units, i.e., KiB, MiB, GiB, etc.,
with identically "helpful" sloppiness, of course honoring the
power-of-two semantics.
Now, however if you use IEC units, i.e., KiB, MiB, GiB, etc.,
the "radius" is taken to be zero, so parted uses precisely
whatever multiple of a power of two you've specified.
Hence, adjusting the example above to use MiB, rather than "MB",
  parted -s -- $dev mklabel gpt mkpart P-NAME 4MiB -34s
With this change, that is equivalent to the following:
  parted -s -- $dev mklabel gpt mkpart P-NAME 4194304B -34s
I.e., it uses the sector containing precisely that byte, and
does not perform any "extra" adjustment.
* libparted/unit.c (is_power_of_2): New function.
(ped_unit_parse_custom): Use it to avoid interpreting a large
input string as "sloppy" (i.e. large radius) when it uses IEC
binary notation like 34KiB, 3GiB and 65TiB.
* tests/t0207-IEC-binary-notation.sh: New test.
* tests/Makefile.am (TESTS): Add it.
* doc/parted.texi (unit): Describe the new behavior.
* NEWS (Changes in behavior): Mention it here, too.

maint: stop using .x-sc_* files to list syntax-check exemptions

Instead, use the new mechanism with which you merely use a
variable (derived from the rule name) defined in cfg.mk to an ERE
matching the exempted file names.
* gnulib: Update to latest, to get maint.mk that implements this.
* .x-sc_GPL_version: Remove file.
* .x-sc_prohibit_always-defined_macros: Likewise.
* Makefile.am (EXTRA_DIST): Remove them from here, too.
* cfg.mk: Define variables to exempt the same files.

maint: rename a variable

* libparted/labels/gpt.c (_header_is_valid): Rename a variable,
and switch to more readable range-checking logic.

gpt: avoid division by zero

* libparted/labels/gpt.c (_header_is_valid): Reject as invalid if
FirstUsableLBA < 3.
(gpt_get_max_supported_partition_count): Ensure that we don't divide
by zero: verify that the GPT header is valid before dividing by its
"size of partition entry".  Under normal circumstances, the on-disk
PE size field is reasonable because we have just written it.  However,
there are two ways it can end up zero: we lose the race when some other
process pokes a 4-byte 0 into just the right location between when
we write it and when we re-read the value we're about to divide by.
Then there's the case that I encountered: using an old USB (8MB) key,
mklabel gpt failed due to division by zero.  The device reported no
failure when writing the initial header, yet when reading back that
very same sector (also successful), parted got all 0 bytes.
* NEWS (Bug fixes): Mention it.

build: enable compilation warnings also for new nilfs2 subdir

* libparted/fs/nilfs2/Makefile.am (AM_CFLAGS): Define.

nilfs2: remove unused nilfs2_clobber function

* libparted/fs/nilfs2/nilfs2.c (nilfs2_clobber): Remove unused
function.
(nilfs2_ops) [clobber]: Always set to NULL.

maint: avoid shadowing warning

* parted/parted.c (do_mkpart): Rename inner added_ok to add_ok.

build: don't let parted_CFLAGS override our AM_CFLAGS setting

* parted/Makefile.am (parted_CFLAGS): Remove unused (and interfering)
definition.

maint: avoid new compiler warnings

* parted/parted.c (do_print): Remove now-unused "error" label.
(_print_disk_info): Use one more "const" attribute, to avoid
compiler warnings.

tests: do not fail if some utilities are missing

* tests/t1700-probe-fs.sh: Skip the test if mkfs.ext3 or tune2fs
is missing or is not in $PATH.

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

tests: test 'parted print' with empty disk

* tests/t0101-print-empty.sh: New test.
* tests/Makefile.am: Add it.

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

parted: rework 'parted print' so that it works with empty disk

'parted $dev print' on a device without a partition table used to fail
with an 'unrecognised disk label' error, without printing the following:
disk model, transport, size, sector size, BIOS geometry
all of which are available, and do not dependent on the partition table.

With this patch, parted prints all of this information (BIOS geometry
only if 'unit cyl' is specified) and reports success even if the disk
does not have a valid partition table.

As a side efect, 'parted $dev print devices/all/list' now prints
information about all the devices, even if $dev does not have a
partition table.

* parted/parted.c (_print_disk_info): New function.
(do_print): Do not immediately fail if the partition table is not
recognized, but print disk information and report success instead.
* NEWS (Changes in behavior): Mention it.

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>