[PATCH] Sanitize printf functions

Kostya Serebryany kcc at google.com
Sat Dec 28 08:50:32 PST 2013



================
Comment at: sanitizer_common/sanitizer_common_interceptors.inc:33
@@ -32,2 +32,3 @@
 
+#include <stddef.h>
 #include <stdarg.h>
----------------
Yury Gribov wrote:
> Kostya Serebryany wrote:
> > Evgeniy Stepanov wrote:
> > > No system includes (stdarg is a necessary evil).
> > can you avoid this? 
> Well, I need size_t for snprintf.
please use SIZE_T and don't include any additional system headers


http://llvm-reviews.chandlerc.com/D2480



More information about the llvm-commits mailing list