On Sun, Jun 8, 2014 at 4:01 PM, Piotr Król <[email protected]> wrote:
> Hello,
> I found that mRunStatus is used uninitialized in VfrCompiler.cpp:56.
> This cause problems at least for some versions of g++ (4.8.3-3,
> 4.7.3-14, 4.6.4-7) on my Debian. Compilation works when mRunStatus
> contain some big value out of COMPILER_RUN_STATUS range (this happens
> when I use i.e. g++ 4.7.2-5), but if it contain for example
> STATUS_FAILED VfrCompiler fails.
I think Reza submitted a patch on April 7 that might fix this.
("BaseTools: fix undefined behaviour in VfrCompiler")
I asked if the commit message could be improved, and I guess nothing
happened with the patch after that.
-Jordan
> Other thing is that VfrCompiler.cpp doesn't contain mRunStatus range
> validation.
>
> What about enabling -Wall for BaseTools ?
>
> Thanks,
> Piotr Król
>
> ------------------------------------------------------------------------------
> Learn Graph Databases - Download FREE O'Reilly Book
> "Graph Databases" is the definitive new guide to graph databases and their
> applications. Written by three acclaimed leaders in the field,
> this first edition is now available. Download your free book today!
> http://p.sf.net/sfu/NeoTech
> _______________________________________________
> edk2-buildtools-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/edk2-buildtools-devel
------------------------------------------------------------------------------
HPCC Systems Open Source Big Data Platform from LexisNexis Risk Solutions
Find What Matters Most in Your Big Data with HPCC Systems
Open Source. Fast. Scalable. Simple. Ideal for Dirty Data.
Leverages Graph Analysis for Fast Processing & Easy Data Exploration
http://www.hpccsystems.com
_______________________________________________
edk2-buildtools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-buildtools-devel
