| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
9acbffd33b9200ffe37833463b4d4478e824295a caused more issues than it solves.
There appears to be a bigger underlying issue that the __weak argument masks.
Until the underlying issue is solved, we need __weak.
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>
|
|
|
|
| |
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>
|
|
|
|
|
|
|
|
|
| |
Weaken 2 declarations. Seems to be the minimal amount for now.
First exhibited issues with EFI but later also with BIOS beyond *pxelinux.0
when compiled with gcc-5.3.1 and gcc-6.
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>
|
|
|
|
|
|
| |
Fix a dprintf() message so it matches reality.
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>
|
|
|
|
|
|
|
| |
When VSHIFT is used, the scrollbar's position is relative to the old
position.
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>
|
|
|
|
|
|
| |
During the string substitution, an addtional '/' is added unnecessarily.
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>
|
|
|
|
|
| |
Reported-by: Ady <ady-sf@hotmail.com>
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>
|
|
|
|
|
|
|
| |
This patch changes only strings, no program code.
Reported-by: Ady <ady-sf@hotmail.com>
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
These functions are defined in sys/sysmacros.h, so add the include to
main.c. This is already handled correctly in mountinfo.c. Otherwise
we get build failures like:
main.o: In function 'find_device_sysfs':
extlinux/main.c:1131: undefined reference to 'minor'
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>
|
|
|
|
|
|
|
| |
FTBFS: com32/hdt/hdt-dump.c:227:39: error: ‘TFTP_OK’ undeclared
(first use in this function)
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>
|
|
|
|
|
|
| |
The previous commit had some unfortunate line-end garbage; remove.
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>
|
|
|
|
|
|
|
|
| |
We already have a core function for setting the IP address of an URL
object based on network lookup or the server default. Export and use
it instead of open-coding the equivalent logic in upload_tftp.c.
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>
|
|
|
|
|
|
|
|
|
| |
We used to need a wrapper around the core function dns_resolv() to
implement pxe_dns(), because the former function required its argument
to live in low memory. This is no longer the case and hasn't been for
a while, so remove this unnecessary level of indirection.
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>
|
|
|
|
|
|
|
|
| |
Provide weak stubs for the case where the tftp functions aren't
available. This prevents link failures for the case of running on top
of non-network cores.
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>
|
|
|
|
|
|
|
| |
The prototypes themselves are not weak, and declaring them as
such forces the core definitions, which should not be weak, as weak.
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>
|
|
|
|
| |
Signed-off-by: Pete Batard <pete@akeo.ie>
|
|
|
|
|
|
| |
MSVC requires prefix and suffix attributes for packing structures.
Signed-off-by: Pete Batard <pete@akeo.ie>
|
|
|
|
|
|
|
|
|
|
| |
The variable 'ok' is never used and generates a warning. Remove it.
Also ntfssect.c is designed to be compiled in non Unicode mode when
using MSVC compilers, so remove all ambiguity about it
(LPCTSTR -> LPCSTR, use of 'A' API calls) so that it doesn't break when
compiled in Unicode mode, which is what Rufus uses with MSVC.
Signed-off-by: Pete Batard <pete@akeo.ie>
|
|
|
|
|
|
|
|
|
| |
NASM 2.11.06 is known to miscompile Syslinux, so error out on that
NASM version.
Kind of embarrassing for me...
Signed-off-by: H. Peter Anvin <hpa@zytor.com>
|
|
|
|
|
|
|
| |
Since kbd 2.0.3 (commit 6ff47cf2) the format generated by 'loadkeys -m' has
changed slightly, using "unsigned short" instead of "u_short"
Signed-off-by: Olivier Brunel <jjk@jjacky.com>
|
|
|
|
|
|
|
|
| |
--unresolved-symbols=report-all apparently doesn't report the case
where a symbol is requested but never referenced. Error out if
necessary.
Signed-off-by: H. Peter Anvin <hpa@zytor.com>
|
|
|
|
|
|
|
| |
http_bake_cookies() is never called from assembly code, remove
the declaration.
Signed-off-by: H. Peter Anvin <hpa@zytor.com>
|
|
|
|
|
|
| |
pm_local_boot is never used, remove it.
Signed-off-by: H. Peter Anvin <hpa@zytor.com>
|
|
|
|
|
|
|
| |
There is absolutely no reason to align to 64K. Correct the note about
this being the heap; COM32R modules were Syslinux 4.x specific.
Signed-off-by: H. Peter Anvin <hpa@zytor.com>
|
|
|
|
|
|
|
| |
Used wrong variable.
Reported-By: Patrick Masotta <masottaus@yahoo.com>
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>
|
|
|
|
|
|
|
| |
HIDDEN was added to ld in binutils version 2.23 published in August 2012.
This helps preserve backwards compatibility for now.
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>
|
|
|
|
|
|
|
| |
Update comments which explains next_extent() semantics to be slightly
clearer, and also correct the statement that next_extent can be NULL.
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>
|
|
|
|
|
|
|
|
|
| |
We don't need any of the _lma symbols, except for the one associated
with .pm_code. Don't export them unconditionally in SECINFO, because
that makes the linker request them as unknown symbols in the dynamic
section.
Signed-off-by: H. Peter Anvin <hpa@zytor.com>
|
|
|
|
|
|
|
|
| |
Remove symbols and data structures not used in the i386-bios build,
and clean up the linker script so that most internal symbols are
HIDDEN.
Signed-off-by: H. Peter Anvin <hpa@zytor.com>
|
|
|
|
|
|
| |
Remove one last stray instance of EFI_BUILD.
Signed-off-by: H. Peter Anvin <hpa@zytor.com>
|
|
|
|
|
|
| |
Clean up the handling of firmware selection inside the Makefiles.
Signed-off-by: H. Peter Anvin <hpa@zytor.com>
|
|
|
|
|
|
|
|
|
| |
gPXE has been deprecated in favor of iPXE for many, many years now.
It is much better than users get it directly from the iPXE project,
since we should no longer need any special modifications for Syslinux
use.
Signed-off-by: H. Peter Anvin <hpa@zytor.com>
|
|
|
|
|
|
| |
Remove several files which aren't used at all during the build.
Signed-off-by: H. Peter Anvin <hpa@zytor.com>
|
|\
| |
| |
| |
| |
| |
| | |
'git://github.com/Celelibi/syslinux.git/fix/bios/tail-call-stack-overflows'
FIXME: this circumvents some of the issue but is not the cleanest and may
reoccur.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The behavior when running a "CONFIG" command line is to reload
ldlinux.c32 with the new file as argument. This call never return.
In order to avoid stacking up the calls to start_ldlinux, this patch
introduce a setjmp/longjmp to return to the first call to start_ldlinux,
thus freeing all the stack space.
Signed-off-by: Sylvain Gault <sylvain.gault@gmail.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
When a COM32 module exits, the functions never return and a new call to
ldlinux_enter_command is made. This could fill the stack and overflow on
some data present in memory.
This patch use setjmp/longjmp to return to the main function and restart
from there when a COM32 module exits.
Signed-off-by: Sylvain Gault <sylvain.gault@gmail.com>
|
| |
| |
| |
| |
| |
| |
| | |
"may be used uninitialized in this function [-Wuninitialized]" even though logic dictates
it must be OK.
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>
|
|\ \ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Besides supporting newer version of xfs file system, this patch also
does some code refactoring and fix completely broken listing and
searching on v2-3 node directories.
Cc: Gene Cumm <gene.cumm@gmail.com>
Cc: H. Peter Anvin <hpa@zytor.com>
Cc: Raphael S. Carvalho <raphael.scarv@gmail.com>
Cc: Ady <ady-sf@hotmail.com>
Signed-off-by: Paulo Alcantara <pcacjr@zytor.com>
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The current btrfs support did not handled multiple stripes stored in
chunk items, hence skipping the physical addresses that were needed to
do the mapping.
Besides, the chunk tree may contain DEV_ITEM keys which store
information on all of the underlying block devices, so we must skip them
instead of finishing lookup.
The bug was reproduced with btrfs-progs v4.2.2.
Cc: Gene Cumm <gene.cumm@gmail.com>
Cc: H. Peter Anvin <hpa@zytor.com>
Signed-off-by: Paulo Alcantara <pcacjr@zytor.com>
---
v1 -> v2:
* Do not set ignore_key multiple times. Set it before parsing chunk
tree.
v2 -> v3:
* Replace an unnecessary goto with a continue statement.
|
| |
| |
| |
| |
| |
| | |
Previous commit changed the dependency filenames to not have a leading '.'
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
We can let compiler driver pass the right options to preprocessor after
processing -Mxy options, right now its bypassing the gcc driver and
handing them straight to cpp
This also helps in other compilers processing these options correctly
for their preprocessors consumption
Signed-off-by: Khem Raj <raj.khem@gmail.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This patch is coming after some bad experience with gfxboot.
GFXboot was padding unexpected options on the chain.c32 commande line.
As a result, chain was confused and didn't provided the expected
behavior.
This patch is a workaround to enforce chain.c32 not considering anything
on the command line after this keyword.
Example:
chain.c32 hd0 exit ...garbage...
|
| |
| |
| |
| |
| |
| |
| |
| | |
When parsing the modules.alias, we shall report it as broken if we
cannot find any valid line in it.
This patch simply count the number of valid lines and report a missing
modules.alias if no valid lines are found.
|
| |
| |
| |
| |
| |
| |
| |
| | |
This option is mandatory when compiling for EFI as some event handlers
may interupt the running code and use the space just above the reserved
stack space.
Signed-off-by: Sylvain Gault <sylvain.gault@gmail.com>
|
| |
| |
| |
| |
| |
| |
| |
| | |
This variable indicates whether or nor the files are compiled for EFI.
The lack of it lead the Makefiles to forget to add some compilation
options specific to EFI.
Signed-off-by: Sylvain Gault <sylvain.gault@gmail.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Merge btrfs_read_adv and xfs_read_adv into a single generic function
ext_read_adv and split ext_write_adv_offset out of ext_write_adv.
Use those new functions in rewrite_boot_image and btrfs_install_file
where it is actually hardcoded.
Signed-off-by: Nicolas Cornu <nicolac76@yahoo.fr>
Signed-off-by: Paulo Alcantara <pcacjr@zytor.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Spaces, ASCII 0x20, have been
* moved
* inserted
* deleted
Signed-off-by: Geert Stappers <stappers@stappers.nl>
|
| |
| |
| |
| |
| |
| |
| | |
As of commit ID 37eef640 (before 6.03-pre12, before 6.03), it is now in
Boot Area B.
Signed-off-by: Gene Cumm <gene.cumm@gmail.com>
|
| |
| |
| |
| |
| |
| |
| | |
Some typo fixes in extlinux/main.c.
None of them affects code execution.
Signed-off-by: Geert Stappers <stappers@stappers.nl>
|