diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2015-09-18 12:27:45 +0100 |
|---|---|---|
| committer | Andras Timar <andras.timar@collabora.com> | 2015-10-17 23:22:13 +0200 |
| commit | c090d20387b0a4a335300a6df8d5c50b200d8104 (patch) | |
| tree | 0f421835beec7ca19155634808b77e24d087fc62 /sd | |
| parent | db4d1e66a8d6a90968d3a7903d5766a5366871a9 (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>
(cherry picked from commit 301391a6d1257d345ca4df19bfd35d2eefe7f9aa)
Diffstat (limited to 'sd')
| -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 |