chain: A sprinkling of documentation

Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

chain: Remove type-assurance macro for declarations

A coding pattern from WinVBlock leaked through whereby a
macro is used to declare functions having the same
prototype.  This makes their declarations look a little less
like a traditional declarations however, so that pattern
has been removed.

Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

chain: Support GPT disks

A GPT disk partition iterator is introduced, so
we can chain-load partitions on such a disk.

Tested-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>
Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

chain: Change disk partition iteration strategy

In order to support more than just MBR and EBR walking, we
introduce the abstraction of a partition iterator.  Currently
there are just the two types: MBR partition iterator and
extended partition iterator.

Tested-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>
Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

chain: Fix allocation size in read_sectors

Oops.  We might allocate more than just one sector.

Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

chain: Add DOS partition dumping routine for debugging

Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

chain: Use struct mbr where applicable

Instead of magic offsets.

Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

chain: Use CHS typedef and macros

A DOS partition table entry contains cylinder, head, sector
tuples which can be convenient to group together and extract
with convenience macros.  Currently unused.

Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

chain: Allow reading more than a single sector

Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

chain: Run Nindent on com32/modules/chain.c

Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

fs: enforce filetypes in open_file

Enforce that the file type in open_file is a plain file.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

opendir: enforce the file type

Don't allow opendir() on a non-directory.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

pxe: set the file type in the inode structure

All "files" on PXE are regular files (there is no official directory
mechanism) and so, at least for now, it's all DT_REG.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

memdiskfind: remove debugging printf

Remove accidentally checked in debugging printf.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

memdiskfind: make sure we don't map unavailable RAM

Sometimes FBM is below the reserved memory limit, which means memory
that is available to protected-mode operating systems only and which
will be claimed by the kernel and therefore be unmappable.  Read
/proc/iomem to make sure the memory range is safe to map.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

mtools: create an empty ADV

Create an install an empty ADV.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

dos: vacuous ADV support

Vacuous ADV support: install an empty ADV.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

setadv: remove unused, nonportable header file

This file does not need <sys/types.h>.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

win32: vacuous ADV support

Install an empty ADV in the Windows installer to keep it from being
broken.  In order to do that, separate the Unix-specific ADV I/O
functions from the generic data structure manipulation.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

win32: use .obj not .o, and build an intermediate library

Use .obj as the extension, rather than .o, to match Windows
conventions.  Furthermore, build an intermediate library for the stuff
that we pick up from elsewhere; this makes it easier to build
additional Windows binaries with helper functions in libinstaller or
another source library.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

pxe: fix the timeout logic for RRQ

Fix the timeout logic for RRQ, to avoid a spew of identical RRQ
packets which at best will annoy the server.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

pxe: fix bugs in DHCP parsing and config file selection

Fix several buffer-handling bugs in DHCP parsing and in the config
file selection.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

pxe: correct the generation of the UUID string

The UUID string was generated incorrectly, starting over for each
component.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Merge remote branch 'erwan/master_new'

hdt: new release 0.3.6.1

Increase subrelease for syslinux4.
hdt just only received code cleaning, no new feature.

hdt: Using syslinux_reboot() call

Rebooting can be achieved via syslinux_reboot() instead of the useless reboot.c32 module.

hdt: updating memtest to 4.10

sysdump, be_srec: increment the address

Need to increment the address for each srec...

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

sysdump: output S-records to the console

Sometimes we have a console path that can be captured (virtual
machine, serial console, ...) but no realistic path for two-way or
binary communication.  Add an option to output S-records to the
console as an inefficient but hopefully reliable way to get data out.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Replacing strncpy by strlcpy

As reported by hpa, strlcpy is safer and could prevent some bugs.
So I replaced all strncpy by strlcpy calls.

Merge branch 'master' of git://git./boot/syslinux/syslinux into master_new

memdisk: add a sector_shift field to the MDI; fix memdiskfind

Add a sector_shift field to the MDI rather than assuming 512-byte
sectors.  Fix memdiskfind to report size in bytes, not in sectors.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

diskstart: if no partition table, we need Hidden == 0

If we don't have a partition table, we need to make sure Hidden gets
set to zero.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

isohybrid: fix another type mismatch

Fix another printf type mismatch in the C version of isohybrid.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

sysdump: smarter ACPI dump

Follow the XSDT as well as the RSDT, at least as long as we can do
that with 32-bit addresses.  Actually keep track of what we have
already dumped.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

sysdump: make TFTP server optional

Make entering the TFTP server optional (defaults to boot server).

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

<syslinux/config.h> fix the order of struct syslinux_ipinfo

Correct the order of the fields in struct syslinux_ipinfo to match
reality.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

memdiskfind: sanity check the FBM value

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

memdiskfind: utility to find an mBFT and output phram parameters

A trivial Linux utility to search for mBFT and output its parameters
in a form which can be used as input to the phram Linux kernel module.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

memdisk: make mstructs.h includable from Linux

Make mstructs.h includable from a Linux utility.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

utils/Makefile: fix build rules

Fix isohybrid build rules.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

isohybrid: fix beginner mistakes

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

core: Make cfarcall IF-preserving

cfarcall does not take a register image on input, so we need to
explicitly preserve IF in the code flow.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

com32: make __farcall() IF-preserving

Make __farcall preserve the current value of IF.  If we need to we can
create a __raw_farcall() operation which does not.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

linux installer: better error message for missing /tmp

Our linux distribution at my job didn't include a /tmp directory. When ever
I tried to use syslinux, I would get the following error:

./syslinux: No such file or directory

I modified the code to report this:

./syslinux: Cannot access the /tmp/ directory.

This would have helped me a lot rather than having to download the source
and run it through the debugger to figure out that my /tmp/ directory was
missing. I suspect if this ever happens again I'll know exactly what was
wrong, but hopefully it will help someone else.

comboot: allow the read disk API call to specify a 64-bit LBA

We already had two MBZ fields reserved, so use one of them to expand
into a 64-bit LBA.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

core, adv: for CHS mode, the top 40 LBA bits must be zero

The top 40 bits of the LBA must be zero on CHS mode.  Enforce this,
and also fix leak of the drive number into a divide instruction with
resulting overflow.

Reported-by: Gert Hulselmans <gerth@zytor.com>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

pxelinux: use alignb not alignz in a bss sections

In .bss sections use alignb, not alignz.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Fixing compilation warning

Reported by Genne Cumm, some more_printf calls generates gcc warning.

diskstart: fix CHS mode, reinstate cyl 1023 check

Fix loading in CHS mode; we were missing a popad.  Also reinstate the
check for exceeding cylinder 1023.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

diskstart: clean up GPT handling

We only need 56 bytes, not 92, and we should check for 0xED as the
partition type.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

dns: don't get stuck on no packet received

Make sure the timeout actually gets processed.  The right thing to do
for the receive loop is to check for timeout, then try to receive, and
loop until an acceptable packet has been received; we might as well do
that explicitly.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

core: Preserve IF through call16()

An intcall should always be invoked with interrupts off, but that is
not necessarily the case for a near or far call; in fact it is quite
the exception.  As such, do not filter IF in our register image, and
for our own internal call16() interface, propagate the protected-mode
IF value into real mode, just as we do for the pm_call interface.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

pxe: fix handling of lost packets in DNS resolution

When we have lost packets in DNS resolution, or otherwise no
response, both rotate through the known servers and advance through
the timeout table.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

pxe: properly null-terminate packet after tftp://

Properly null-terminate the output from a parsed tftp:// URL.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

pxe: correct the parsing of tftp:// URLs

Correct the parsing of tftp:// URLs.  DNS handling still needs to be
unbroken.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

pxe: don't misidentify non-tftp URLs as tftp

Fix reversed test for tftp URLs

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

core: align the EPA

Align the Extended Patch Area.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Merge syslinux/extlinux patch code and core code

Merge the SYSLINUX and EXTLINUX patching code and core code, removing
EXTLINUX as a separate derivative.  All the disk-based systems now use
the same code.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Reduce sector 1 space pressure; further merge installer code

Reduce sector 1 space pressure by moving objects that aren't needed by
Sector 1 proper into an "extended patch area".  While we're mucking
with the installer code, make the syslxint and extlinux installer code
even more similar.  It should now be pretty straightforward to
outright merge the code.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

fat: fix off-by-one error in the FAT installer

Fix off-by-one error in the FAT installer, which caused the install to
incorrectly abort without the patched version of the data.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

linux/syslinux: handle the null pathname case

Fix mishandling of the null pathname case in the syslinux installer,
and generally clean up the handling of the subpath name.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Merge remote branch 'sha0/for_hpa' into pathbased

Merge branch 'pathbased' of ssh://terminus.zytor.com/pub/git/syslinux/syslinux into pathbased

pxe: mark ipinfo as IPv4

IP information is IPv4 for now.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Merge branch 'pathbased' of ssh://terminus.zytor.com/pub/git/syslinux/syslinux into pathbased

core, pxe: remove stray debugging printf

Remove debugging printf added for testing.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

sysdump: dump ACPI information

Dump ACPI tables.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

sysdump: use lmalloc/lfree

Use lmalloc/lfree instead of using the fixed (obsolete) bounce buffer.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

sysdump: tftp: use lmalloc(), and set the gateway

Use lmalloc() rather than the fixed (obsolete) bounce buffer, and set
the gateway for PXE stacks that need it.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

pxe: centralize all the IP information and export it to modules

It appears that there still are PXE stacks in the field which needs
the crutch of being pointed to the default gateway.  As such, put all
the IP information into a single memory structure and allow modules to
see it.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Merge branch 'pathbased' of ssh://terminus.zytor.com/pub/git/syslinux/syslinux into pathbased

Merge remote branch 'sha0/boot_args' into pathbased

eltorito: Scan upwards instead of downwards

A developer for GRUB4DOS called tinybit has reported that some
BIOSes crash when the DOS ElTorito.Sys driver scans drive number
0xFF.  According to the El Torito specification, drive 0x7F
means "terminate all," so a plausible explanation suggested
by H. Peter Anvin is that the drive 0xFF is a shadow of drive
0x7F, thus contributing to a failure on these BIOSes.

Now we scan from 0x80 through 0xFF, instead.

[ hpa: dropped checkin of eltorito.sys ]

Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Merge remote branch 'sha0/mdiskchk_enh2' into pathbased

Merge branch 'pathbased' of ssh://terminus.zytor.com/pub/git/syslinux/syslinux into pathbased

Move Linux ioctl header magic into a single file

Put all the Linux ioctl header magic into a single shared file, and
try to make it as generally useful as possible.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

installer: fix use of FIEMAP

Correct the implementation of block mapping using FIEMAP (as opposed
to FIBMAP).

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

extlinux: don't compile with -O0

-O0 is good for debugging, but sucks for production.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

dos: fix get_64_sl() and set_64_sl()

We must make sure we actually use the modified pointer returned from
set_fs(), otherwise bad things happen...

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Merge branch 'pathbased' of ssh://terminus.zytor.com/pub/git/syslinux/syslinux into pathbased

Fix prototype for generate_extents

Add missing const

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

syslinux: synchronize generate_extents() with extlinux

Synchronize the code for generate_extents() from extlinux to syslinux.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

extlinux: remove debugging printf's

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Merge branch 'pathbased' of ssh://terminus.zytor.com/pub/git/syslinux/syslinux into pathbased

Switch to 64-bit sector pointers everywhere

Switch to consistent use of 64-bit sector pointers; this should enable
booting even for individual *partitions* larger than 2 TB.  In order
to not slow down the boot too much, switch the initial load from an
enumeration to an extent map.  This means the table gets larger (since
we have to assume the worst case), but it simplifies the Sector 1 code
(since we can push all the hard stuff into the installer), and will
speed up booting in the general case.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

mdiskchk: Add --no-sequential mode

It might be useful to suppress MDISKCHK.COM's classic behaviour
of probing all BIOS drive numbers in search of MEMDISKs.  Some BIOSes
might not enjoy being probed.

Reported-by: bylokk
Reported-by: Gert Hulselmans <gerth@zytor.com>
Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

memdisk: Correct El Torito termination response

Two changes to the INT 0x13, AH=0x7B El Torito function:

1. Support the DL=0x7F "terminate all" parameter
2. Only respond to DL=0x7F or DL=our particular emulated drive number

This prevents tools like Bart Lagerweij's El Torito tools from
receiving false positives when probing all drive numbers.

Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

eltorito: Include drive 0xFF in scan

Although drive 0xFF could yield a false positive as the El Torito
booted-from drive number, some users of GRUB4DOS actually use
this drive number, so we should include it in our scan.

Signed-off-by: Shao Miller <shao.miller@yrdsb.edu.on.ca>

Merge branch 'pathbased' of ssh://terminus.zytor.com/pub/git/syslinux/syslinux into pathbased

core: add hlt to infinite loop

We should never hit it, but if we do, don't burn the CPU to a crisp.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Implement !GPT protocol in Syslinux core; handle offset > 2 TB

Implement the !GPT handover protocol in the Syslinux core, and handle
partition offsets above 2 TB.  We do not yet handle filesystem sizes
above 2 TB, but that should be a reasonably straightforward extension
at this time (need to switch to 8-byte block pointers).

This finally meant moving getlinsec out of the boot sector, since it
no longer fits.  Instead have a very simple getonesec implementation
in the boot sector.  getlinsec still fits in Sector 1, although it
doesn't leave space for very many block pointers.  That's still better
than adding yet another boot loader stage, however.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

gptmbr: change synthetic partition type to 0xED

The committee didn't like re-using 0xEE for this purpose and wants
0xED instead.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

Merge remote branch 'sherbszt/gfxboot32' into pathbased

fat: handle .. pointing back to the root directory

.. pointing back to the root directory will have a cluster number of
0, even for FAT32 where there is an actual cluster number for the root
directory.  Handle this as a special case.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>

gfxboot: set media type

Set media type according to the drive number and cd mode.

Signed-off-by: Sebastian Herbszt <herbszt@gmx.de>

ifcpuXX: Support multiple parameters with labels

The ifcpu.c32 and ifcpu64.c32 modules take arguments including
the form <true_label> -- <false_label>.  It is convenient to
allow these labels to be not just a single word, but to include
parameters.  This is useful for <TAB>-editing and modifying the
command-line to include parameters you'd like to pass.

Not-yet-tested: Shao Miller <shao.miller@yrdsb.edu.on.ca>

gptmbr: save four bytes

Save four bytes by observing that none of our code relies on
saturate_stosl not actually corrupting %eax.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

gptmbr: fix offset for MSW of partition end

Fix the offset for the MSW of the partition endpoint.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

gptmbr: implement the new T13-approved GPT protocol

My GPT-based protocol was modified by the UEFI and T13 committees (the
former responsible for the GPT format, the latter for EDD and
therefore for the disk-related part of the BIOS specification.)  This
is thus now on its way to become an official protocol, so change the
implementation to match.

This still needs testing, and the Syslinux core needs to be adjusted
to leverage the extended information.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>