onlineupdate/source/update/common/updatelogging.cxx | 4 +++- svl/source/numbers/zforscan.hxx | 4 ++++ svx/source/table/tableundo.hxx | 9 +++++++++ svx/source/unodraw/unoshtxt.cxx | 2 +- sw/qa/core/data/ww8/pass/hang-3.doc |binary sw/source/filter/ww8/ww8par.cxx | 4 ++-- 6 files changed, 19 insertions(+), 4 deletions(-)
New commits: commit 0592da19b625975886d3442477f4399660736a69 Author: Caolán McNamara <caol...@redhat.com> Date: Tue Aug 11 20:58:22 2015 +0100 avoid another hang with certain .docs Change-Id: If16e90c5ba1a43ceb9702e752835928da7b3ef32 diff --git a/sw/qa/core/data/ww8/pass/hang-3.doc b/sw/qa/core/data/ww8/pass/hang-3.doc new file mode 100644 index 0000000..4188b80 Binary files /dev/null and b/sw/qa/core/data/ww8/pass/hang-3.doc differ diff --git a/sw/source/filter/ww8/ww8par.cxx b/sw/source/filter/ww8/ww8par.cxx index 96c71b7..aca1430 100644 --- a/sw/source/filter/ww8/ww8par.cxx +++ b/sw/source/filter/ww8/ww8par.cxx @@ -2287,7 +2287,7 @@ void SwWW8ImplReader::Read_HdFtText(WW8_CP nStart, WW8_CP nLen, SwFrameFormat* p bool SwWW8ImplReader::isValid_HdFt_CP(WW8_CP nHeaderCP) const { // Each CP of Plcfhdd MUST be less than FibRgLw97.ccpHdd - return (nHeaderCP < m_pWwFib->ccpHdr); + return (nHeaderCP < m_pWwFib->ccpHdr && nHeaderCP >= 0); } bool SwWW8ImplReader::HasOwnHeaderFooter(sal_uInt8 nWhichItems, sal_uInt8 grpfIhdt, @@ -2304,7 +2304,7 @@ bool SwWW8ImplReader::HasOwnHeaderFooter(sal_uInt8 nWhichItems, sal_uInt8 grpfIh { bool bOk = true; if( m_bVer67 ) - bOk = ( m_pHdFt->GetTextPos(grpfIhdt, nI, nStart, nLen ) && nLen >= 2 ); + bOk = ( m_pHdFt->GetTextPos(grpfIhdt, nI, nStart, nLen ) && nStart >= 0 && nLen >= 2 ); else { m_pHdFt->GetTextPosExact( static_cast< short >(nNumber + (nSect+1)*6), nStart, nLen); commit 7b09e2a90cd8b69b466601f70d0513cfe2862e7c Author: Caolán McNamara <caol...@redhat.com> Date: Tue Aug 11 21:01:35 2015 +0100 cppcheck: nullPointerRedundantCheck Change-Id: I8f893fe86d1f09a6ed7256830bca5599f94a187b diff --git a/svx/source/unodraw/unoshtxt.cxx b/svx/source/unodraw/unoshtxt.cxx index b6577ae..1ff2e0e 100644 --- a/svx/source/unodraw/unoshtxt.cxx +++ b/svx/source/unodraw/unoshtxt.cxx @@ -643,7 +643,7 @@ SvxTextForwarder* SvxTextEditSourceImpl::GetBackgroundTextForwarder() mpOutliner->SetText( *pOutlinerParaObject ); // #91254# put text to object and set EmptyPresObj to FALSE - if( mpText && bOwnParaObj && pOutlinerParaObject && mpObject->IsEmptyPresObj() && pTextObj->IsReallyEdited() ) + if( mpText && bOwnParaObj && mpObject->IsEmptyPresObj() && pTextObj->IsReallyEdited() ) { mpObject->SetEmptyPresObj( false ); static_cast< SdrTextObj* >( mpObject)->NbcSetOutlinerParaObjectForText( pOutlinerParaObject, mpText ); commit cf58196691785d284fcffc653626fcf135c9ba69 Author: Caolán McNamara <caol...@redhat.com> Date: Tue Aug 11 20:49:50 2015 +0100 cppcheck: uninitMemberVar Change-Id: I7c6addac04246eb17827033e7a291dd2e5600384 diff --git a/svx/source/table/tableundo.hxx b/svx/source/table/tableundo.hxx index 630068e..e18974d 100644 --- a/svx/source/table/tableundo.hxx +++ b/svx/source/table/tableundo.hxx @@ -186,6 +186,15 @@ private: bool mbIsVisible; bool mbIsStartOfNewPage; OUString maName; + + Data() + : mnColumn(0) + , mnWidth(0) + , mbOptimalWidth(false) + , mbIsVisible(false) + , mbIsStartOfNewPage(false) + { + } }; void setData( const Data& rData ); commit 3d80bb27d2416086026f4edf08ba2bff953901d6 Author: Caolán McNamara <caol...@redhat.com> Date: Tue Aug 11 20:46:52 2015 +0100 cppcheck: noCopyConstructor Change-Id: I20dabf3106d8f74fac5cc59f93a006875b9e1f91 diff --git a/svl/source/numbers/zforscan.hxx b/svl/source/numbers/zforscan.hxx index 54eec03..5f51eda 100644 --- a/svl/source/numbers/zforscan.hxx +++ b/svl/source/numbers/zforscan.hxx @@ -194,6 +194,10 @@ private: // Private section sal_uInt8 nNatNumModifier; // Thai T speciality + // Copy assignment is forbidden and not implemented. + ImpSvNumberformatScan (const ImpSvNumberformatScan &) SAL_DELETED_FUNCTION; + ImpSvNumberformatScan & operator= (const ImpSvNumberformatScan &) SAL_DELETED_FUNCTION; + void InitKeywords() const; void InitSpecialKeyword( NfKeywordIndex eIdx ) const; void InitCompatCur() const; commit a2c6c2e50054ff6cee70cff936d3682a5b0f5a55 Author: Caolán McNamara <caol...@redhat.com> Date: Tue Aug 11 20:44:19 2015 +0100 cppcheck: uninitMemberVar Change-Id: I78acdab89764a27291ca05b0bb1d448608202208 diff --git a/onlineupdate/source/update/common/updatelogging.cxx b/onlineupdate/source/update/common/updatelogging.cxx index 4044e26..2f1472e 100644 --- a/onlineupdate/source/update/common/updatelogging.cxx +++ b/onlineupdate/source/update/common/updatelogging.cxx @@ -14,7 +14,9 @@ #include "updatelogging.h" -UpdateLog::UpdateLog() : logFP(nullptr) +UpdateLog::UpdateLog() + : logFP(nullptr) + , sourcePath(nullptr) { }
_______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits