[syslinux] EFI: HP + syslinux = crash [ brown paper bag update ]
Gene Cumm
gene.cumm at gmail.com
Sun Sep 13 05:31:28 PDT 2015
On Tue, Aug 11, 2015 at 9:11 AM, Patrick Masotta via Syslinux
<syslinux at zytor.com> wrote:
> >>>
> >
> Please implement these new changes over your last non-debug
> > working build (multi-nic fix) and let's see how it goes.
> >
> > https://github.com/ppatpat/syslinux/compare/master...ppatpat:zlib?expand=1
> >
> > Please report your results.
>
> Results are in. They are very good.
> No crashing even with the original config!
> <<<
>
> It is also working here; I think this particular PNG issue is fixed.
>
> It's worth to mention that the replaced code looks valid to me; that's why I think
> this is probably some sort of compiler bug. It was really hard to find.
> We should keep an eye on this issue; it might appear somewhere else.
Eww. The original code was merely an empty define. That's pretty nasty.
Alternatively, we could void them
-# define Assert(cond,msg)
+# define Assert(cond,msg) ((void)0)
--
-Gene
More information about the Syslinux
mailing list