================
@@ -1161,7 +1167,7 @@ static bool hasUnsafePrintfStringArg(const CallExpr 
&Node, ASTContext &Ctx,
     // It is a fprintf:
     const Expr *UnsafeArg;
 
-    if (hasUnsafeFormatOrSArg(&Node, UnsafeArg, 1, Ctx, false)) {
+    if (hasUnsafeFormatOrSArg(Ctx, &Node, UnsafeArg, 1)) {
----------------
ojhunt wrote:

Given you're changing this line you should update the FmtIdx argument to 
include the parameter name:
```suggestion
    if (hasUnsafeFormatOrSArg(Ctx, &Node, UnsafeArg, /*FmtIdx=*/1)) {
```

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