[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-5.3' - 6 commits - sc/qa svtools/source sw/qa sw/source vcl/win xmloff/source
Michael Stahl
mstahl at redhat.com
Tue Sep 26 08:38:09 UTC 2017
sc/qa/unit/subsequent_export-test.cxx | 22 ++++++++++++++++++++++
svtools/source/contnr/treelistbox.cxx | 2 ++
sw/qa/extras/ooxmlexport/ooxmlexport3.cxx | 5 ++++-
sw/source/filter/ww8/ww8scan.cxx | 3 +--
vcl/win/gdi/salfont.cxx | 6 +++---
xmloff/source/core/xmlexp.cxx | 2 +-
xmloff/source/draw/shapeexport.cxx | 22 +++++++++++++++-------
7 files changed, 48 insertions(+), 14 deletions(-)
New commits:
commit ec83f778ab97bb7089425d95cb138797ac9d01b7
Author: Michael Stahl <mstahl at redhat.com>
Date: Tue Sep 19 21:43:23 2017 +0200
xmloff: fix stack-use-after-return in SvXMLExport::exportDoc()
PropertySetInfo(PropertyMapEntry const * pMap) stores the pointers,
and its lifetime is controlled by refcounting, so the lifetime
of aInfoMap can't be automatic.
ERROR: AddressSanitizer: stack-use-after-return on address ...
... is located in stack of thread T46
at offset 224 in frame SvXMLExport::exportDoc(xmloff::token::XMLTokenEnum)
This frame has 39 object(s):
...
[224, 288) 'aInfoMap' <== Memory access at offset 224 is inside this variable
Change-Id: I4eaa9d38bab708b222d999b0982100d7ef97e95c
(cherry picked from commit ee599ea46365adc37f4d495d9ff9778c25c04c92)
Reviewed-on: https://gerrit.libreoffice.org/42505
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm at redhat.com>
Tested-by: Caolán McNamara <caolanm at redhat.com>
(cherry picked from commit e68c82d2c194aafc6406f9f27caa2d1e70a7614e)
diff --git a/xmloff/source/core/xmlexp.cxx b/xmloff/source/core/xmlexp.cxx
index aced8d37b245..cb12d33b11d5 100644
--- a/xmloff/source/core/xmlexp.cxx
+++ b/xmloff/source/core/xmlexp.cxx
@@ -1296,7 +1296,7 @@ sal_uInt32 SvXMLExport::exportDoc( enum ::xmloff::token::XMLTokenEnum eClass )
{
try
{
- ::comphelper::PropertyMapEntry const aInfoMap[] =
+ static ::comphelper::PropertyMapEntry const aInfoMap[] =
{
{ OUString("Class"), 0,
::cppu::UnoType<OUString>::get(),
commit 2c0b882fdb640cf96f56eaced234792916a52c90
Author: Michael Stahl <mstahl at redhat.com>
Date: Wed Sep 20 17:45:14 2017 +0200
tdf#112005 xmloff: ODF export: export embedded object text *first*
... before adding any attributes, so that the attributes don't
erroneously get added to the paragraph element.
Also fix the problem that if you set "ODF format version" to 1.1 or 1.2,
export asserts in SvXMLNamespaceMap::GetQNameByKey() because the "loext"
namespace isn't registered, and creates an invalid <p> element.
Change-Id: I637f9d3ff746d877ced480e35ef53f4545a06a4b
(cherry picked from commit 86f256596c8566e80993e1cf6035bc3534b6f816)
Reviewed-on: https://gerrit.libreoffice.org/42569
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm at redhat.com>
Tested-by: Caolán McNamara <caolanm at redhat.com>
(cherry picked from commit 65c99825ceb08743a70b6e598f65491a67d9a884)
diff --git a/sc/qa/unit/subsequent_export-test.cxx b/sc/qa/unit/subsequent_export-test.cxx
index cb7f563a6947..9466fbe8f28d 100644
--- a/sc/qa/unit/subsequent_export-test.cxx
+++ b/sc/qa/unit/subsequent_export-test.cxx
@@ -126,6 +126,7 @@ public:
void testLandscapeOrientationXLSX();
void testInlineArrayXLS();
+ void testEmbeddedChartODS();
void testEmbeddedChartXLS();
void testCellAnchoredGroupXLS();
@@ -241,6 +242,7 @@ public:
CPPUNIT_TEST(testAllRowsHiddenXLSX);
CPPUNIT_TEST(testLandscapeOrientationXLSX);
CPPUNIT_TEST(testInlineArrayXLS);
+ CPPUNIT_TEST(testEmbeddedChartODS);
CPPUNIT_TEST(testEmbeddedChartXLS);
CPPUNIT_TEST(testCellAnchoredGroupXLS);
@@ -338,6 +340,8 @@ void ScExportTest::registerNamespaces(xmlXPathContextPtr& pXmlXPathCtx)
{ BAD_CAST("office"), BAD_CAST("urn:oasis:names:tc:opendocument:xmlns:office:1.0") },
{ BAD_CAST("table"), BAD_CAST("urn:oasis:names:tc:opendocument:xmlns:table:1.0") },
{ BAD_CAST("text"), BAD_CAST("urn:oasis:names:tc:opendocument:xmlns:text:1.0") },
+ { BAD_CAST("style"), BAD_CAST("urn:oasis:names:tc:opendocument:xmlns:style:1.0") },
+ { BAD_CAST("draw"), BAD_CAST("urn:oasis:names:tc:opendocument:xmlns:drawing:1.0") },
{ BAD_CAST("xlink"), BAD_CAST("http://www.w3c.org/1999/xlink") },
{ BAD_CAST("xdr"), BAD_CAST("http://schemas.openxmlformats.org/drawingml/2006/spreadsheetDrawing") },
{ BAD_CAST("x"), BAD_CAST("http://schemas.openxmlformats.org/spreadsheetml/2006/main") },
@@ -2016,6 +2020,24 @@ void ScExportTest::testInlineArrayXLS()
xDocSh->DoClose();
}
+void ScExportTest::testEmbeddedChartODS()
+{
+ ScDocShellRef xShell = loadDoc("embedded-chart.", FORMAT_XLS);
+ CPPUNIT_ASSERT(xShell.Is());
+
+ std::shared_ptr<utl::TempFile> pTempFile(
+ ScBootstrapFixture::exportTo(xShell.get(), FORMAT_ODS));
+
+ xmlDocPtr pDoc = XPathHelper::parseExport(pTempFile, m_xSFactory, "content.xml");
+ CPPUNIT_ASSERT(pDoc);
+ assertXPath(pDoc,
+ "/office:document-content/office:body/office:spreadsheet/table:table[2]/table:table-row[7]/table:table-cell[2]/draw:frame/draw:object",
+ "notify-on-update-of-ranges",
+ "Chart1.B3:Chart1.B5 Chart1.C2:Chart1.C2 Chart1.C3:Chart1.C5");
+
+ xShell->DoClose();
+}
+
void ScExportTest::testEmbeddedChartXLS()
{
ScDocShellRef xShell = loadDoc("embedded-chart.", FORMAT_XLS);
diff --git a/xmloff/source/draw/shapeexport.cxx b/xmloff/source/draw/shapeexport.cxx
index 6d8f0185c249..e4ec98689841 100644
--- a/xmloff/source/draw/shapeexport.cxx
+++ b/xmloff/source/draw/shapeexport.cxx
@@ -1514,6 +1514,13 @@ bool XMLShapeExport::ImpExportPresentationAttributes( const uno::Reference< bean
void XMLShapeExport::ImpExportText( const uno::Reference< drawing::XShape >& xShape, TextPNS eExtensionNS )
{
+ if (eExtensionNS == TextPNS::EXTENSION)
+ {
+ if (mrExport.getDefaultVersion() <= SvtSaveOptions::ODFVER_012)
+ {
+ return; // do not export to ODF 1.1/1.2
+ }
+ }
uno::Reference< text::XText > xText( xShape, uno::UNO_QUERY );
if( xText.is() )
{
@@ -2780,6 +2787,14 @@ void XMLShapeExport::ImpExportOLE2Shape(
if( !bIsEmptyPresObj || bSaveBackwardsCompatible )
{
+ // tdf#112005 export text *before* adding any attributes
+ if (!bIsEmptyPresObj && supportsText(eShapeType))
+ {
+ // #i118485# Add text export, the draw OLE shape allows text now
+ // fdo#58571 chart objects don't allow text:p
+ ImpExportText( xShape, TextPNS::EXTENSION );
+ }
+
if (pAttrList)
{
mrExport.AddAttributeList(pAttrList);
@@ -2815,13 +2830,6 @@ void XMLShapeExport::ImpExportOLE2Shape(
if( !sClassId.isEmpty() )
mrExport.AddAttribute(XML_NAMESPACE_DRAW, XML_CLASS_ID, sClassId );
- if(supportsText(eShapeType))
- {
- // #i118485# Add text export, the draw OLE shape allows text now
- // fdo#58571 chart objects don't allow text:p
- ImpExportText( xShape, TextPNS::EXTENSION );
- }
-
if(!bExportEmbedded)
{
// xlink:href
commit c87e63fdb084dfff44c765993680fe343050ee88
Author: Tor Lillqvist <tml at collabora.com>
Date: Thu Apr 13 17:36:24 2017 +0300
Make the testTdf106974_int32Crop pass on my Mac
For some reason, on my Mac the Right value is 41226, both times the
function gets called. (On my Linux box, it is 46387 and 46394.) So
compare to 41000 instead of 46000.
Or would it be better to just bypass the test on macOS, as we already
do for other tests in this file...?
(cherry picked from commit 1391f3702a3daaefc67a8ee4b62ac38959db283f)
Make the testTdf106974_int32Crop pass on my Mac, too
cf. 1391f3702a3daaefc67a8ee4b62ac38959db283f "Make the testTdf106974_int32Crop
pass on my Mac". For some reason, on my Mac the Right value is 40474 resp.
40471 the two times the function gets called.
(cherry picked from commit a900c72c675c8605fc004a4f63ba1d82eaeed9d5)
Change-Id: I731ff9c5cf76be9d4817ad14f296807017d10dbd
Reviewed-on: https://gerrit.libreoffice.org/42642
Reviewed-by: Michael Stahl <mstahl at redhat.com>
Tested-by: Jenkins <ci at libreoffice.org>
(cherry picked from commit b012ddbf39eaa51a05aec590f35e2f4d46cddd57)
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport3.cxx b/sw/qa/extras/ooxmlexport/ooxmlexport3.cxx
index 151b1cc65c53..bb33e40c003a 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport3.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport3.cxx
@@ -582,7 +582,10 @@ DECLARE_OOXMLEXPORT_TEST(testTdf106974_int32Crop, "tdf106974_int32Crop.docx")
css::text::GraphicCrop aGraphicCropStruct;
imageProperties->getPropertyValue( "GraphicCrop" ) >>= aGraphicCropStruct;
- CPPUNIT_ASSERT( sal_Int32( 46000 ) < aGraphicCropStruct.Right );
+
+ CPPUNIT_ASSERT_MESSAGE(
+ OString::number(aGraphicCropStruct.Right).getStr(),
+ sal_Int32( 40470 ) < aGraphicCropStruct.Right );
}
DECLARE_OOXMLEXPORT_TEST(testLineSpacingexport, "test_line_spacing.docx")
commit e2bc6d9201c7d3733df8336609545b8deddccbe1
Author: Caolán McNamara <caolanm at redhat.com>
Date: Fri Sep 22 09:37:43 2017 +0100
ofz#3457 Unknown Read
similar to
commit 35bac83ed2b5d48233c653cc7dc4eab5c234f7ac
Date: Mon Aug 7 19:24:37 2017 +0100
ofz#2877: crash in SVTB16Short
Change-Id: Idc2ca89647a5e17484effaca49afce349b98f0a3
Reviewed-on: https://gerrit.libreoffice.org/42624
Reviewed-by: Michael Stahl <mstahl at redhat.com>
Tested-by: Michael Stahl <mstahl at redhat.com>
(cherry picked from commit 46882bb61b1381dd257cd96a255ed71f7af03166)
diff --git a/sw/source/filter/ww8/ww8scan.cxx b/sw/source/filter/ww8/ww8scan.cxx
index 4d138aa6f5f7..82e56f82bca9 100644
--- a/sw/source/filter/ww8/ww8scan.cxx
+++ b/sw/source/filter/ww8/ww8scan.cxx
@@ -3540,8 +3540,7 @@ const sal_uInt8* WW8PLCFx_SEPX::HasSprm( sal_uInt16 nId, sal_uInt8 n2nd ) const
return nullptr;
sal_uInt8* pSp = pSprms;
-
- sal_uInt16 i=0;
+ size_t i = 0;
while (i + maSprmParser.MinSprmLen() <= nSprmSiz)
{
// Sprm found?
commit e028c4e5d7d00536dfdba81a3ca35e7a39b2da5e
Author: Caolán McNamara <caolanm at redhat.com>
Date: Tue Sep 5 13:06:20 2017 +0100
Resolves: tdf#112180: avoid crash with specific ttf
Change-Id: I8cde147279173bffec0c991eb7676f5d4641138d
Reviewed-on: https://gerrit.libreoffice.org/41935
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm at redhat.com>
Tested-by: Caolán McNamara <caolanm at redhat.com>
(cherry picked from commit f263692de96ac68e73eeb953b7e92a18d149f30e)
Reviewed-on: https://gerrit.libreoffice.org/42507
(cherry picked from commit 70f8b4b9b0330b9150c5d6c3f066834f20023578)
diff --git a/vcl/win/gdi/salfont.cxx b/vcl/win/gdi/salfont.cxx
index 8c120c00881f..cb9899a303af 100644
--- a/vcl/win/gdi/salfont.cxx
+++ b/vcl/win/gdi/salfont.cxx
@@ -90,9 +90,9 @@ RawFontData::RawFontData( HDC hDC, DWORD nTableTag )
{
// get required size in bytes
mnByteCount = ::GetFontData( hDC, nTableTag, 0, nullptr, 0 );
- if( mnByteCount == GDI_ERROR )
- return;
- else if( !mnByteCount )
+ if (mnByteCount == GDI_ERROR)
+ mnByteCount = 0;
+ if (!mnByteCount)
return;
// allocate the array
commit c9c1cdb332796c6601ef713553101ea5c7b086e7
Author: Jim Raykowski <raykowj at gmail.com>
Date: Sat Sep 16 00:18:56 2017 -0800
tdf#49634 Fix positioning of focus rectangle in Navigator content tree
https://bugs.documentfoundation.org/show_bug.cgi?id=49634
Change-Id: I3beb41142281b2cda337b457ed287a2cc1f665a6
diff --git a/svtools/source/contnr/treelistbox.cxx b/svtools/source/contnr/treelistbox.cxx
index 14baee04e1ec..0ee0c866b67c 100644
--- a/svtools/source/contnr/treelistbox.cxx
+++ b/svtools/source/contnr/treelistbox.cxx
@@ -3001,6 +3001,8 @@ void SvTreeListBox::PreparePaint(vcl::RenderContext& /*rRenderContext*/, SvTreeL
Rectangle SvTreeListBox::GetFocusRect( SvTreeListEntry* pEntry, long nLine )
{
+ pImpl->UpdateContextBmpWidthMax( pEntry );
+
Size aSize;
Rectangle aRect;
aRect.Top() = nLine;
More information about the Libreoffice-commits
mailing list