================
@@ -1175,7 +1181,7 @@ static bool hasUnsafePrintfStringArg(const CallExpr 
&Node, ASTContext &Ctx,
 
     if (auto *II = FD->getIdentifier())
       isKprintf = II->getName() == "kprintf";
-    if (hasUnsafeFormatOrSArg(&Node, UnsafeArg, 0, Ctx, isKprintf)) {
+    if (hasUnsafeFormatOrSArg(Ctx, &Node, UnsafeArg, 0, -1, isKprintf)) {
----------------
ojhunt wrote:

```suggestion
    if (hasUnsafeFormatOrSArg(Ctx, &Node, UnsafeArg, /*FmtIdx=*/0, 
/*FmtArgIdx=*/std::nullopt, isKprintf)) {
```

I'll stop commenting on including the parameter name, but every call that 
you're updating should correct that :D

https://github.com/llvm/llvm-project/pull/173096
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to