search for: start_ldlinux

...lelibi--- via Syslinux wrote: > From: Sylvain Gault <sylvain.gault at gmail.com> > > 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 at gmail.com> We don't re*load* ldlinux.c32, I hope? I assume we just re-execute it? -hpa

From: Sylvain Gault <sylvain.gault at gmail.com> 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 at gmail.com> --- core/elflink/load_env32.c | 58 ++++++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 57 insertions(...

From: Sylvain Gault <sylvain.gault at gmail.com> Hello there, I propose 2 patches that fix two possible stack overflows either when running a COM32 module or when loading a new config file. I didn't find a better way to do this than to use the infamous setjmp/longjmp functions to restore the stack to a previous state. This makes the logic a bit more complex, but the behavior is not

...wrote: >> From: Sylvain Gault <sylvain.gault at gmail.com> >> >> 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 at gmail.com> > > We don't re*load* ldlinux.c32, I hope? I assume we just re-execut...

...m: Sylvain Gault <sylvain.gault at gmail.com> >>> >>> 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 at gmail.com> >> >> We don't re*load* ldlinux.c32, I hope? I...

On Wed, 12 Jun, at 11:17:44AM, Gerardo Exequiel Pozzi wrote: > Cool thanks!. Now looks better, but still not work. > > For some reason, "ldlinux.c32" is apparently sent but "Failed to load" > by PXELINUX and few seconds later, dnsmasq shows an error message > "failed sending": Argh! The patch was broken. I missed the new core/path.c file. My bad.

...;> That's excellent to know. Thanks. > > And numerical value of 6 is EFI_NOT_READY, indicating the keyboard > buffer is empty, as expected. > Gene, Yeah I guess that could be good or bad news. Also I have determined where it is getting hung at -- core/elflink/load_env32.c in start_ldlinux at rv = spawn_load(LDLINUX, argc, argv); Do you have any other ideas for troubleshooting?

...t;); - PATH = malloc(strlen(CurrentDirName) + 1); - if (!PATH) { + if (strlen(CurrentDirName) && !path_add(CurrentDirName)) { printf("Couldn't allocate memory for PATH\n"); goto out; } - strcpy(PATH, CurrentDirName); - init_module_subsystem(&core_module); start_ldlinux(1, argv); @@ -159,30 +156,15 @@ void load_env32(com32sys_t * regs __unused) if (!core_getcwd(path, sizeof(path))) goto out; - if (!strlen(PATH)) { - PATH = realloc(PATH, strlen(path) + 1); - if (!PATH) { - printf("Couldn't allocate memory for PATH\n"); - goto out;...

...t;); - PATH = malloc(strlen(CurrentDirName) + 1); - if (!PATH) { + if (strlen(CurrentDirName) && !path_add(CurrentDirName)) { printf("Couldn't allocate memory for PATH\n"); goto out; } - strcpy(PATH, CurrentDirName); - init_module_subsystem(&core_module); start_ldlinux(1, argv); @@ -159,30 +156,15 @@ void load_env32(com32sys_t * regs __unused) if (!core_getcwd(path, sizeof(path))) goto out; - if (!strlen(PATH)) { - PATH = realloc(PATH, strlen(path) + 1); - if (!PATH) { - printf("Couldn't allocate memory for PATH\n"); - goto out;...

On Mon, 10 Jun, at 07:57:50AM, H. Peter Anvin wrote: > Either that or make the path a list rather than a string, using the > normal word separators when entered on the command line, a bit like the > (t)csh does. That is a bigger change but is probably a better solution. How would this solution handle filenames containing spaces? Would we need to escape (presumably with a backslash)

...; - if (!PATH) { > + if (strlen(CurrentDirName) && !path_add(CurrentDirName)) { > printf("Couldn't allocate memory for PATH\n"); > goto out; > } > > - strcpy(PATH, CurrentDirName); > - > init_module_subsystem(&core_module); > > start_ldlinux(1, argv); > @@ -159,30 +156,15 @@ void load_env32(com32sys_t * regs __unused) > if (!core_getcwd(path, sizeof(path))) > goto out; > > - if (!strlen(PATH)) { > - PATH = realloc(PATH, strlen(path) + 1); > - if (!PATH) { > - printf("Couldn't allocate m...

From: Matt Fleming <matt.fleming at intel.com> The following patch series is available at, git://git.zytor.com/users/mfleming/syslinux.git elflink All patches are against the 'elflink' branch. This series fixes a few serious bugs and some behavioural incompatibilities with the 4.x series. Matt Fleming (19): ldlinux: Initialise 'p' before using it. ldlinux: Parse

...ialized variable ldlinux: Fix build warnings caused by refstrdup() usage core: Add open_config() prototype ldlinux: Use '\0' instead of NULL when dealing with characters ldlinux: Cast some arguments to avoid compiler warnings ldlinux: Fix number of arguments to start_ldlinux() ldlinux: Add prototype for new_linux_kernel() ldlinux: Fix uninitialized variable warning ldlinux: Use signed char consistently core: Move write_serial() prototype to core.h ldlinux: Include write_serial() prototype meminfo: If we allocate with lmalloc() we sho...

From: Matt Fleming <matt.fleming at intel.com> The first two patches in this series are fixes for bugs reported by someone in #syslinux. The third makes dmitest.c32 actually wait for user input. If no one has any objections/concerns, I'll pull this into the official elflink branch. Matt Fleming (3): core: Print error message if we don't load ldlinux.c32 Clean up $(GPLLIB) leak

...n Mon, Aug 31, 2015 at 8:23 PM, Derrick M <derrick.martinez at gmail.com> >> wrote: >> > Gene, >> > >> > Yeah I guess that could be good or bad news. Also I have determined >> > where it >> > is getting hung at -- core/elflink/load_env32.c in start_ldlinux at rv = >> > spawn_load(LDLINUX, argc, argv); >> > >> > Do you have any other ideas for troubleshooting? >> >> spawn_load() should be the call to load ldlinux.* and in this case, >> use the network to retrieve it, likely the first network IO operation. &...

On Mon, Aug 31, 2015 at 06:08:19AM -0400, Gene Cumm via Syslinux wrote: > On Aug 30, 2015 8:42 PM, "Derrick" <derrick22 at gmail.com> wrote: > > > > Gene thanks, here is the output > > > > My IP is 10.2.49.10 > > Img @ 71d89718 = 8cdcd40ca5f0 > > Udp @ 71d89718 = 8cdcd40ca5f0 > > Udp @ 71d89718 = 8cdcd40ca5f0 > > Udp @ 71d89718 =

...; + ldlinux_enter_command(!noescape); + } else if (type == IMAGE_TYPE_CONFIG) { char *argv[] = { "ldlinux.c32", NULL }; /* kernel contains the config file name */ @@ -105,9 +114,10 @@ void execute(const char *cmdline, enum kernel_type type) mangle_name(config_cwd, args); start_ldlinux(argv); - } else if (type == KT_LOCALBOOT) { + } else if (type == IMAGE_TYPE_LOCALBOOT) { local_boot(strtoul(kernel, NULL, 0)); - } else if (type == KT_PXE || type == KT_BSS || type == KT_BOOT) { + } else if (type == IMAGE_TYPE_PXE || type == IMAGE_TYPE_BSS || + type == IMAGE_TYPE_BOOT) {...

From: Matt Fleming <matt.fleming at intel.com> Since we can't use __intcall() for EFI, and since we can now have the ELF module code resolve all our symbols at runtime, we should delete as many references to __intcall() as possible and just access the symbols directly. The most interesting patch is the support for weak symbols. We need to be able to reference derivative-specific

Hello, the following patches should get multidisk access working. The syntax accepted is the following: (hdx,y)/path/to/file where x is the disk number and start at 0 and the y is the partition number starting at 1. So (hd0,1) is the first partition of the first disk. the other accepted syntax is using MBR's 32 bits disk signature so for example: (mbr:0x12345678,2)/foo/bar would address