[PATCH] D20572: [profile] clean up file handling code

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Tue May 24 14:54:04 PDT 2016


MatzeB added a subscriber: MatzeB.

================
Comment at: lib/profile/InstrProfilingFile.c:129
@@ -112,1 +128,3 @@
+  if (NumHosts)
+    strncpy(lprofCurFilename.Hostname, HostStr, COMPILER_RT_MAX_HOSTLEN);
 
----------------
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);`


http://reviews.llvm.org/D20572





More information about the llvm-commits mailing list