diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2015-09-18 12:27:45 +0100 |
|---|---|---|
| committer | Andras Timar <andras.timar@collabora.com> | 2015-09-19 21:32:24 +0200 |
| commit | beacd1efacf0d75cd7f404c9e690fa639202d5c6 (patch) | |
| tree | 0e6f10bf9d956397c7c4dcf1ac795e907055128f | |
| parent | b9d790e3ca99064dcdd38d6d40255f531dfc32df (diff) | |
reusing i here from the outer loop is definitely bogus
Change-Id: I4ee52c76b2c1723639185bc99b85d83be5b4fdd7
(cherry picked from commit 637ad743bb74a826ddd4db5efbcd33779b9132cd)
Reviewed-on: https://gerrit.libreoffice.org/18686
Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>
| -rw-r--r-- | sd/source/filter/ppt/propread.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sd/source/filter/ppt/propread.cxx b/sd/source/filter/ppt/propread.cxx index ca9cbe0dc7a1..e4dc401dbab3 100644 --- a/sd/source/filter/ppt/propread.cxx +++ b/sd/source/filter/ppt/propread.cxx @@ -302,8 +302,8 @@ bool Section::GetDictionary( Dictionary& rDict ) nSize >>= 1; aStream.Seek( nPos ); sal_Unicode* pWString = reinterpret_cast<sal_Unicode*>(pString); - for ( i = 0; i < nSize; i++ ) - aStream.ReadUInt16( pWString[ i ] ); + for (sal_uInt32 j = 0; j < nSize; ++j) + aStream.ReadUInt16(pWString[j]); aString = OUString(pWString, lcl_getMaxSafeStrLen(nSize)); } else |