<div dir="ltr">I was going to comment on that, but this code will go away once I finish the work on the symbol visitor and start using libcodeview from here.</div><br><div class="gmail_quote"><div dir="ltr">On Fri, May 20, 2016 at 4:12 PM Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">rnk added a subscriber: rnk.<br>
<br>
================<br>
Comment at: llvm/trunk/lib/DebugInfo/PDB/Raw/SymbolStream.cpp:38<br>
@@ +37,3 @@<br>
+  ulittle16_t seg;<br>
+  char name[1];<br>
+};<br>
----------------<br>
In the future, please don't use these kind of flexible array members, they are how you get buffer overruns when the strings are not null terminated. Look at how we are deserializing codeview types instead.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D20480" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20480</a><br>
<br>
<br>
<br>
</blockquote></div>