================
@@ -1406,9 +1406,17 @@ void
Sema::checkFortifiedBuiltinMemoryFunction(FunctionDecl *FD,
}
case Builtin::BIsprintf:
- case Builtin::BI__builtin___sprintf_chk: {
- size_t FormatIndex = BuiltinID == Builtin::BIsprintf ? 1 : 3;
- auto *FormatExpr = TheCall->getArg(FormatIndex)->IgnoreParenImpCasts();
+ case Builtin::BI__builtin_sprintf:
+ case Builtin::BI__builtin___sprintf_chk:
+ case Builtin::BIvsprintf:
+ case Builtin::BI__builtin_vsprintf:
+ case Builtin::BI__builtin___vsprintf_chk: {
+ bool IsChk = BuiltinID == Builtin::BI__builtin___sprintf_chk ||
+ BuiltinID == Builtin::BI__builtin___vsprintf_chk;
+ std::optional<unsigned> FormatIndex = Checker.TranslateIndex(IsChk ? 3 :
1);
----------------
gburgessiv wrote:
nice catch - probably worth a short note in the squashed commit message that
we're fixing a latent bug here
https://github.com/llvm/llvm-project/pull/230672
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
