pxechn: Fix merge botch s/pxe_dns/pxe_dns_resolv/

pxe_dns() was deleted in commit 6aba981cd93 ("pxe: resolve names via
DNS from protected-mode code"), so when pxechn was merged from
'master' we should really have updated pxechn.c to use
pxe_dns_resolv().

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

Delete all references to __com32.cs_bounce

The COM32 cs_bounce buffer is not usable with ELF modules, as we're
trying to move to an environment where memory is dynamically
allocated. All users of __com32.cs_bounce have been converted to using
lmalloc() to allocate low memory.

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

module: Actually use last component of DT_NEEDED pathname

The comment in module_load() claims that we strip everything from the
DT_NEEDED pathnames but the last component. That's not true as we
leave the initial '/', which works fine if the module is in the root
directory, but not so well if we're in a sub-directory. Worse still,
if the DT_NEEDED entry pathname doesn't include a '/' the module is
skipped entirely.

We should also be loading dependencies in reverse order, as modules
loaded because of DT_NEEDED entries may also have dependencies and
their dependencies should be loaded first.

Also, now we need the strrchr() implementation in the core, so move
strrchr.o into $CORELIBOBJS.

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

ldlinux: fixes bug that happens when using fullpath for a COM32 module

When using full path for a com32 module, for example,
/boot/syslinux/ls.c32 it fails without any error message. This patch
fixes it by looking first if the argv[0] is the path to a module before
looking for it at PATH.

Since we're using fopen to open module files (which works for both
absolute paths and paths relative to the current working directory) we
no longer need to include "." in PATH and neither the code to handle it.

Signed-off-by: Andre Ericson <de.ericson@gmail.com>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>

Merge remote-tracking branch 'mfleming/merge/elflink/master' into elflink

chainboot: clean up old assemblyisms

Delete a bunch of stuff related to the old assembly implementation of
chainloading.  There are still a few, like the use of pm_load_high()
that should be cleaned up.

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

execute(): move chainbooting code to its own file

To improve readability, move chainbooting to its own source file.

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

Merge remote-tracking branch 'mfleming/elflink-dot-zero' into elflink

Resolved Conflicts:
com32/elflink/ldlinux/execute.c

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

elflink: Fix boot sector booting

This adds missing support for booting from a boot sector file such as
.bs, .bss or .0, by re-implementing the old asm bootsec code from
core/bootsect.inc in C.

This has resulted in some external changes. We've had to make StackBuf
a global symbol because we access it directly from execute.c. Also, we
need to move dsinfo.c into MINLIBOBJS because ldlinux now needs to
reference __syslinux_derivative_info.

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

Merge remote-tracking branch 'zytor/master' into merge/elflink/master

A lot of development has gone on in the 'master' branch since the last
time we merged; new features, bug fixes, etc, etc.

Conflicts:
Makefile
com32/Makefile
com32/lib/Makefile
com32/lib/syslinux/load_linux.c
com32/modules/Makefile
com32/modules/chain.c
core/bootsect.inc
core/init.inc
version

core: remove unused variable in scan_highmem_area()

Remove a completely unused variable giving a "set but not used"
warning.

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

Merge commit 'd76af0754db7ddd4c06215d922d213c93178c0af' into elflink

relocs: Move stop to the end

The Linux kernel puts the stop word at the beginning of the relocation
list (the list is processed backwards); Syslinux puts the stop word at
the beginning of the relocation list (the list is processed forwards.)

Missed that change when syncing with the kernel version.

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

pxe: move hardwired DHCP options into high memory

If we have hardwired DHCP options, move them into high memory instead
of into the trackbuf.  This eliminates yet another trackbuf user.

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

pxe: remove remaining stray trackbuf reference

We aren't using trackbuf, stop referring to it...

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

pxe: stop using trackbuf for DHCP packets

Get rid of yet another trackbuf usage.

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

core/font.c: remove another reference to trackbuf

Remove a dead reference to trackbuf.

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

core/font: clean up and remove trackbuf usage

This code was pretty messed up; it used the trackbuf for simple file
I/O, but didn't use lowmem for the actual font buffer, which is
touched by BIOS.

This also removes yet another user of trackbuf.

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

relocs: sync with the Linux kernel

Sync the relocs tool with the Linux kernel.  The new version of this
tool correctly verifies that any absolute symbol is either listed as
allowed absolute or is listed as relative.

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

core: add support for .init/fini_table

We don't actually have any constructors/destructors in the core at
this time, but handle .init/fini_table in case that happens...

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

com32.ld: handle .init_array and .fini_array

Handle constructors/destructors via .init_array and .fini_array, as
generated by newer gccs.

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

com32: fix warning in host module

host.c: In function ‘main’:
host.c:56:9: warning: implicit declaration of function ‘exit’
                            [-Wimplicit-function-declaration]
host.c:56:9: warning: incompatible implicit declaration of built-in
                                function ‘exit’ [enabled by default]

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

com32: fix warning in elf_module

sys/module/elf_module.c: In function ‘module_load’:
sys/module/elf_module.c:454:8: warning: ‘ctors’ may be used
                    uninitialized in this function [-Wuninitialized]
sys/module/elf_module.c:404:17: note: ‘ctors’ was declared here

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

localboot: remove unnecessary real-mode call

syslinux_local_boot() used to call local_boot() from INT 0x22
(AX=0x0014) as it should just call local_boot() function.

The local_boot() function is now exported by core/include/localboot.h to
be used in COM32 library space so that syslinux_local_boot() can use it.

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

Merge remote-tracking branch 'hdt/master'

Merge remote-tracking branch 'genec/diag-mbr-fix-for-hpa'

diag/mbr: Clarify README

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

Update diag/mbr instruction to match the current filename.

I suspect that some instructions about how to use the diag/mbr was not
updated when the source file was renamed to handoff.S. Here is a simple
proposition to fix that only into the handoff.S file.

Jean-Christian de Rivaz

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

pxechn.c32: PXE NBP chainloader

Designed as a more versatile COM32-based alternative to pxechain.com.
It can use the PXE RESTART or chain to the new NBP without the PXE
stack.  It also enables a user to boot Microsoft Windows Server 2008R2
Windows Deployment Services's wdsnbp.com from PXELINUX.

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

COM32: add DHCP pack/unpack functions

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

hdt: 0.5.2-pre2 release

hdt: Adding say & postexec demo

When building official hdt image, let's add a say & postexec example.

hdt: Adding say command

This command is just for displaing a message to the cli during a defined
period of time.

Syntax is like the following : say `my message`%<number_of_seconds>
An example :

say `This is my text message to display during 5 seconds`%5

hdt: Adding nomodule support in cli

Some new commands might need being able to manage the argv directly
instead of the much more oriented scheme we had until now.

This commit add a .nomodule option to cli object to explically tell they
don't have a module as parameter but only arguments.

This will be needed for the 'say' command.

hdt: Fixing argument mgmt in cli

Sizeof(char *) is definetly wrong for getting the length of a string.

hdt: Adding more debug traces in cli

Adding more debug traces in the cli management to ease debugging &
feature adding.

hdt: Fixing chain32 target in makefile

chain32 moved, let's adjust the "release" target.

Merge remote-tracking branch 'hpa/master'

graphics: make use of syslinux_force_text_mode()

The syslinux_force_text_mode() function used in library
space made unnecessary INT 0x22 call that could be simply
used with vgaclearmode() function.

This patche renames vgaclearmode() to
syslinux_force_text_mode() as this naming is generally more
carefully considered and gets rid of "forcetext.c" source
file.

Also all the VGA-related functions and variables that were
exported in core/include/bios.h have been moved out to
core/include/graphics.h which makes more sense actually.

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

CLI: don't highlight the prompt

Don't gratuitously highlight the prompt; it is just annoying.

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

syslinux_boot_linux(): revert to text mode unless vga=current

When loading a Linux kernel, revert to text mode before invoking
unless vga=current.

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

com32: make syslinux_dump_*() pure debugging functions

Make the syslinux_dump_*() functions pure debugging functions; that is
the way that they are used anyway, and this way they log to the same
place as dprintf.

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

Use <dprintf.h> instead of an ad hoc definition

Instead of using ad hoc definitions of dprintf, dprintf2, vdprintf and
vdprintf2, use <dprintf.h> everywhere possible.

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

dprintf: always define dprintf2/vdprintf2

Always define dprintf2/vdprintf2 even with no debugging at all
enabled.

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

lib/shuffle: use <dprintf.h>

Another straggler not yet using <dprintf.h> for debugging.

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

dprintf: add the ability to log to stdio

Add the ability to redirect dprintf to stdio when there is no other
choice.

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

com32/lib/dprintf.c
com32/lib/vdprintf.c

vdprintf.c: undo double-printing

Undo double-printing change so we can sync with master.

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

CLI: Stick to the classic "boot: " prompt rather than "syslinux$"

Just for familiarity...

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

fs.h: A slighly more useful default PATH

/bin is clearly insane...

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

Makefile: install library modules, too

We need the library modules to be added to MODULES.

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

elflink: Make sure to install ldlinux.c32 at install time

We need to reach ldlinux.c32 during early boot...

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

elflink: make dprintf() usable in the core

Make dprintf() usable in the core, as it it on the mainline branch.

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

syslinux.mk: use $(MAKEDIR) not $(makefiledir)

Everything else in the system seems to use $(MAKEDIR), not
$(makefiledir)...

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

graphics: report video mode change from protected-mode code

Syslinux used to call __intcall() for calling routines of the old
COMBOOT API to report video mode change (INT 22h, AX=0x0017) that seemed
pointless, since INT 22h, AX=0x0017 does call the protected-mode
function pm_using_vga() already when calling INT 22h, AX=0x0017. So for
reporting video mode changes (VGA in this case) we must call
graphics_using_vga() instead for now.

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

com32: Do not use hard-coded values for PXE flags

There is already a syslinux/pxe_api.h header file which contains PXE
flags, so these hard-coded values are unnecessaries.

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

pxe: resolve names via DNS from protected-mode code

Syslinux used to call __intcall() for calling routines of the old
COMBOOT API to resolve names via DNS (INT 22h, AX=0x0010) that seemed
pointless, since INT 22h, AX=0x0010 does call the protected-mode
function pm_pxe_resolv_dns() when calling INT 22h, AX=0x0010. So, for
resolving names via DNS we must call pxe_dns_resolv() (a protected-mode
function) instead for now.

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

pxe: rename pxe_dns_resolv() function to pm_pxe_dns_resolv()

As pxe_dns_resolv() (a protected-mode function) is just a wrapper for
real mode (used in the old COMBOOT API), so it makes more sense to
have it named as pm_pxe_dns_resolv().

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

host.c32: do not use INT 22h, AX=0x0010 to resolve DNS

Use pxe_dns() function (assigned in syslinux/pxe.h) instead for
resolving DNS.

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

PXELINUX: do not use hard-coded values

PXENV_RESTART_TFTP and PXENV_FILE_EXEC flags are already declared in
core/pxe.inc, so we don't need to use hard-coded values there.

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

ISOLINUX: use bailmsg string in kaboom()

Every code which implements kaboom(), uses bailmsg string. So we must
use it in core/isolinux.asm as well.

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

ISOLINUX: trivial fix

Fix minor typo.

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

Merge remote-tracking branch 'mfleming/elflink' into elflink

ldlinux: Parse ALLOWOPTIONS directive

Specifying the ALLOWOPTIONS directive doesn't have any effect at the
moment. Fix this so that if ALLOWOPTIONS 0 is used then command line
arguments are ignored.

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

elflink: Fix TIMEOUT and TOTALTIMEOUT handling

Paulo reported that his default command line wasn't being executed
when the timeout specified in his config file elapsed. This is because
mygetkey() wasn't correctly applying the timeout when waiting for
input.

Furthermore, it seems the ONTIMEOUT parsing was also broken.

Reported-by: Paulo Alcantara <pcacjr@zytor.com>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>

ldlinux: Don't drop args if we auto-lookup cmd extension

If we automatically lookup the extension for a command and the user
has supplied an argument to that command we incorrectly stamp the NUL
byte after the extension, and not after the argument, thereby
effectively erasing the argument.

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

Merge remote-tracking branch 'genec/diag-geodsp-perl-for-hpa'

get_key: Valid key values are positive

Make sure we return positive values for valid keystrokes.

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

core/graphics.c: Fixes and cleanups

Fix up various conversion bugs from assembly.  Eventually this whole
file should be removed and the functionality moved to using the VESA
framework.

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

hw/vga.h: Header file for VGA hardware registers

Header file for VGA hardware registers; so far only the base registers
and sub-register indicies, but no bit masks or bit positions; those
can be added as needed.

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

Merge branches 'auto-extension' and 'fix-build-warnings' into elflink

Conflicts:
com32/elflink/ldlinux/ldlinux.c

Mark unused function parameters as __unused

Move the __unused tag from com32/hdt/ to com32/include/klibc/ so that
it can be used by the entire code base, and mark unused function
parameters as __unused to stop the following kind of build warning,

elflink/load_env32.c: In function ‘load_env32’:
elflink/load_env32.c:107:30: warning: unused parameter ‘regs’

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

elflink: Remove unused variables

Reduce the number of build warnings by deleting unused variables.

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

fs: Include unistd.h for chdir() prototype

fs/lib/chdir.c: In function ‘generic_chdir_start’:
fs/lib/chdir.c:5:2: warning: implicit declaration of function ‘chdir’

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

fs: Include header for close() prototype

... which avoids the following build warning,

fs/fs.c: In function ‘open_config’:
fs/fs.c:97:2: warning: implicit declaration of function ‘close’

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

elflink: Allocate space for 'realname'

search_dirs() expects the 'realname' argument to point to space
allocated for storing the real path name. Currently we're passing an
uninitialized pointer which was highlighted by the following build
warning,

elflink/load_env32.c:155:18: warning: ‘realname’ may be used uninitialized in this function

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

elflink: Delete obsolete constructor code

Since commit 8e0ed96bff75 ("elf: Support __constructor and
__destructor") the code for looping over ctors in ldlinux has been
unnecessary. Delete since this is all now handled in the core module
loading code.

This also gets rid of the following build warning,

elflink/load_env32.c: At top level:
elflink/load_env32.c:28:15: warning: array ‘__ctors_start’ assumed to have one element
elflink/load_env32.c:28:32: warning: array ‘__ctors_end’ assumed to have one element

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

conio: Fix up compiler warnings

conio.c: In function ‘pm_getchar’:
conio.c:336:2: warning: pointer targets in passing argument 1 of ‘getchar’ differ in signedness
conio.c:268:6: note: expected ‘char *’ but argument is of type ‘uint8_t *’
conio.c: In function ‘pm_pollchar’:
conio.c:261:1: warning: control reaches end of non-void function

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

graphics: Fix GXPix* assignment

We need assign cols/rows to GXPixCols/GXPixRows separately, we can't
rely on the compiler writing the 32-bit value to two consecutive
16-bit memory locations like the assembly version did. The value we
were storing was actually being truncated by the compiler (see the
warning below),

Also fixup the following warnings,

graphics.c: In function ‘vgasetmode’:
graphics.c:95:18: warning: cast from pointer to integer of different size
graphics.c:102:2: warning: large integer implicitly truncated to unsigned type
graphics.c: In function ‘outputvga’:
graphics.c:222:12: warning: cast from pointer to integer of different size
graphics.c:227:11: warning: cast from pointer to integer of different size
graphics.c: In function ‘vgadisplayfile’:
graphics.c:303:4: warning: passing argument 1 of ‘outputvga’ from incompatible pointer type
graphics.c:213:13: note: expected ‘uint32_t *’ but argument is of type ‘uint8_t *’

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

font: Cast away compiler warning

font.c: In function ‘adjust_screen’:
font.c:167:30: warning: initialization makes pointer from integer without a cast

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

writestr: Include core.h for writechr() prototype

This eliminates the following compiler warning,

writestr.c: In function ‘crlf’:
writestr.c:26:2: warning: implicit declaration of function ‘writechr’

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

hello: Delete unused variable 'console_init'

Which also gets rid of the following warning,

hello.c:10:12: warning: ‘console_init’ defined but not used

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

init: Fix up compiler warnings

Fix up the following compiler warnings by including appropriate
headers, applying some casts, and changing some data types,

init.c:7:1: warning: large integer implicitly truncated to unsigned type
init.c: In function ‘check_escapes’:
init.c:32:10: warning: cast from pointer to integer of different size
init.c:46:4: warning: implicit declaration of function ‘writestr’
init.c: In function ‘bios_timer_init’:
init.c:57:19: warning: initialization makes pointer from integer without a cast
init.c:61:8: warning: assignment makes integer from pointer without a cast
init.c: In function ‘init’:
init.c:74:2: warning: implicit declaration of function ‘adjust_screen’
init.c:75:2: warning: implicit declaration of function ‘printf_init’

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

localboot: Include header files for function prototypes

... to avoid the following warnings,

localboot.c: In function ‘local_boot’:
localboot.c:38:2: warning: implicit declaration of function ‘vgaclearmode’
localboot.c:40:2: warning: implicit declaration of function ‘writestr’
localboot.c:41:2: warning: implicit declaration of function ‘crlf’
localboot.c:42:2: warning: implicit declaration of function ‘cleanup_hardware’
localboot.c:76:2: warning: implicit declaration of function ‘memcpy’

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

meminfo: If we allocate with lmalloc() we should free with lfree()

Since commit 74518b8b691c ("elflink: Make ELF the default object
format") we've been using lmalloc() with free() instead of lfree().
This bug was pointed out by the following build warnings,

meminfo.c: In function ‘dump_e820’:
meminfo.c:93:5: warning: implicit declaration of function ‘free’
meminfo.c:93:5: warning: incompatible implicit declaration of built-in function ‘free’
vesainfo.c: In function ‘print_modes’:
vesainfo.c:82:2: warning: implicit declaration of function ‘free’
vesainfo.c:82:2: warning: incompatible implicit declaration of built-in function ‘free’

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

ldlinux: Include write_serial() prototype

Include core.h to avoid the following build warning,

eprintf.c: In function ‘veprintf’:
eprintf.c:23:6: warning: implicit declaration of function ‘write_serial’

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

core: Move write_serial() prototype to core.h

There's nothing inherently BIOS-specific about write_serial(), so it
doesn't make sense to have it in bios.h. Move the prototype to core.h
so that files can access the prototype without pulling in all the BIOS
symbols. Also add some missing function prototypes while we're at it.

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

ldlinux: Use signed char consistently

In get_key() we use unsigned chars but seem to use signed chars in
other functions. There's no real reason to use unsigned chars so let's
be consistent. This also eliminates the following warnings,

get_key.c: In function ‘get_key’:
get_key.c:187:2: warning: pointer targets in passing argument 1 of ‘get_key_decode’ differ in signedness
get_key.c:129:5: note: expected ‘char *’ but argument is of type ‘unsigned char *’
serirq.c:29:29: warning: pointer targets in initialization differ in signedness
serirq.c:30:29: warning: pointer targets in initialization differ in signedness

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

ldlinux: Fix uninitialized variable warning

Fix the following warning by initializing 'kernel_name' to NULL,

kernel.c:18:14: warning: ‘kernel_name’ may be used uninitialized in this function

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

ldlinux: Add prototype for new_linux_kernel()

Make sure we're passing the correct arguments to new_linux_kernel() by
defining a prototype. This also fixes the following warning,

execute.c:108:3: warning: implicit declaration of function ‘new_linux_kernel’

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

ldlinux: Fix number of arguments to start_ldlinux()

Commit 3a316db1 ("ldlinux: Loading a config file should cause
re-initialisation") added a call to start_ldlinux() with an incorrect
number of arguments. Add the prototype to core.h so this error doesn't
go unnoticed again.

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

ldlinux: Cast some arguments to avoid compiler warnings

Specifically these warnings,

execute.c: In function ‘execute’:
execute.c:87:3: warning: passing argument 1 of ‘lfree’ discards qualifiers from pointer target type
../../../com32/include/com32.h:125:6: note: expected ‘void *’ but argument is of type ‘const char *’
execute.c:88:3: warning: passing argument 1 of ‘create_args_and_load’ discards qualifiers from pointer target type
execute.c:40:12: note: expected ‘char *’ but argument is of type ‘const char *’
execute.c:111:2: warning: passing argument 1 of ‘lfree’ discards qualifiers from pointer target type
../../../com32/include/com32.h:125:6: note: expected ‘void *’ but argument is of type ‘const char *’

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

ldlinux: Use '\0' instead of NULL when dealing with characters

We should be using '\0', not NULL when signalling 'end of string'.
Using NULL results in the following warning,

readconfig.c: In function ‘parse_one_config’:
readconfig.c:1355:17: warning: assignment makes integer from pointer without a cast

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

core: Add open_config() prototype

... to fix the following warning,

readconfig.c: In function ‘parse_one_config’:
readconfig.c:1339:3: warning: implicit declaration of function ‘open_config’

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

ldlinux: Fix build warnings caused by refstrdup() usage

Sprinkle 'const' qualifiers when dealing with refstrdup() to fix the
following warnings,

readconfig.c: In function ‘parse_config_file’:
readconfig.c:1127:12: warning: assignment discards qualifiers from pointer target type
readconfig.c:1139:12: warning: assignment discards qualifiers from pointer target type
readconfig.c:1152:12: warning: assignment discards qualifiers from pointer target type
readconfig.c:1315:12: warning: assignment discards qualifiers from pointer target type

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

ldlinux: Silence gcc warning about uninitialized variable

Initialize 'comm_counter' so that gcc will no longer complain with the
following,

cli.c: In function ‘edit_cmdline’:
cli.c:142:25: warning: ‘comm_counter’ may be used uninitialized in this function

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

ldlinux: Add print_labels() prototype to config.h

... to fix the following build warning,

cli.c: In function ‘edit_cmdline’:
cli.c:411:3: warning: implicit declaration of function ‘print_labels’

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

ldlinux: Fix check for last command line entry

The expression that checks to see when we've reached the end of the
command line entries is incorrect, probably because the list macros
weren't used. Use list_is_last() to make the expression correct and
self-documenting.

This bug was highlighted by the following compiler warning,

cli.c: In function ‘cmd_reverse_search’:
cli.c:102:20: warning: comparison of distinct pointer types lacks a cast

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

ldlinux: Match arg types to function prototypes

Change the types of some local variables to match the type in the
function prototypes, thereby avoiding the following warnings,

ldlinux.c:132:2: warning: pointer targets in passing argument 2 of ‘syslinux_getadv’ differ in signedness
../../../com32/include/syslinux/adv.h:55:22: note: expected ‘size_t *’ but argument is of type ‘int *’
ldlinux.c:132:6: warning: assignment discards qualifiers from pointer target type
ldlinux.c:156:3: warning: pointer targets in passing argument 1 of ‘load_kernel’ differ in signedness
ldlinux.c:58:13: note: expected ‘const char *’ but argument is of type ‘uint8_t *’

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

ldlinux: Tag __syslinux_init() as __constructor

We don't need to have control over exactly when __syslinux_init() is
executed, it just needs to be before we call syslinux_get_adv(). Let's
apply the __constructor tag to __syslinux_init() so that it is
executed before we enter ldlinux.c32's main(). This partially reverts
commit 487b67a9ce ("ldlinux: Remove __constructor attribute") because
we can now have __constructor functions in ldlinux.

By not calling __syslinux_init() directly we also avoid the following
build warning,

ldlinux.c: In function ‘main’:
ldlinux.c:131:2: warning: implicit declaration of function ‘__syslinux_init’

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

ldlinux: parse_kernel_type() parameter should be const

Since we always pass a const char * to parse_kernel_type(), change the
type of the parameter accordingly to get rid of the following
warnings,

ldlinux.c:66:3: warning: passing argument 1 of ‘parse_kernel_type’ discards qual
ifiers from pointer target type
ldlinux.c:14:25: note: expected ‘char *’ but argument is of type ‘const char *’
ldlinux.c:80:2: warning: passing argument 1 of ‘parse_kernel_type’ discards qual
ifiers from pointer target type
ldlinux.c:14:25: note: expected ‘char *’ but argument is of type ‘const char *’
ldlinux.c: In function ‘main’:

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