diff options
author | Caolán McNamara <caolanm@redhat.com> | 2014-12-15 09:12:04 +0000 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2014-12-15 09:33:08 +0000 |
commit | 3c4d0c4904867444454986b7d9aa0af64646572a (patch) | |
tree | 034ae8f17574abffbdd46971910cf9b7a347d324 /sc | |
parent | f91e577ad81da79f0d8a92e02b9bdddce6dec85c (diff) |
coverity#1242939 Untrusted loop bound
Change-Id: Id76a90cf61023fb2dbc276d54ab6f0ec63e25f9f
Diffstat (limited to 'sc')
-rw-r--r-- | sc/source/filter/starcalc/scflt.cxx | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sc/source/filter/starcalc/scflt.cxx b/sc/source/filter/starcalc/scflt.cxx index 03557014c1b6..19ee464f3dc3 100644 --- a/sc/source/filter/starcalc/scflt.cxx +++ b/sc/source/filter/starcalc/scflt.cxx @@ -2311,7 +2311,8 @@ void Sc10Import::LoadObjects() double nPPTY = ScGlobal::nScreenPPTY; long nStartX = 0; - for (SCsCOL nX=0; nX<GraphHeader.CarretX; nX++) + SCCOL nMaxCol = SanitizeCol(GraphHeader.CarretX); + for (SCCOL nX = 0; nX < nMaxCol; ++nX) nStartX += pDoc->GetColWidth(nX, static_cast<SCTAB>(GraphHeader.CarretZ)); nStartX = (long) ( nStartX * HMM_PER_TWIPS ); nStartX += (long) ( GraphHeader.x / nPPTX * HMM_PER_TWIPS ); |