diff options
author | Marc Mutz <marc.mutz@kdab.com> | 2017-06-06 18:11:27 +0200 |
---|---|---|
committer | Marc Mutz <marc.mutz@kdab.com> | 2017-06-08 07:39:48 +0000 |
commit | 0793dd90d674d0523998750d8dff4bc406780b2f (patch) | |
tree | 32d87b853d370e2b2f2faffaf5d803e0a826277d /src/gui/kernel/qkeysequence.cpp | |
parent | df32c9a1127cfec15aab834cc2df363fa49ca5bd (diff) |
QKeySequence: fix size mismatch
'accel' is 'str.toLower()' and as such may have a different size, so don't
use str.size() to index into 'accel'.
Change-Id: I6a140ded45ecedd811b9618e1facb63d522eb235
Reviewed-by: Lars Knoll <lars.knoll@qt.io>
Reviewed-by: Olivier Goffart (Woboq GmbH) <ogoffart@woboq.com>
Diffstat (limited to 'src/gui/kernel/qkeysequence.cpp')
-rw-r--r-- | src/gui/kernel/qkeysequence.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/gui/kernel/qkeysequence.cpp b/src/gui/kernel/qkeysequence.cpp index 20eac2ed15..01a308372e 100644 --- a/src/gui/kernel/qkeysequence.cpp +++ b/src/gui/kernel/qkeysequence.cpp @@ -1153,7 +1153,7 @@ int QKeySequencePrivate::decodeString(const QString &str, QKeySequence::Sequence lastI = i + 1; } - int p = accel.lastIndexOf(QLatin1Char('+'), str.length() - 2); // -2 so that Ctrl++ works + int p = accel.lastIndexOf(QLatin1Char('+'), accel.length() - 2); // -2 so that Ctrl++ works QStringRef accelRef(&accel); if(p > 0) accelRef = accelRef.mid(p + 1); |