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-02 14:58:29 +0200 |
| commit | c82283fde7f66f7480d671423aec70f5de3d3119 (patch) | |
| tree | febf15e8b5b349d3e61b716b1a0105dab0ab4357 | |
| parent | 1e1a654a4adf112910fbc6756a9435792cbbf78f (diff) | |
reusing i here from the outer loop is definitely bogus
(cherry picked from commit 637ad743bb74a826ddd4db5efbcd33779b9132cd)
Change-Id: I4ee52c76b2c1723639185bc99b85d83be5b4fdd7
Reviewed-on: https://gerrit.libreoffice.org/18687
Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>
| -rw-r--r-- | sd/source/filter/ppt/propread.cxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sd/source/filter/ppt/propread.cxx b/sd/source/filter/ppt/propread.cxx index 08987b58f0d5..f73d26fbaa3f 100644 --- a/sd/source/filter/ppt/propread.cxx +++ b/sd/source/filter/ppt/propread.cxx @@ -301,9 +301,9 @@ bool Section::GetDictionary( Dictionary& rDict ) { nSize >>= 1; aStream.Seek( nPos ); - sal_Unicode* pWString = (sal_Unicode*)pString; - for ( i = 0; i < nSize; i++ ) - aStream.ReadUInt16( pWString[ i ] ); + sal_Unicode* pWString = reinterpret_cast<sal_Unicode*>(pString); + for (sal_uInt32 j = 0; j < nSize; ++j) + aStream.ReadUInt16(pWString[j]); aString = OUString(pWString, lcl_getMaxSafeStrLen(nSize)); } else |