diff options
author | Volker Hilsheimer <volker.hilsheimer@qt.io> | 2021-02-23 14:05:46 +0100 |
---|---|---|
committer | Volker Hilsheimer <volker.hilsheimer@qt.io> | 2021-02-23 15:15:45 +0100 |
commit | 4f24e1e9b1230b97a45c6aab366fecd29795e885 (patch) | |
tree | 7b9fe81f324066568b6d19409cb168363e468595 /src | |
parent | a944de7742176dfa6a94b171a2f72b551a164d29 (diff) |
Initialize out-variables
QSplitterPrivate::getRange might return early, and then the variables
will contain garbage.
Fixes warning from clang static analyzer.
Pick-to: 6.1
Change-Id: I0081ad9847f158da4440b945ba2db7e7f5d4780b
Reviewed-by: Richard Moe Gustavsen <richard.gustavsen@qt.io>
Diffstat (limited to 'src')
-rw-r--r-- | src/widgets/widgets/qsplitter.cpp | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/src/widgets/widgets/qsplitter.cpp b/src/widgets/widgets/qsplitter.cpp index 619b3788c8..1c04f02422 100644 --- a/src/widgets/widgets/qsplitter.cpp +++ b/src/widgets/widgets/qsplitter.cpp @@ -1404,10 +1404,10 @@ void QSplitter::moveSplitter(int pos, int index) { Q_D(QSplitter); QSplitterLayoutStruct *s = d->list.at(index); - int farMin; - int min; - int max; - int farMax; + int farMin = 0; + int min = 0; + int max = 0; + int farMax = 0; #ifdef QSPLITTER_DEBUG int debugp = pos; @@ -1471,7 +1471,10 @@ void QSplitter::getRange(int index, int *min, int *max) const int QSplitter::closestLegalPosition(int pos, int index) { Q_D(QSplitter); - int x, i, n, u; + int x = 0; + int i = 0; + int n = 0; + int u = 0; return d->adjustPos(pos, index, &u, &n, &i, &x); } |