From: Ted Kremenek <kremenek@apple.com>
Date: Fri, 12 Oct 2007 00:11:27 +0000 (+0000)
Subject: Printf argument checking now supports dynamically-passed precision
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=bef679c131920ee15a059c80969e6f3f7a0c0ae7;p=clang

Printf argument checking now supports dynamically-passed precision
specifiers.


git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@42886 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/Sema/SemaChecking.cpp b/Sema/SemaChecking.cpp
index 863882ec98..ea63b7dc1c 100644
--- a/Sema/SemaChecking.cpp
+++ b/Sema/SemaChecking.cpp
@@ -282,11 +282,16 @@ Sema::CheckPrintfArguments(Expr *Fn,
 
     // Seen '%'.  Now processing a format conversion.
     switch (Str[StrIdx]) {
+        // Handle dynamic precision specifier.     
+      case '*':
+        if (Str[StrIdx-1] == '.') ++numConversions;          
+        break;
+        
       // Characters which can terminate a format conversion
       // (e.g. "%d").  Characters that specify length modifiers or
       // other flags are handled by the default case below.
       //
-      // TODO: additional checks will go into the following cases.
+      // FIXME: additional checks will go into the following cases.                
       case 'i':
       case 'd':
       case 'o':