diff options
author | Marc Mutz <marc.mutz@qt.io> | 2022-05-03 00:41:21 +0200 |
---|---|---|
committer | Marc Mutz <marc.mutz@qt.io> | 2022-05-04 07:09:25 +0200 |
commit | c9f4cb2c9267fd1ad0d90b1a817e7417f1ea23ff (patch) | |
tree | 018b0fcd424aae1789db73a9104ca9f98f14c287 /src/widgets | |
parent | 9b3885248b80cbb22826bcc6e339231db11aab1f (diff) |
QTeamCityLogger: fix .arg() placeholder injection
Since each .arg() call starts from scratch, a file name containing a
suitable %n would mess up the formatting of the following .arg() call.
Fix by using multiArg(), which requires to pre-format the line into a
QString, but which performs only a single-pass, so doesn't suffer from
the placeholder injection problem that plagues .arg()-chaining.
Pick-to: 6.3 6.2 5.15
Change-Id: I549527643da657fca0bea63d5e3becadac529d4b
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Sona Kurazyan <sona.kurazyan@qt.io>
Diffstat (limited to 'src/widgets')
0 files changed, 0 insertions, 0 deletions