From 7dc585763ee2a6d196760a5530c70b6db96351fe Mon Sep 17 00:00:00 2001 From: Benjamin Kramer Date: Tue, 25 Oct 2016 12:39:28 +0000 Subject: [PATCH] Fix diagnostic format string for err_os_log_argument_to_big Patch by Sam McCall, test case by me. Differential Revision: https://reviews.llvm.org/D25936 llvm-svn: 285065 --- clang/include/clang/Basic/DiagnosticSemaKinds.td | 2 +- clang/test/SemaObjC/format-strings-oslog.m | 3 +++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/clang/include/clang/Basic/DiagnosticSemaKinds.td b/clang/include/clang/Basic/DiagnosticSemaKinds.td index 320fb5d..4b0ed6b 100644 --- a/clang/include/clang/Basic/DiagnosticSemaKinds.td +++ b/clang/include/clang/Basic/DiagnosticSemaKinds.td @@ -7570,7 +7570,7 @@ def warn_cfstring_truncated : Warning< def err_os_log_format_not_string_constant : Error< "os_log() format argument is not a string constant">; def err_os_log_argument_too_big : Error< - "os_log() argument %d is too big (%d bytes, max %d)">; + "os_log() argument %0 is too big (%1 bytes, max %2)">; def warn_os_log_format_narg : Error< "os_log() '%%n' format specifier is not allowed">, DefaultError; diff --git a/clang/test/SemaObjC/format-strings-oslog.m b/clang/test/SemaObjC/format-strings-oslog.m index 3101a67..15c88e1 100644 --- a/clang/test/SemaObjC/format-strings-oslog.m +++ b/clang/test/SemaObjC/format-strings-oslog.m @@ -36,6 +36,9 @@ void test_os_log_format(const char *pc, int i, void *p, void *buf) { wchar_t wcs[] = {'a', 0}; __builtin_os_log_format(buf, "%S", wcs); printf("%S", wcs); + + struct { char data[0x100]; } toobig; + __builtin_os_log_format(buf, "%s", toobig); // expected-error {{os_log() argument 2 is too big (256 bytes, max 255)}} } // Test os_log_format primitive with ObjC string literal format argument. -- 2.7.4