sw/source/filter/ww8/ww8par6.cxx | 31 +++++++++++++++++++------------
sw/source/filter/ww8/ww8struc.hxx | 13 +++++++++++--
2 files changed, 30 insertions(+), 14 deletions(-)
New commits:
commit 4d145c272fa00a7425b1b1a9ef9f35719b905d8b
Author: Jürgen Schmidt <j...@apache.org>
Date: Fri Mar 28 10:45:27 2014 +0000
Resolves: #i124468# add checks for the read numbers of column for a section
note acecfab9dde94abe733d63260777007337b9aff0 covers the same territory
and fixes mostly the same problem, but ccolM1 isn't clipped for the
other places where it gets used
(cherry picked from commit 32d9f959cf3f133b9257c066a979848a967f7081)
Conflicts:
sw/source/filter/ww8/ww8par6.cxx
Change-Id: Idad7d95b28d38b7c52b56cab16c533565d9ecfea
diff --git a/sw/source/filter/ww8/ww8par6.cxx b/sw/source/filter/ww8/ww8par6.cxx
index 3677fd7..73d2f24 100644
--- a/sw/source/filter/ww8/ww8par6.cxx
+++ b/sw/source/filter/ww8/ww8par6.cxx
@@ -884,7 +884,14 @@ void wwSectionManager::CreateSep(const long nTxtPos, bool
/*bMustHaveBreak*/)
// sprmSFBiDi
aNewSection.maSep.fBiDi = eVer >= ww::eWW8 ? ReadBSprm(pSep, 0x3228, 0) :
0;
+ // Reading section property sprmSCcolumns - one less than the number of
columns in the section.
+ // It must be less than MAX_NO_OF_SEP_COLUMNS according the WW8
specification.
aNewSection.maSep.ccolM1 = ReadSprm(pSep, pIds[3], 0 );
+ if ( aNewSection.maSep.ccolM1 >= MAX_NO_OF_SEP_COLUMNS )
+ {
+ // fallback to one column
+ aNewSection.maSep.ccolM1 = 0;
+ }
//sprmSDxaColumns - Default-Abstand 1.25 cm
aNewSection.maSep.dxaColumns = ReadUSprm( pSep, pIds[4], 708 );
@@ -898,34 +905,34 @@ void wwSectionManager::CreateSep(const long nTxtPos, bool
/*bMustHaveBreak*/)
aNewSection.maSep.fEvenlySpaced =
sal_uInt8(ReadBSprm(pSep, (eVer <= ww::eWW7 ? 138 : 0x3005), 1) ?
true : false);
- const sal_uInt8 numrgda =
SAL_N_ELEMENTS(aNewSection.maSep.rgdxaColumnWidthSpacing);
if (aNewSection.maSep.ccolM1 > 0 && !aNewSection.maSep.fEvenlySpaced)
{
- aNewSection.maSep.rgdxaColumnWidthSpacing[0] = 0;
- int nCols = aNewSection.maSep.ccolM1 + 1;
- int nIdx = 0;
- for (int i = 0; i < nCols; ++i)
+ int nColumnDataIdx = 0;
+ aNewSection.maSep.rgdxaColumnWidthSpacing[nColumnDataIdx] = 0;
+
+ const sal_uInt16 nColumnWidthSprmId = ( eVer <= ww::eWW7 ? 136 :
0xF203 );
+ const sal_uInt16 nColumnSpacingSprmId = ( eVer <= ww::eWW7 ? 137 :
0xF204 );
+ const sal_uInt8 nColumnCount = static_cast< sal_uInt8
>(aNewSection.maSep.ccolM1 + 1);
+ for ( sal_uInt8 nColumn = 0; nColumn < nColumnCount; ++nColumn )
{
//sprmSDxaColWidth
- const sal_uInt8* pSW = pSep->HasSprm( (eVer <= ww::eWW7 ? 136
: 0xF203), sal_uInt8( i ) );
+ const sal_uInt8* pSW = pSep->HasSprm( nColumnWidthSprmId,
nColumn );
OSL_ENSURE( pSW, "+Sprm 136 (bzw. 0xF203) (ColWidth) fehlt" );
sal_uInt16 nWidth = pSW ? SVBT16ToShort(pSW + 1) : 1440;
- if (++nIdx < numrgda)
- aNewSection.maSep.rgdxaColumnWidthSpacing[nIdx] = nWidth;
+ aNewSection.maSep.rgdxaColumnWidthSpacing[++nColumnDataIdx] =
nWidth;
- if (i < nCols-1)
+ if ( nColumn < nColumnCount - 1 )
{
//sprmSDxaColSpacing
- const sal_uInt8* pSD = pSep->HasSprm( (eVer <= ww::eWW7 ?
137 : 0xF204), sal_uInt8( i ) );
+ const sal_uInt8* pSD = pSep->HasSprm(
nColumnSpacingSprmId, nColumn );
OSL_ENSURE( pSD, "+Sprm 137 (bzw. 0xF204) (Colspacing)
fehlt" );
if( pSD )
{
nWidth = SVBT16ToShort(pSD + 1);
- if (++nIdx < numrgda)
- aNewSection.maSep.rgdxaColumnWidthSpacing[nIdx] =
nWidth;
+
aNewSection.maSep.rgdxaColumnWidthSpacing[++nColumnDataIdx] = nWidth;
}
}
}
diff --git a/sw/source/filter/ww8/ww8struc.hxx
b/sw/source/filter/ww8/ww8struc.hxx
index a19b326..98e0f20 100644
--- a/sw/source/filter/ww8/ww8struc.hxx
+++ b/sw/source/filter/ww8/ww8struc.hxx
@@ -1005,6 +1005,9 @@ struct WW8_WKB
# pragma pack(pop)
#endif
+// Maximum number of columns according the WW8 specification
+static const sal_uInt8 MAX_NO_OF_SEP_COLUMNS = 44;
+
struct SEPr
{
SEPr();
@@ -1064,7 +1067,7 @@ struct SEPr
sal_uInt32 dzaGutter;
sal_uInt32 dyaHdrTop;
sal_uInt32 dyaHdrBottom;
- sal_Int16 ccolM1;
+ sal_Int16 ccolM1; // have to be less than MAX_NO_OF_SEP_COLUMNS
according the WW8 specification
sal_Int8 fEvenlySpaced;
sal_Int8 reserved3;
sal_uInt8 fBiDi;
@@ -1072,7 +1075,13 @@ struct SEPr
sal_uInt8 fRTLGutter;
sal_uInt8 fRTLAlignment;
sal_Int32 dxaColumns;
- sal_Int32 rgdxaColumnWidthSpacing[89];
+
+ // Fixed array - two entries for each SEP column to store width of column
and spacing to next column.
+ // At odd index values [1,3,5,...] the column widths are stored.
+ // At even index values [2,4,6,...] the spacings to the next columns are
stored.
+ // Value at index 0 is initialized with 0 and used for easier interation
on the array
+ sal_Int32 rgdxaColumnWidthSpacing[MAX_NO_OF_SEP_COLUMNS*2 + 1];
+
sal_Int32 dxaColumnWidth;
sal_uInt8 dmOrientFirst;
sal_uInt8 fLayout;
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
