<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 24, 2016 at 2:54 PM, Matthias Braun <span dir="ltr"><<a href="mailto:matze@braunis.de" target="_blank">matze@braunis.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">MatzeB added a subscriber: MatzeB.<br>
<span class=""><br>
================<br>
Comment at: lib/profile/InstrProfilingFile.c:129<br>
@@ -112,1 +128,3 @@<br>
+  if (NumHosts)<br>
</span>+    strncpy(lprofCurFilename.Hostname, HostStr, COMPILER_RT_MAX_HOSTLEN);<br>
<br>
----------------<br>
Note that `strncpy` is a really silly function: If you actually happen to write n or more characters you will not have a terminating '\0' character... My personal rule is to never use `strncpy()` but something like `snprintf(buf, bufsize, "%s%s", one, two);`<br>
<br></blockquote><div><br></div><div>heh -- I agree with you in general ;). Here it won't be a problem as the source of the strncpy is  produced with snprintf. Anyway I will do a follow up to clean it more.</div><div><br></div><div>thanks,</div><div><br></div><div>David</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<a href="http://reviews.llvm.org/D20572" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20572</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div>