bios: Don't truncate memory size needed to 16 bits

We can't truncate the memory size needed to 16 bits *before* we
convert it to kilobytes...

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

bios: Fix lowmem check

The lowmem check was backwards and would report error when there
wasn't any.  This was missed because the Ctrl key check was *also*
backwards and so the check would only be done when Ctrl was held down.

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

Merge branch 'syslinux-4.xx' into syslinux-5.xx

isohybrid: Improve help/man

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

Add manpages for isohybrid, memdiskfind

Signed-off-by: Peter Jones <pjones@redhat.com>

com32/moduless/poweroff.c: use __unused attribute instead

Recommended-By: Matt Fleming <matt@console-pimps.org>
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

com32/modules/poweroff.c: fix main()

Wrong prototype; use GCC unused

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

ldlinux/readconfig.c: remove bail: from do_include()

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

Add const qualifiers to loadleys

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

localboot: Clear register image before using

Using a register image on the stack without initializing it is not a
good idea.

Reported-by: Erwan Velu <erwanaliasr1@gmail.com>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

Merge remote-tracking branch 'origin/syslinux-4.xx' into syslinux-5.xx

Resolved Conflicts:
version

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

Makefiles: don't include *.tmp

Since checkin:

bd09a6d828fa Major Makefile cleanups; gcc 4.3.0 compatiblity

... we include *.tmp into the Makefiles as well as .*.d.  This seems
to have been a mistake in made when adding *.tmp to cleanup rules,
probably using a sed script.

This causes problems, because *.tmp files are generated by the gcc_ok
macro and do not contain Makefile rules at all.

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

version: Update year to 2014

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

version: Update to 4.08, update year to 2014

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

Merge branch 'master' into elflink

Resolved Conflicts:
NEWS
core/fs/fs.c
version

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

diag/geodsp: README fixes

Should clarify the situation; also word-wrap & save example

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

diag/geodsp: fix Makefile

Results in null image

Reported-By: ioannis
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

core/legacynet: Enable dot quad resolution

in dnsresolv.c:dns_resolv()

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

Remove some whitespace

Signed-off-by: Ruben Kerkhof <ruben at rubenkerkhof.com>
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

isolinux: Clear upper half of EDX before using

In checkin:

    cb015497a4e4 isolinux: Update LBA in getlinsec loop

... we use EDX as a sector count, but the sector count is actually in
DX, and the upper half of EDX is uninitialized.  If the BIOS enters
with a nonzero value in the upper half of EDX, this breaks horribly.

At least one set of BIOSes has been identified where if the LBA > 64K
then the upper half of EDX will be nonzero.

Reported-by: Carl Duff <cdrw2400@gmail.com>
Reported-by: Philip Müller <philm@manjaro.org>
Tested-by: Gerardo Exequiel Pozzi <vmlinuz386@yahoo.com.ar>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

pxe, efi: Bad read of file size over TFTP on EFI64

A fancy pointers logic has been replaced with a plain old if / else
branches. It was assigning only half of a 64 bits integer which is then
assigned to a size_t. Thus leading to a bug on platform where size_t is
64 bits.

Resolves bug #26

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

isolinux: Clear upper half of EDX before using

In checkin:

    cb015497a4e4 isolinux: Update LBA in getlinsec loop

... we use EDX as a sector count, but the sector count is actually in
DX, and the upper half of EDX is uninitialized.  If the BIOS enters
with a nonzero value in the upper half of EDX, this breaks horribly.

At least one set of BIOSes has been identified where if the LBA > 64K
then the upper half of EDX will be nonzero.

Reported-by: Carl Duff <cdrw2400@gmail.com>
Reported-by: Philip Müller <philm@manjaro.org>
Tested-by: Gerardo Exequiel Pozzi <vmlinuz386@yahoo.com.ar>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

com32: Fix a bug on history of commands.

Previously, even zero-length commands would be added to the history when
they shoudn't, e.g: just typing enter.

For example, if you type: FOO -> (ENTER) -> (ENTER),
then to get FOO from the history you would have to press the UP key
twice. It also saves a bit of memory.

Signed-off-by: Raphael S.Carvalho <raphael.scarv@gmail.com>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>

com32: Fix bugs on cmd_reverse_search (Triple fault dimension)

cmd_reverse_search has a bug that the variable cursor is updated even if a command
wasn't found. If this happens, and the next key falls into the default case,
memmove's size parameter would be a negative number.

This bug can be reproduced by doing the following:
On cmd_reverse_search (ctrl-r), type multiple keys at the same time.
'Enjoy' the triple fault and a screen of random colors.

There is also a small bug that turns the task of using (ctrl-r) on the first command
impossible. Previously, this command was discarded.

Signed-off-by: Raphael S.Carvalho <raphael.scarv@gmail.com>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>

com32/disk: Code cleanup at disk_write_sectors and disk_read_sectors.

Pulled common code out of these functions into new ones.
The functions chs_setup and ebios_setup were created for this purpose.

Signed-off-by: Raphael S.Carvalho <raphael.scarv@gmail.com>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>

PXE ISR: Force polling on select hardware WORKAROUND

By OUI == 00:23:ae and flags == 0xdc1b, detect select hardware.

On select platforms (Dell OptiPlex 760, Dell OptiPlex 960; perhaps
more), the interrupt appears to go "deaf" after a few seconds.  By
matching MAC OUI and flags value, force polling on these select
platforms.  I'm not sure if there's any better data available that
shallow in the core.  I believe PCI IDs can be fetched with functions
from other libraries and the UUID and DMI data (the most likely to be
useful is SYSPRODUCT) is available in ldlinux.c32.

Commit message expanded with Matt Fleming's assistance

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

core/lwip: Fix NULL pointer check

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

PXE: use ddprintf macro

Messages are needed before ldlinux.c32 is loaded

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

com32: Define ddprintf() macro

Duplicating Debug Printf;  Certain warning/error printf()
 statements are not visible if they occur before ldlinux.c32
 is loaded

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

PXELINUX: specify PXE/lwIP

Styled after SYSLINUX

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

undiif: show thread of execution on UNDIIF_ID_DEBUG

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

core: dprintf() the banner.

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

PXELINUX: Use sendto() instead of connect()/send()/disconnect()

This commit prevents a race-condition on systems that have functional
interrupts (observed with iPXE and select other Dell systems).  Without
this, the reply packet could be received by the core prior to the
disconnect() call, see that it doesn't have a matching PCB (protocol
control block, iirc) since the reply has a different far-end UDP port
than the original request, and lwIP will discard the packet before
PXELINUX can see it.

net_core_sendto() instead of
net_core_connect() net_core_send() net_core_disconnect()

Commit message expanded with Matt Fleming's assistance

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

core: make mbox_post()/__sem_down_slow() check if valid

If the mailbox/semaphore is invalid, disallow additions.

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

core: mbox/semaphore NULL checks

Also set mbox invalid and mbox pointer NULL when free()d

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

core/lwip/undi: Improve UNDIIF_ID_DEBUG messages

Remove extra space; add TCP flags

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

NEWS: document changes in 4.07

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

Allow for UNDIIF_ID_DEBUG in 1 mk line

Add comments and 1 line to mk/devel.mk to debug headers of all
packets sent/recieved through undiif.c.

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

core/lwip/undiif: dprint MAC/ARP/IP/ICMP/TCP/UDP headers; poll debug

Encapsulate in #ifdef; use snprintf and 1 dprintf-like statement.
Print debug message that polling has been forced

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

lwip/undiif: split arphdr._hwlen_protolen

lwIP already split this in etharp.h

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

lwip: Allow LWIP_PLATFORM_* output to use dprintf

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

lwip: undiif: Fix debug options to unique *_DEBUG macros

Don't reuse *_DEBUG macros intended for other source files.

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

version: bump version and date

Welcome to the 4.07 release cycle

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

Merge branch 'dynamic-debug' into elflink

mk: enable dynamic debug support

This will probably be disabled for releases, but it's definitely
worthwhile having enabled for all prereleases.

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

debug.c32: Dynamically enable/disable debug code at runtime

It's useful to be able to enable (and disable) debug code at runtime,
particularly for allowing users that are unable to build their own
Syslinux releases the chance to provide useful debugging output.

For example, say a user reports trouble with their PXE stack but doesn't
have a development environment setup to turn on the debug code
themselves. With this change you can simply request that they do,

    debug.c32 -e pxe_call unload_pxe open_file

to enable the debug in those functions. By only turning on code in
specific functions we reduce the chance of disrupting the buggy
behaviour and improve the signal to noise ratio for print statements.

To disable debug code use the -d flag,

    debug.c32 -d pxe_call

To use this new feature simply do,

    if (syslinux_debug_enabled) {
        debug1();
        debug2();
        ....
    }

from within the function you wish to add debug code. Note that this
feature is not limited to print statements - you can put any code within
the conditional, such as verifying a checksum or checking for memory
leaks.

The plan is to leave the dynamic debug code built in for all prereleases
and to turn it off for final releases.

People may still want to build with all dprintf() statements enabled,
and so a new symbol, DYNAMIC_DEBUG, was introduced rather than
repurposing the old DEBUG, DEBUG_STDIO and DEBUG_PORT symbols.

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

mk: include $(GCCWARN) when building ELF objects

It's currently impossible to turn on dprintf() statements in
com32/modules because $(GCCWARN) isn't used as part of $(CFLAGS).

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

Merge branch 'idle' into elflink

idle: Consolidate idle implementations

Duplicate code is bad. Move all the idle code to C and delete the old
assembly stuff.

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

idle: enable interrupts before idling

Users are *still* reporting executing __idle() with interrupts disabled,
which ultimately leads to a hang. Just enable them explicitly before
idling.

Cc: H. Peter Anvin <hpa@zytor.com>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>

Merge branch 'doc-for-mfleming' of git://github.com/geneC/syslinux into elflink

Pull doc/ and txt/ spelling corrections from Gene Cumm,

* 'doc-for-mfleming' of git://github.com/geneC/syslinux:
  doc/ & txt/: Spelling fix

core: we only need one reset_idle()

We don't need both an assembly version of reset_idle and an
implementation in C. Having these two functions has led to a bug where
we may idle with interrupts disabled because an 'sti' is only performed
in the assembly version. This lead to a hang waiting for user input.

Cc: H. Peter Anvin <hpa@zytor.com>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>

doc/ & txt/: Spelling fix

Also add octal note.

Reported-By: Ady <ady-sf@hotmail.com>
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

load_linux: validate sanity of 'initrd_addr_max'

Some kernel headers have bogus 'initrd_addr_max' fields. This field
should never be zero. Set it to the old upper limit if unspecified.
Failure to set a non-zero value for the field results in a bogus
'memlimit' value, thereby unnecessarily reserving part of the memmap.

Cc: H. Peter Anvin <hpa@zytor.com>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>

load_linux: Add dprintf statements to aid debugging

It's useful to know why a kernel is failing to load. Sprinkle some
dprintf() statements with informative messages.

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

conio: Fix pollchar() for serial console

The check of the LSR value was inverted. This resulted in pollchar()
always claiming that there was data to be read from the serial console.

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

ldlinux: fix INCLUDE regression

We don't want to overwrite 'ConfigName' when parsing a config file via
the INCLUDE directive, which commit 5447ef821 ("ldlinux: Always update
ConfigName when opening a config file") failed to take into account. In
the INCLUDE case we're only parsing config fragments, and not a main
config file.

Rename parse_one_config() to parse_main_config() to more accurately
reflect when it should be invoked (i.e. not for INCLUDE).

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

PXELINUX: implement our own version of local_boot16

We need to do different things for PXE, such as reset the PXE
environment when booting from the local disk from PXELINUX.

This fixes a localboot regression.

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

Merge branch 'ipappend-fix-for-mfleming' of git://github.com/geneC/syslinux into elflink

Pull patches to standardize and document IPAPPEND/SYSAPPEND from Gene Cumm,

* 'ipappend-fix-for-mfleming' of git://github.com/geneC/syslinux:
  doc/ & txt/: Document IPAPPEND/SYSAPPEND format
  core & menu: fix IPAPPEND/SYSAPPEND conversion

doc/ & txt/: Document IPAPPEND/SYSAPPEND format

Previously the maximum value was 7 which didn't require
differentiation between decimal and hexadecimal.

core & menu: fix IPAPPEND/SYSAPPEND conversion

Make both use the same functions.

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

core: remove unused header file vkernel.inc

vkernel.inc is related to the old assembly-based config file parser;
it is no longer relevant.

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

Merge branch 'menu-ipappend-1-for-mfleming' of git://github.com/geneC/syslinux into elflink

Pull a fix for the SYSAPPEND directive from Gene Cumm,

* 'menu-ipappend-1-for-mfleming' of git://github.com/geneC/syslinux:
  menu.c32: Fix SYSAPPEND

Merge branch 'doc-for-mfleming' of git://github.com/geneC/syslinux into elflink

Pull txt/ and doc/ updates from Gene Cumm,

* 'doc-for-mfleming' of git://github.com/geneC/syslinux:
  txt/syslinux.txt: rewrap long command
  txt/Makefile: add isolinux.txt, pxelinux.txt
  txt/: Add isolinux.txt, pxelinux.txt
  txt/syslinux-cli.txt: Version on Ctrl-N
  txt/Makefile: order-only prerequisite
  txt/: Add common file for derivatives
  txt/syslinux.cfg.txt: Updates
  txt/syslinux.txt: synopsis, extlinux.sys, wrap long command
  txt/syslinux-cli.txt: Path rules
  txt/syslinux.cfg.txt: Add SENDCOOKIES, example config
  txt/syslinux.cfg.txt: Add SYSAPPEND
  doc/syslinux.txt: grammar
  Fix SERIAL directive in docs

menu.c32: Fix SYSAPPEND

Using the SYSAPPEND directive would examine "D" instead of the
value after the directive.  Also allows a 1-step switch to strtol()

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

txt/syslinux.txt: rewrap long command

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

txt/Makefile: add isolinux.txt, pxelinux.txt

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

txt/: Add isolinux.txt, pxelinux.txt

Reformatted from previous in doc/.  Reflow to try to be more
manpage-like and put more commonly used information on the top.

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

txt/syslinux-cli.txt: Version on Ctrl-N

Only on some versions.

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

cli: don't disable linewrap when redrawing the cmdline

It messes things up when we want to use printf(), for example when
printing labels after the user has hit the TAB key.

This fixes a bug where labels would be printed on a single line on the
screen (with some being displayed off screen) when they should have been
printed across multiple lines.

I can't think of a reason to disable linerap when redrawing the command
line - so let's not.

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

ldlinux: fix loading of kbdmap files

Use the canonical code sequence for opening the kbdmap file instead of
referencing KernelName, which presumably is a remanent of the old
assembly code.

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

txt/Makefile: order-only prerequisite

No need to rebuild when the directory's timestamp is updated.

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

PATH: use a linked list internally

In retrospect, choosing the colon character as the entry separator for
the PATH directive was not a smart move, as that character is also used
in TFTP-style paths. This conflict manifests as PXELINUX being unable to
find and load files.

An example dnsmasq log looks like,

dnsmasq-tftp: sent /arch/boot/syslinux/lpxelinux.0 to 192.168.0.90
dnsmasq-tftp: file /arch/ldlinux.c32 not found
dnsmasq-tftp: file /arch//ldlinux.c32 not found
dnsmasq-tftp: file /arch//boot/isolinux/ldlinux.c32 not found
dnsmasq-tftp: file /arch//isolinux/ldlinux.c32 not found
dnsmasq-tftp: file /arch//boot/syslinuxldlinux.c32 not found
dnsmasq-tftp: sent /arch//boot/syslinux/ldlinux.c32 to 192.168.0.90
dnsmasq-tftp: error 0 No error, file close received from 192.168.0.90
dnsmasq-tftp: failed sending /arch//boot/syslinux/ldlinux.c32 to 192.168.0.90
dnsmasq-tftp: sent /arch/boot/syslinux/archiso.cfg to 192.168.0.90
dnsmasq-tftp: sent /arch/boot/syslinux/whichsys.c32 to 192.168.0.90
dnsmasq-tftp: file /arch/libcom32.c32 not found
dnsmasq-tftp: file /arch//libcom32.c32 not found
dnsmasq-tftp: file /arch/libcom32.c32 not found
dnsmasq-tftp: file /arch//arch//boot/syslinux/libcom32.c32 not found

The last line of the log is the indication that there's a problem.
Internally, Syslinux adds the location of ldlinux.c32 to PATH by
querying the current working directory once ldlinux.c32 is successfully
loaded. Under PXELINUX that means the initial PATH string will be,

   "::/arch/boot/syslinux/"

The PATH parsing code doesn't know how to correctly parse the "::"
string and hence, the file is searched for relative to the 210 dhcp
option directory - /arch/.

Implement PATH with a linked list which *greatly* simplifies the path
code, and means we no longer have to parse strings backwards and
forwards.

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

txt/: Add common file for derivatives

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

txt/syslinux.cfg.txt: Updates

DEFAULT often references a LABEL; TIMEOUT action affected by UI.
IPAPPEND/SYSAPPEND: wrap SYSAPPEND; notes on mask values 2,1.
TIMEOUT-related clarifications
SERIAL is sticky
PATH updated for 5.10-next with notes

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

txt/syslinux.txt: synopsis, extlinux.sys, wrap long command

Add -v/--version to synopsis; add note on extlinux.sys from old
versions; Wrap altmbr command for cleaner manpage

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

txt/syslinux-cli.txt: Path rules

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

version: Welcome to the 5.11 release cycle

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

ldlinux: fix serial port handling regression

commit c4fa33189f1d ("serial: Clean up and abstract handling of serial
ports") removed an assignment to 'port' in the config parsing code which
is used to setup the serial ports. Restore the assignment.

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

txt/syslinux.cfg.txt: Add SENDCOOKIES, example config

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

txt/syslinux.cfg.txt: Add SYSAPPEND

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

doc/syslinux.txt: grammar

Reported-by: Ady <ady-sf@hotmail.com>
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

Fix SERIAL directive in docs

Signed-off-by: Gene Cumm <gene.cumm@gmail.com>

NEWS: Mention noteworthy changes in 5.10

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

NEWS: Clarify network stack files

Point users in the right direction when looking for the new and legacy
network stacks.

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

font: write to 'fontbuf', not random memory

commit be5a345d385d ("font: load data as a single block") cleaned up the
loop for reading into the font buffer but made the mistake of dropping
the assignment to 'p', which means that the _fread() call is writing
through a garbage pointer. Since 'p' was only used to keep track while
writing into the font buffer in a loop, we can now delete it and
reference 'fontbuf' directly.

Also delete the unused variable 'i'.

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

serial: Clean up and abstract handling of serial ports

The special handling of serial ports 0-3 meaning "look in a BIOS
table" is at least officially BIOS-specific, so create an inline
function and move it to bios.h.

While we are at it, make the function look slightly less like
converted assembly.

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

font: load data as a single block

The only reason for a loop is to interleave data, which we don't need
anymore; instead the data is stored in packed form.

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

serial: fix SERIAL directive for ports > 0

There is a bug in the SERIAL directive parsing code that means that
using a port other than the first (SERIAL 0) results in a non-working
serial console. We need to use the serial port number from the config
file to offset the BIOS I/O port base address, otherwise we always
lookup the I/O port address for the first serial port instead of the one
the user specified.

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

font: increment pointer to fontbuf by font height

Instead of always assuming that the font height is 32 bytes, use
hdr.height to increment our fontbuf pointer. hdr.height is the size of
the chunks we read from the .psf file.

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

linux: Clear up error message

Passing opt.device to perror() in do_open_file() results in confusing
error messages, as Jack reports,

  Last night, I was trying to install syslinux (5.01) to a SD card of
  mine, writing to a subdirectory to keep the layout neat:

  syslinux-5.01/linux# ./syslinux -i -d /boot /dev/mmcblk0p1
  /dev/mmcblk0p1: No such file or directory

  The message is coming from do_open_file() in syslinux.c: the call to
  open() fails because the containing directory isn't present, and then it
  calls perror(opt.device).

We should instead be passing 'name' to perror() so that we know _which_
file doesn't exist.

Reported-by: Jack Kelly <jack@jackkelly.name>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>

load_linux: update errno before returning

syslinux_boot_linux() only returns control to the caller when something
has gone wrong, under normal circumstances a kernel is booted.

Update errno accordingly in the error paths, so the user no longer sees
inaccurate errno values which result in the following kinds of messages,

    "Booting kernel failed: Bad file descriptor"

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

Fix support for Linux kernel images with no protected mode code

Some kernel images use the Linux kernel boot protocol and header
structure, but do not actually have any protected-mode code.  For
instance, grub's 1024-byte lnxboot.img consists of 1024 real-mode bytes
and 0 protected-mode bytes; you can concatenate it with a full grub
core.img to produce a self-contained bootable kernel, but you can also
use it standalone as the kernel with the core.img loaded as an initrd.

syslinux 4 supports this, but it no longer works with syslinux 5: the
memmap functions do not correctly handle a request to work with a 0-byte
region.  With lnxboot.img, this would cause syslinux to bail because it
thinks it has no space at the 1M load location but cannot relocate
lnxboot.img.  (In bailing, it gives the confusing error message "Bad
file descriptor", not because that error actually occurred when
attempting to boot the kernel, but because errno has that value on entry
to syslinux_boot_linux and nothing clears or sets it.)

Fix the regression by handling the corner case of no protected-mode code
explicitly.

Signed-off-by: Josh Triplett <josh@joshtriplett.org>
Signed-off-by: Burt Triplett <burt@pbjtriplett.org>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>

pxe: disentangle the legacy and lwip stacks without #ifdef

Disentangle the dependencies via the legacy and the lwip stacks.  Now
there are no more #ifdefs (except for a single one in pxelinux.asm)
and the only difference between the pxelinux and lpxelinux archives
are which files are included.

To make this happen:

1. Replace the private part of the socket structure with a union.
2. Move the list of URL methods into a stack-dependent file.

Reported-by: Josh Triplett <josh@joshtriplett.org>
Reviewed-by: Josh Triplett <josh@joshtriplett.org>
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>

pxe: Fix compiler warnings and real bug

commit 00a6f13139 ("Partially revert "pxe: Pass absolute path to
pxe_chdir()"") introduced a bug by comparing the function 'url_type'
with URL_SUFFIX. This resulted in the following warning,

fs/pxe/pxe.c: In function ‘pxe_chdir’:
fs/pxe/pxe.c:460:18: warning: comparison between pointer and integer [enabled by default]
fs/pxe/pxe.c:458:19: warning: unused variable ‘path_type’ [-Wunused-variable]
fs/pxe/pxe.c: In function ‘pxe_chdir’:
fs/pxe/pxe.c:460:18: warning: comparison between pointer and integer [enabled by default]
fs/pxe/pxe.c:458:19: warning: unused variable ‘path_type’ [-Wunused-variable]

which unfortunately went unnoticed.

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

ldlinux: Fixup lwip merge botch

The lwip merge changed the code for copying cmdline arguments from,

    strcpy(q, p);

to this,

    do {
*q++ = ch = *p++;
    } while (ch);

which means 'p' no longer points at 'args'. Use 'args' explicitly for
the case where we need to apply a filename extension, otherwise users
end up seeing errors like,

    Failed to load COM32 file .c32

Reported-by: Mattias Schlenker <ms@mattiasschlenker.de>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>

Partially revert "pxe: Pass absolute path to pxe_chdir()"

This partially reverts commit b208ba467f678ed8e73f8d11fc0609634120cb83.

It isn't correct to always pass URL_OLD_TFTP to chdir(), since the
path prefix option may contain a full url, e.g. http://

Specialise the one case where we need to build a ::-style TFTP path
inside of get_prefix() to maintain backwards compatability, since from
Syslinux 5.00 onwards we may move out of the root directory ("::")
while searching for ldlinux.c32.

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

readconfig: Don't display "(null)" labels when hitting TAB

It is entirely legitimate for a menu entry to have a NULL label or
displayname. Since these entries are not used for execution, skip
displaying them in print_labels() when hitting the TAB key.

Reported-by: Bernd Blaauw <bblaauw@home.nl>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>

ldlinux: Always update ConfigName when opening a config file

With the introduction of commit aa7dd29db684 ("ldlinux: Pass config
filename as argv[1] to ldlinux.c32") we stopped using ConfigName to
store the filename of the config file we intend to open in execute()
since it interfered with the filesystem-specific open_config()
implementations. But a side-effect of this change is that
syslinux_config_file() no longer works if we explicitly pass a
filename to parse_config_file().

The logical place to keep ConfigName in sync is in parse_config_file()
since it's expected that all the filesystem-specific open_config()
functions will fill out ConfigName for us. In the case where the
filename is supplied by the caller, we need to fill it out ourselves.

Signed-off-by: Matt Fleming <matt.fleming@intel.com>

Merge branch 'lwip-merge' into elflink