Age | Commit message (Collapse) | Author |
|
together three boolean values. In today's englightened world we
know about '||'.
Pointed out by naddy@
|
|
Suggestion, original diff and ok naddy@
|
|
so remove their references;
while here undo a predefined string (as mandoc_char(7) calls them);
|
|
Useful for seeing exactly what is currently on the disk. Suggested
by a request from kettenis@.
Man page & usage ok jmc@
|
|
print the disk's name in an error message.
|
|
any undesirable GPT hanging around. i.e. same as 'reinit' from the
fdisk(8) command line. Noticed now that install media relies on
'-i' rather than issuing 'reinit' in a script.
|
|
default; OK krw@
|
|
|
|
Suggested by & ok tim@
|
|
krw's refactorings which hoisted disk-opening to the top.
tested by krw
|
|
by presenting the existing offset and size as the defaults. Enhance
getuint64() to take a minimum value as ask_num() does. Use this to
ensure that GPT partitions are constrained to the valid area of the
disk. Leave MBR partition constraints alone for the moment.
Original problem(s) noted by tim@
|
|
and Xgedit() to a new mechanism. Keep a copy of the old partition
and compare final edited version to that to decide if it is dirty.
Fixes a number of (but not all!) nits in editing partitions.
Original problem(s) noted by, suggestions from, much testing and
ok tim@
|
|
table is marked dirty so that it gets written when "quit" is issued.
This commit contains the MBR part of the fix only. The GPT code needs a few
adjustments before the GPT part can be fixed properly.
OK krw@
|
|
Reduce overburden of unnecessary #include's. Note that param.h is
included only for DEV_BSIZE. Use INT64_MAX instead of LLONG_MAX
so stdint.h is enough. General tidying up of #include sections.
ok GCC
|
|
included only for DEV_BSIZE. Use INT64_MAX instead of LLONG_MAX
so stdint.h is enough. General tidying up of #include sections.
ok GCC
|
|
|
|
partition table in this variable when doing an 'u'pdate of the MBR.
Remove now superfluous MBR_pcopy() and its re-read of the on-disk MBR.
|
|
|
|
1) If mbr_file is NULL use built-in mbr for -i, -u and 'reinit'.
2) If mbr_file cannot be opened issue a warning and use built-in mbr for -i,
-u, and 'reinit'.
3) If mbr_file can't be read, bail out of fdisk.
4) Use the mbr read from mbr_file for -i, -u, and 'reinit'.
Remove inappropriate GPT dancing.
This restores pre-GPT-editing mbr_file handling and makes the logic
clearer at the expense of a tiny bit of duplication.
|
|
and re-read it from disk if appropriate. i.e. when the re-read MBR
is a protective MBR.
|
|
protective MBR.
|
|
from jan stary
|
|
returning a value to tell callers to zap it. Use consistant idiom
to check for the presence of a GPT (check gh.gh_sig). When zapping
GPT, always zap both the header and partition table just for
paranoia's sake.
|
|
invalid and mention the constraint in usage(). Modify the appropriate
field(s) of the global 'disk' as -l/-c/-h/-s are encountered. As a
result use the disklabel information to populate the global 'disk'
only when it has not already been populated. Move test for 'disk'
validity into DISK_open().
|
|
usage() verbiage a bit so it's clear all of '-c -h -s' are required
if any are used.
|
|
when none of -i, -e or -u are specified.
Prodded by deraadt@
|
|
MBR or GPT is printed. Do not attempt to read the MBR template file,
construct an initial mbr, etc.
|
|
-g needing -i. The usage() text is quite clear.
usage() doesn't return so eliminate unneeded 'else'.
|
|
|
|
verbiage in the usage() output.
|
|
and saving the fd in the global 'disk' structure. Stop passing around
fd's and just use the global.
Makes pledge() feasible.
Prompted by and ok deraadt@
|
|
before constructing default MBR.
Change last bzero() to memset().
|
|
the new GPT.
|
|
OK krw@
|
|
attempt to use data past the end of the input.
Since the return value of ask_cmd() is never checked, make it void instead
of int.
Problems found, original diff from and ok tim@.
|
|
'disk too large' message only when no GPT is found.
Problems found, fix tested and ok naddy@.
|
|
UINT32_MAX sectors. i.e. don't use the truncated value aligned to cylinder
size that we use for 'real' MBR partitions.
Makes GPT on large disks work.
Problem found & fix tested by naddy@.
|
|
in the GPT partition structure. And don't run off the end of the
name buffer by confusing sizeof() with the number of elements in
an array. Use the new GPTPARTNAMESIZE #define instead. While here,
zap the old partition name before setting the new one, lest a short
new name leave bits of an old long name in place.
Originally spotted by jsg@ and his friend cppcheck.
ok jsg@ for slightly different version.
|
|
|
|
|
|
|
|
Prompted by deraadt@.
|
|
|
|
|
|
MBR. If '-b' is specified an EFI System partition of the requested size is
created. All remaining space is put into an OpenBSD partition.
Minimal enhancement necessary for upcoming UEFI install support.
Committed first to flush out any unexpected impacts on 'normal' MBR
operation and install media.
ok deraadt@
|
|
is marked with '*'
due to comments from sthen and kettenis
|
|
with the simple facts a user needs.
ok kettenis krw
|
|
|
|
of SYNOPSIS and usage();
|
|
special boot partition on architectures that need it.
For now those architectures are amd64 and i386, where this option will
create an EFI system partition to store our new UEFI bootloader.
ok deraadt@, krw@
|