diff options
| author | Olivier Goffart <olivier@woboq.com> | 2017-02-23 16:54:17 +0300 |
|---|---|---|
| committer | Markus Goetz <markus@woboq.com> | 2017-02-23 16:54:17 +0300 |
| commit | 7bfe0613827aa7cee93a77ccbedab9b07ea9ca78 (patch) | |
| tree | 585a4921dcf43b67495e14525905a662a4fb38ae /src/gui/sharelinkwidget.cpp | |
| parent | 13332529022b0e1f6f02ec3a62d2233fce3f8d9c (diff) | |
Verify that all strings are properly escaped (#5558)
- I checked every occurence of a '%2' and make correct use of the
QString::arg overload that takes several argument instead of chaining
them, because the first argument can contains a '%1'
- I tried to look for every label that they either use plain text or richtext
and escape the user provided strings in there.
Diffstat (limited to 'src/gui/sharelinkwidget.cpp')
| -rw-r--r-- | src/gui/sharelinkwidget.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/gui/sharelinkwidget.cpp b/src/gui/sharelinkwidget.cpp index 116cea703..456c02be7 100644 --- a/src/gui/sharelinkwidget.cpp +++ b/src/gui/sharelinkwidget.cpp @@ -321,7 +321,7 @@ void ShareLinkWidget::redrawElidedUrl() const QUrl realUrl(_shareUrl); QString elidedUrl = fm.elidedText(_shareUrl, Qt::ElideRight, linkLengthPixel); - u = QString("<a href=\"%1\">%2</a>").arg(realUrl.toString(QUrl::None)).arg(elidedUrl); + u = QString("<a href=\"%1\">%2</a>").arg(Utility::escape(realUrl.toString(QUrl::None)), Utility::escape(elidedUrl)); } _ui->_labelShareLink->setText(u); } |