diff options
author | Ben Wiederhake <BenWiederhake.GitHub@gmx.de> | 2021-09-09 17:06:15 +0200 |
---|---|---|
committer | Linus Groh <mail@linusgroh.de> | 2021-09-11 15:16:26 +0100 |
commit | 2e4ec891dadb49965f9de900082d2fd3e52a6577 (patch) | |
tree | c56eab743ccaa2b8e643b5dc59b71c7348080bda /Tests | |
parent | 7dfecbee4471d9e34b9a8f099479ff30f145ea4e (diff) | |
download | serenity-2e4ec891dadb49965f9de900082d2fd3e52a6577.zip |
Everywhere: Fix format-vulnerabilities
Command used:
grep -Pirn '(out|warn)ln\((?!["\)]|format,|stderr,|stdout,|output, ")' \
AK Kernel/ Tests/ Userland/
(Plus some manual reviewing.)
Let's pick ArgsParser as an example:
outln(file, m_general_help);
This will fail at runtime if the general help happens to contain braces.
Even if this transformation turns out to be unnecessary in a place or
two, this way the code is "more obviously" correct.
Diffstat (limited to 'Tests')
-rw-r--r-- | Tests/LibSQL/TestSqlStatementExecution.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Tests/LibSQL/TestSqlStatementExecution.cpp b/Tests/LibSQL/TestSqlStatementExecution.cpp index cd9a35f2d9..4f202b1270 100644 --- a/Tests/LibSQL/TestSqlStatementExecution.cpp +++ b/Tests/LibSQL/TestSqlStatementExecution.cpp @@ -24,7 +24,7 @@ RefPtr<SQL::SQLResult> execute(NonnullRefPtr<SQL::Database> database, String con auto statement = parser.next_statement(); EXPECT(!parser.has_errors()); if (parser.has_errors()) { - outln(parser.errors()[0].to_string()); + outln("{}", parser.errors()[0].to_string()); } SQL::AST::ExecutionContext context { database }; auto result = statement->execute(context); |