Make 'swap' command work like fdisk(8) says -- you must supply two valid
partition numbers.

Use strtonum() instead of strtol() inside ask_num(). Many overflows
are thus avoided. Since bounds are now reliable don't check the
returned value for being in-bounds. Since default value is
forced i

Use strtonum() instead of strtol() inside ask_num(). Many overflows
are thus avoided. Since bounds are now reliable don't check the
returned value for being in-bounds. Since default value is
forced inside bounds, don't bother being tricky and passing a
default that is outside the bounds being specified.

ok beck@

show more ...

set_pid() does not need prompt string, low or high parameters. These
are all constants. Independantly suggested by guenther@.

Split out an ask_pid() function rather than over-parameterizing the
ask_num() function. Remove now unneeded 'flags' and 'help' parameters
from both. Display out of range values in hex in ask_pid(), s

Split out an ask_pid() function rather than over-parameterizing the
ask_num() function. Remove now unneeded 'flags' and 'help' parameters
from both. Display out of range values in hex in ask_pid(), since
we are seeking hex input.

ok guenther@

show more ...

Call Xsetpid() to edit the partition type from Xedit() rather than
duplicating the code. Allows simplification of the EDIT() #define
in Xedit() since we always ASK_DEC and use a NULL help parameter w

Call Xsetpid() to edit the partition type from Xedit() rather than
duplicating the code. Allows simplification of the EDIT() #define
in Xedit() since we always ASK_DEC and use a NULL help parameter when
invoking ask_num().

show more ...

Nuke useless EDIT() #define in Xsetpid.

A #define to simplify a function calling snippet does not simply or
help when it is used only once.

Replace tricker atoi() and hand rolled parsing with strsep() and
strtonum(). Make related error messages consistant.

ok haesbaert@

Replace atoi() with strtonum() where it's easy. Make related error
messages consistant. Check for valid partition when 'select'ing a
partition.

ok haesbaert@

Before writing it to disk, warn the user if their new MBR contains
more than one OpenBSD partition.

ok deraadt

A zero-length partition is a non-partition, so reject a partition
length of 0. And anyway PRT_fix_CHS() writes garbage when length
is zero.

Refresh the in-memory copy to reflect any data that was wr

A zero-length partition is a non-partition, so reject a partition
length of 0. And anyway PRT_fix_CHS() writes garbage when length
is zero.

Refresh the in-memory copy to reflect any data that was written
out, so you can see that your changes have taken effect.

Surprise behaviours noted by matthew@ at c2k11.

show more ...

on error, getuint() will return UINT_MAX. Instead of actually using
that value, print an error message and repost the question

ok krw@ deraadt@

spelling fixes, from Brad Tilley; we will not fix src/sbin/dump/dump.h
as neither arrayified not arrayfied exist -- sanctioned dictionaries
like Merriam-Webster ones suggest a few alternatives (e.g.,

spelling fixes, from Brad Tilley; we will not fix src/sbin/dump/dump.h
as neither arrayified not arrayfied exist -- sanctioned dictionaries
like Merriam-Webster ones suggest a few alternatives (e.g., arrayed),
however these made up words are easy to understand and we are not
certain that current ones are not ok.

ok jmc@

show more ...

Eliminate excessive verbiage for 'fdisk -i' and 'fdisk -u'. Especially
the multi-line banner announcing that the MBR is being changed.
Also the listing of the partition table in 'fdisk -u'. Display a

Eliminate excessive verbiage for 'fdisk -i' and 'fdisk -u'. Especially
the multi-line banner announcing that the MBR is being changed.
Also the listing of the partition table in 'fdisk -u'. Display a
consistant message when the MBR is written.

While here cleanup and shrink code without changing any semantics.
Started with a diff posted on tech@ by Tobias Ulmer.

"I like it" marco@ ok jsing@

show more ...

Add checks to each write(2) attempt and try to keep fdisk in a
consistent state upon write failure.

OK krw@ and deraadt@.

Fix a typo, paritition -> partition.
From Alexey Dobriyan <adobriyan@gmail.com>, thanks.

man page is gzip'd, should store as unsigned char array.

Allow setting of partition flag with an optional explicit value. PR 4143.
ok weingart@ deraadr@

add a swap primitive; ok various, will be polished after by tom

gzip embedded man pages, and pipe through gunzip when viewing.
ok deraadt@

some delinting; ok otto

Remove unused argument 'int pn' to PRT_fix_CHS(). The 'int pn'
argument to PRT_parse() then becomes unused, so remove that too.

ok henning@, otto@

Make the interactive "update" command match the command-line "-u"
option by updatng/setting the 0xAA55 signature at the end of the sector.

ok nick@ weingart@

ansification

Nuke terms 3 & 4.

allow reinitializing mbr in the ext parts; from Matt Watson <mwatson@apple.com>