dmi: fix more pointer subtraction bugs

Two more places with bogus pointer subtraction...

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

dmi: fix incorrect pointer subtraction, add sanity checks

We're dealing with BIOS-provided data, so we better do sanity checks
all over the place.

More immediately, fix a pointer-subtraction bug which would hang the
system.

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

dmi: add additional strings

Add additional strings, and create fallbacks when the strings are
empty.  Unfortunately we can't easily fall back for crap like "Insert
name of OEM here".

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

menu: need to advance the pointer, once again...

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

pxe, http: Use Syslinux/<version> as the User-agent:

Use Syslinux as the User-agent name.

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

dmi: strip whitespace from DMI strings

Some BIOS vendors seem to have large spaces in DMI strings, presumably
for easy patching.  Therefore, clean up the strings before we use
them.

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

pxe, http: correct the header overflow test

sizeof header_buf isn't very useful when that's an actual pointer...

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

pxe, undiif: install the gateway properly

Actually install the gateway, not a random pointer value...

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

menu: mangle whitespace in ipappend/sysappend strings

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

Additional sysappend strings from DMI; pre-bake the http cookies

- Add additional sysappend strings from DMI; we may want to add even
  more but let's think about it first.
- Pre-generate http cookies.
- Add a "sendcookies" command to mask out some of the information.

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

Merge remote-tracking branch 'origin/master' into lwip

hdt: add back -I$(com32)

hdt picks up include files from outside the include directory and so
needs an explicit -I$(com32).  This probably should be fixed.

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

sysdump: add $(LIBS) to the link rule

$(C_LIBS) doesn't include $(LIBS) so we need to add it explicitly.

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

Merge remote-tracking branch 'mfleming/for-hpa/makefile-cleanup'

Resolved Conflicts:
com32/hdt/Makefile
com32/sysdump/Makefile

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

pxe, http: send the sysappend/ipappend strings as cookies

When using http, send the sysappend/ipappend strings as cookies
prefixed with _Syslinux_ ... the implementation of this may change.
This is using the "old" cookie format for compactness (single header,
for all cookies, and no $Version tag.)  PHP at least seems perfectly
happy to deal with it.

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

Generalize ipappend handling as "sysappend", and move to PM code

Generalize the ipappend handling to cover all the derivatives, and
rename it "sysappend" ("ipappend" is a valid alias for all
derivatives.)  Move all the string handling to protected mode.
Currently only pxelinux exports strings, but the plan is to change
that in the future.

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

Merge remote-tracking branch 'origin/master' into lwip

Resolved Conflicts:
NEWS

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

isolinux: remove broken and obsolete open_file_t

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

isolinux: remove .img file support

Remove disk image support from ISOLINUX.  Very few BIOSes support this
call correctly, and it is likely to have been broken for a very long
time anyway.

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

pxe: remove non-lwIP definition of DNS_MAX_SERVERS

There is only need for one definition of DNS_MAX_SERVERS, and it needs
to be the one in lwIP, so use it everywhere.

Furthermore, there is no longer any need for us to define
DNS_MAX_PACKET.

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

Merge remote-tracking branch 'origin/master' into lwip

Resolved Conflicts:
com32/include/netinet/in.h

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

com32: inet_ntoa() takes struct in_addr

The standard definition for inet_ntoa() is to take struct in_addr, and
not doing that causes a conflict on the lwip branch.

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

hdt: remove set-but-unused variable

Remove a set but unused variable in HDT, which prevents building with
gcc 4.6.  This is probably a bug; presumably the active flag should be
presented to the user, too.

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

core: remove the .hugebss section

There are no more users of the .hugebss section, so remove it from
existence.

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

core, diskio: allocate the disk cache from the heap

Since network buffers are allocated from the heap, and the heap is now
dynamically sized, it makes sense we allocate the disk cache here,
too.

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

core: dynamically size the heap

Dynamically size the heap to 1/16 of available high memory.

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

core: layout.inc: remove duplicate .stack segment definition

.stack was redefined for unknown reasons.  This is not present in the
master branch.

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

Merge remote-tracking branch 'sherbszt/gfxboot32'

gfxboot: pass current working dir to gfxboot core

Pass current working dir to (and back from) gfxboot core via gfx_config
struct. This way I can implement a chdir() in the gfxboot core without
the need for additional 16-bit comboot calls. gfxboot may change the
directory, so you have to read and apply the current setting in boot() (to
read kernel etc. from right location).

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

Merge remote-tracking branch 'hdt/master'

diag/geodsp: zero-fill sectors in -1 mode

In -1 mode, zero-fill the sector instead of writing whatever garbage
was in the buffer already.

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

diag/geodsp: fix ordering of args to fread/fwrite in mk-lba-img

If we want the return value to be in bytes, make sure we pass the byte
count in the nmemb field instead of the size field.

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

diag/geodsp: Avoid large intermediate files

Allow the generation program mk-lba-img to merge the prefix file and
output to stdout, so it can be fed directly to the compressor; that
way we avoid generating huge intermediate files on disk.

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

ppmtolss16: remove obsolete Perl construct

defined() on an array is considered obsolete; instead check for a
nonzero number of elements with scalar().

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

PXE Cleanups, allow for 128 open files

Clean up dead code in the PXE stack.

The buffer assignment no longer limits the number of open files
either, so raise it to something more than reasonable.

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

NEWS: document FTP

pxe: use a separate poll thread instead of using an idle thread hook

The idle thread can never sleep, so it's not really safe to do
anything inside it.  Instead, run a separate poll thread at low
priority; we can also do that to poll the serial console if needed.

Overall, the "classic" Syslinux idle handling really should go away
and be replaced by the idle thread.

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

idle.c: remove -*- fundamental -*- tag

This is a normal C file, no reason for a -*- fundamental -*- tag as
used in NASM source code.

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

pxe, tftp: unbreak the TFTP state machine

Some of the code motion had broken the TFTP state machine, make it
work properly again.  Furthermore, remove some no longer necessary
fields in the pxe inode structure.

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

pxe, tftp: let lwIP manage port numbers

lwIP needs to manage port numbers for TCP and for DNS, so just let it
do it for TFTP as well.

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

pxe, ftp: remove a bogus include

Remove an include of a header file which never actually existed.

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

pxe: make lwip use IANA-blessed local port numbers

Make lwip use the IANA-blessed local port numbers 49152 and higher.

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

pxe: allow an empty input to dns_resolv()

If dns_resolv() gets an empty input, just return failure.

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

pxe: continue to bounce URLs with unknown schemes to gPXE

If we're running on top of a gPXE/iPXE stack, and get a URL we don't
know, continue to bounce it to gPXE.  It isn't entirely clear how well
this will actually work with an UNDI-based network stack, however...

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

pxe: make url_set_ip() static

url_set_ip() is now only used in one place, so make it static.

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

pxe: move redirect processing to the URL-parsing level

Move redirect processing out of the HTTP code and into the top
URL-processing level.  This both makes the code simpler and also deals
with absurdities like HTTP redirecting to an FTP URL (which is legal.)

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

pxe: add support for FTP

Add support for the FTP protocol.

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

pxe: tcp: close the connection before freeing the buffers

As long as the connection is opened, we may end up receiving data, so
don't free the receive buffers until after the connection is closed.

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

pxe: more verbose lwIP asserts

If we're going to bother compiling in the lwIP asserts, then make them
a bit more useful in terms of the message delivered.  This may be
worth tweaking further to make a smaller footprint.

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

urlparse: fix url_unescape()

Fix double pointer advacing in url_unescape().

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

pxe: tcp: always call the close method

Always call the designated close method, instead of assuming it is
tcp_close_file().  This isn't the case for FTP, which also needs to
manage the control connection.

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

gfxboot: handle INITRD config line

Handle case where there is no initrd= option but a separate initrd config line.

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

pxe: make tftp_pktbuf a dynamic buffer

We don't need tftp_pktbuf for the TCP-based protocols, so allocate it
on demand.  It should be possible to get rid of it for TFTP as well.

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

pxe: move operations common to TCP-based protocols to a common file

Move operations that are common to all TCP-based protocols into a
common file, tcp.c.

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

pxe: use the unified URL parsing framework

Use the unified URL parsing framework for TFTP and HTTP.  This should
also make it easier to add new protocols (e.g. FTP) in the near
future.

Note that HTTP redirects are still handled wrong: they really should
be sent all the way back to the top of URL parsing; there are sites in
the field which redirect to FTP URLs, for example.

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

pxe: urlparse: add comments

Add comments to the URL parser functions.

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

pxe: Add a much more general URL-parsing framework

Add a URL-parsing framework that we can use for multiple protocols.

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

thread: free curr->rmstack

Free curr->rmstack instead of double free of curr->stack.

Signed-off-by: Sebastian Herbszt <herbszt@gmx.de>
Signed-off-by: H. Peter Anvin <hpa@zytor.com>

pxe: put a semaphore around PXE calls

Make sure we can't accidentally invoke the PXE stack from multiple
threads (except for the ISR, which presumably has to allow for
reentrancy, since the spec implies the ISR can be invoked from inside
the PXE stack strategy routine.)

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

thread: free(NULL) is safe, lmalloc() always return para-aligned

free(NULL) is safe and permitted, no reason to conditionalize calling
free().

lmalloc() will always return a paragraph-aligned memory block, so we
don't have to play games with the offset.

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

lwip: handle UNDI stacks which need to be polled

If the UNDI stack reports either IRQ 0 or does NOT report the NDIS IRQ
supported flag, then poll the interrupt routine from the idle thread
instead.

This is somewhat limited; we really should have a chain of idle poll
routines to support things like serial console.

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

thread: thread-switch the real-mode stack, too

When this code was originally written, we didn't have lmalloc().  Now
when lmalloc() is implemented, let each real-mode task have its own
stack.

Note that this means we absolutely have to continue to support the
SS != CS, DS model in the real-mode code, which should already be the
case, but...

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

NEWS: document lwIP

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

version: call this branch 4.10

A totally different network stack is definitely grounds for calling it
4.10 instead of 4.0x...

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

Merge branch 'master' into lwip-threads

version: now working on version 4.05

4.04 is out, next version will be 4.05 (or later).

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

pxe: isr: fix formatting

Fix code formatting.

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

pxe: fix gcc 4.6 problems

Remove unused variables, which give gcc 4.6 problems.

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

hdt: Final release 0.5.0

Thanks for all who participe in this release.

hdt: Using dashed instead of . into label name

When generating floppy images some version of mtools (< 4.0.14) are
buggy and generate incorrect labels. So using > 4.0.13 is recommended.

Anyway, let's use _ into the volume name, that cleaner :

We now have
  disk label="HDT_0_5_0_B"
instead of
  disk label="HDT_0~5~0-B"

hdt: Fixing ACPI display

Some screenshots shows that we have some troubles for aligning items.
This is now fixed.

hdt: New release beta5

libupload: Adding tftp.h missing file

Oups, this while wasn't commited :/

hdt: Showing specific message for auto mode

That's a nice info to know that some output were generated by the
automatic mode.

hdt: Removing useless printf in dump mode

This message was clearly boring to read ;o)

libupload: Fixing tftp return value

The default return value for the tftp mode.
This made the tftp to return error even if the upload was succesful.

hdt: New release Beta4

hdt: Adding dump mode on pre-built images

As we now have a dump mode, let's add an entry in our menu for that.

hdt: Preventing dump mode if no PXE config

Dump mode can only work if PXE is present.

Merge commit 'syslinux-4.04' into dynamic-sector

spec: Use BuildRequires, add diag/ subdirectory

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

diag: Don't remove BTARGET on make clean

"make clean" should not remove BTARGET

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

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

Merge remote-tracking branch 'pcacjr/syslinux-update-for-hpa'

hdt: Reporting TFTP errors in dump mode

If we get some errors from the tftp, let's report it to the user.
This will make user's life easier to understand why the dumping mode
isn't working.

libupload: Implementing error code on tftp

When using the tftp backend, let's return the errors we got on the
server.

menu: allow "menu hiddenkey" to take multiple keys; run unlabel

- Allow "menu hiddenkey" to take a comma-separated list of keys.
- Run unlabel() on the commands passed to menu hiddenkey.
  XXX: Consider moving unlabelling to post-menu instead.

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

hdt: Fixing automatic mode parsing

Parsing of the auto='' mode was weak an non functionnal in many
configuration like

auto='dump'

This is now fixed.

NEWS: Document MENU HIDDENKEY

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

libupload: Fixing memset call

We have to use the size of the element, not the size of the pointer.

hdt: New release 0.5.0-beta3

cpuid: Using generic model if cyrix id is unknown

When using the specific Cyrix call to determine the model, if the model
is unknown, let's consider the generic call was right.

This insure a better display of the model id.

lua: Delete superfluous $(LIBS) prerequisite

com32.mk already includes the necessary library in $(C_LIBS), we don't
need to list them again in $(LIBS).

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

sysdump: Delete superfluous $(LIBS) prerequisite

com32.mk already includes the necessary libraries in $(C_LIBS), we
don't need to list them again in $(LIBS).

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

hdt: Delete superfluous $(LIBS) prerequisites

com32.mk already includes the necessary libraries in $(C_LIBS), we
don't need to list them again in $(LIBS).

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

menu.c32: Delete superfluous $(LIBS) prerequistite

com32.mk already includes the necessary libraries in $(C_LIBS), we
don't need to list them again in $(LIBS).

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

cpuid: Improving Cyrix/NSC detection

This code add the specific detection code for Cyrix/NSC processor.
Code came from the Linux kernel.

hdt: Fixing buffer corruption in PXE menu

Jim Cromie reported that PXE menu behave like :

gPXE Detected: ]No

This commit avoid this buffer corruption.

syslinux: check --update option properly

Signed-off-by: Paulo Alcantara <pcacjr@gmail.com>

extlinux: remove already_installed

Signed-off-by: Paulo Alcantara <pcacjr@gmail.com>