New Defects reported by Coverity Scan for LibreOffice
scan-admin at coverity.com
scan-admin at coverity.com
Mon Apr 24 08:26:46 UTC 2017
Hi,
Please find the latest report on new defect(s) introduced to LibreOffice found with Coverity Scan.
8 new defect(s) introduced to LibreOffice found with Coverity Scan.
3 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan.
New defect(s) Reported-by: Coverity Scan
Showing 8 of 8 defect(s)
** CID 1405477: Uninitialized members (UNINIT_CTOR)
/cui/source/tabpages/tpgradnt.cxx: 135 in SvxGradientTabPage::SvxGradientTabPage(vcl::Window *, const SfxItemSet &)()
________________________________________________________________________________________________________
*** CID 1405477: Uninitialized members (UNINIT_CTOR)
/cui/source/tabpages/tpgradnt.cxx: 135 in SvxGradientTabPage::SvxGradientTabPage(vcl::Window *, const SfxItemSet &)()
129 m_pLbColorTo->SetSelectHdl( aLink3 );
130
131 // #i76307# always paint the preview in LTR, because this is what the document does
132 m_pCtlPreview->EnableRTL( false );
133
134 setPreviewsToSamePlace(pParent, this);
>>> CID 1405477: Uninitialized members (UNINIT_CTOR)
>>> Non-static class member "m_pPos" is not initialized in this constructor nor in any functions that it calls.
135 }
136
137 SvxGradientTabPage::~SvxGradientTabPage()
138 {
139 disposeOnce();
140 }
** CID 1405476: Uninitialized members (UNINIT_CTOR)
/cui/source/tabpages/tphatch.cxx: 130 in SvxHatchTabPage::SvxHatchTabPage(vcl::Window *, const SfxItemSet &)()
________________________________________________________________________________________________________
*** CID 1405476: Uninitialized members (UNINIT_CTOR)
/cui/source/tabpages/tphatch.cxx: 130 in SvxHatchTabPage::SvxHatchTabPage(vcl::Window *, const SfxItemSet &)()
124
125 m_pBtnAdd->SetClickHdl( LINK( this, SvxHatchTabPage, ClickAddHdl_Impl ) );
126 m_pBtnModify->SetClickHdl( LINK( this, SvxHatchTabPage, ClickModifyHdl_Impl ) );
127
128 m_pCtlPreview->SetDrawMode( GetSettings().GetStyleSettings().GetHighContrastMode() ? OUTPUT_DRAWMODE_CONTRAST : OUTPUT_DRAWMODE_COLOR );
129
>>> CID 1405476: Uninitialized members (UNINIT_CTOR)
>>> Non-static class member "m_pPos" is not initialized in this constructor nor in any functions that it calls.
130 }
131
132 SvxHatchTabPage::~SvxHatchTabPage()
133 {
134 disposeOnce();
135 }
** CID 1405475: Uninitialized members (UNINIT_CTOR)
/chart2/qa/extras/PivotChartTest.cxx: 139 in <unnamed>::Value::Value(const rtl::OUString &)()
________________________________________________________________________________________________________
*** CID 1405475: Uninitialized members (UNINIT_CTOR)
/chart2/qa/extras/PivotChartTest.cxx: 139 in <unnamed>::Value::Value(const rtl::OUString &)()
133 double mfValue;
134 bool mbIsValue;
135
136 Value(OUString const & rString)
137 : maString(rString)
138 , mbIsValue(false)
>>> CID 1405475: Uninitialized members (UNINIT_CTOR)
>>> Non-static class member "mfValue" is not initialized in this constructor nor in any functions that it calls.
139 {}
140
141 Value(double fValue)
142 : mfValue(fValue)
143 , mbIsValue(true)
144 {}
** CID 1405474: Integer handling issues (SIGN_EXTENSION)
/sw/inc/htmltbl.hxx: 429 in SwHTMLTableLayout::GetCell(unsigned short, unsigned short) const()
________________________________________________________________________________________________________
*** CID 1405474: Integer handling issues (SIGN_EXTENSION)
/sw/inc/htmltbl.hxx: 429 in SwHTMLTableLayout::GetCell(unsigned short, unsigned short) const()
423 {
424 m_aColumns[nCol] = std::move(pCol);
425 }
426
427 inline SwHTMLTableLayoutCell *SwHTMLTableLayout::GetCell( sal_uInt16 nRow, sal_uInt16 nCol ) const
428 {
>>> CID 1405474: Integer handling issues (SIGN_EXTENSION)
>>> Suspicious implicit sign extension: "this->m_nCols" with type "sal_uInt16" (16 bits, unsigned) is promoted in "nRow * this->m_nCols + nCol" to type "int" (32 bits, signed), then sign-extended to type "unsigned long" (64 bits, unsigned). If "nRow * this->m_nCols + nCol" is greater than 0x7FFFFFFF, the upper bits of the result will all be 1.
429 return m_aCells[nRow*m_nCols+nCol].get();
430 }
431
432 inline void SwHTMLTableLayout::SetCell( std::unique_ptr<SwHTMLTableLayoutCell> pCell,
433 sal_uInt16 nRow, sal_uInt16 nCol )
434 {
** CID 1405473: Integer handling issues (SIGN_EXTENSION)
/sw/inc/htmltbl.hxx: 429 in SwHTMLTableLayout::GetCell(unsigned short, unsigned short) const()
________________________________________________________________________________________________________
*** CID 1405473: Integer handling issues (SIGN_EXTENSION)
/sw/inc/htmltbl.hxx: 429 in SwHTMLTableLayout::GetCell(unsigned short, unsigned short) const()
423 {
424 m_aColumns[nCol] = std::move(pCol);
425 }
426
427 inline SwHTMLTableLayoutCell *SwHTMLTableLayout::GetCell( sal_uInt16 nRow, sal_uInt16 nCol ) const
428 {
>>> CID 1405473: Integer handling issues (SIGN_EXTENSION)
>>> Suspicious implicit sign extension: "nRow" with type "sal_uInt16" (16 bits, unsigned) is promoted in "nRow * this->m_nCols + nCol" to type "int" (32 bits, signed), then sign-extended to type "unsigned long" (64 bits, unsigned). If "nRow * this->m_nCols + nCol" is greater than 0x7FFFFFFF, the upper bits of the result will all be 1.
429 return m_aCells[nRow*m_nCols+nCol].get();
430 }
431
432 inline void SwHTMLTableLayout::SetCell( std::unique_ptr<SwHTMLTableLayoutCell> pCell,
433 sal_uInt16 nRow, sal_uInt16 nCol )
434 {
** CID 1405472: Integer handling issues (SIGN_EXTENSION)
/sw/inc/htmltbl.hxx: 435 in SwHTMLTableLayout::SetCell(std::unique_ptr<SwHTMLTableLayoutCell, std::default_delete<SwHTMLTableLayoutCell>>, unsigned short, unsigned short)()
________________________________________________________________________________________________________
*** CID 1405472: Integer handling issues (SIGN_EXTENSION)
/sw/inc/htmltbl.hxx: 435 in SwHTMLTableLayout::SetCell(std::unique_ptr<SwHTMLTableLayoutCell, std::default_delete<SwHTMLTableLayoutCell>>, unsigned short, unsigned short)()
429 return m_aCells[nRow*m_nCols+nCol].get();
430 }
431
432 inline void SwHTMLTableLayout::SetCell( std::unique_ptr<SwHTMLTableLayoutCell> pCell,
433 sal_uInt16 nRow, sal_uInt16 nCol )
434 {
>>> CID 1405472: Integer handling issues (SIGN_EXTENSION)
>>> Suspicious implicit sign extension: "this->m_nCols" with type "sal_uInt16" (16 bits, unsigned) is promoted in "nRow * this->m_nCols + nCol" to type "int" (32 bits, signed), then sign-extended to type "unsigned long" (64 bits, unsigned). If "nRow * this->m_nCols + nCol" is greater than 0x7FFFFFFF, the upper bits of the result will all be 1.
435 m_aCells[nRow*m_nCols+nCol] = std::move(pCell);
436 }
437
438 inline long SwHTMLTableLayout::GetBrowseWidthMin() const
439 {
440 return (long)( (!m_nWidthOption || m_bPrcWidthOption) ? m_nMin : m_nRelTabWidth );
** CID 1405471: Integer handling issues (SIGN_EXTENSION)
/sw/inc/htmltbl.hxx: 435 in SwHTMLTableLayout::SetCell(std::unique_ptr<SwHTMLTableLayoutCell, std::default_delete<SwHTMLTableLayoutCell>>, unsigned short, unsigned short)()
________________________________________________________________________________________________________
*** CID 1405471: Integer handling issues (SIGN_EXTENSION)
/sw/inc/htmltbl.hxx: 435 in SwHTMLTableLayout::SetCell(std::unique_ptr<SwHTMLTableLayoutCell, std::default_delete<SwHTMLTableLayoutCell>>, unsigned short, unsigned short)()
429 return m_aCells[nRow*m_nCols+nCol].get();
430 }
431
432 inline void SwHTMLTableLayout::SetCell( std::unique_ptr<SwHTMLTableLayoutCell> pCell,
433 sal_uInt16 nRow, sal_uInt16 nCol )
434 {
>>> CID 1405471: Integer handling issues (SIGN_EXTENSION)
>>> Suspicious implicit sign extension: "nRow" with type "sal_uInt16" (16 bits, unsigned) is promoted in "nRow * this->m_nCols + nCol" to type "int" (32 bits, signed), then sign-extended to type "unsigned long" (64 bits, unsigned). If "nRow * this->m_nCols + nCol" is greater than 0x7FFFFFFF, the upper bits of the result will all be 1.
435 m_aCells[nRow*m_nCols+nCol] = std::move(pCell);
436 }
437
438 inline long SwHTMLTableLayout::GetBrowseWidthMin() const
439 {
440 return (long)( (!m_nWidthOption || m_bPrcWidthOption) ? m_nMin : m_nRelTabWidth );
** CID 1405470: Null pointer dereferences (FORWARD_NULL)
/chart2/source/controller/main/ControllerCommandDispatch.cxx: 416 in chart::impl::ModelState::update(const com::sun::star::uno::Reference<com::sun::star::frame::XModel> &)()
________________________________________________________________________________________________________
*** CID 1405470: Null pointer dereferences (FORWARD_NULL)
/chart2/source/controller/main/ControllerCommandDispatch.cxx: 416 in chart::impl::ModelState::update(const com::sun::star::uno::Reference<com::sun::star::frame::XModel> &)()
410 bSupportsAxes = ChartTypeHelper::isSupportingMainAxis( xFirstChartType, nDimensionCount, 0 );
411
412 bIsThreeD = (nDimensionCount == 3);
413 if (xChartDoc.is())
414 {
415 ChartModel* pModel = dynamic_cast<ChartModel*>(xChartDoc.get());
>>> CID 1405470: Null pointer dereferences (FORWARD_NULL)
>>> Passing null pointer "pModel" to "hasInternalDataProvider", which dereferences it. (The dereference happens because this is a virtual function call.)
416 bHasOwnData = pModel->hasInternalDataProvider();
417 bHasDataFromPivotTable = pModel->isDataFromPivotTable();
418 }
419
420 bHasMainTitle = TitleHelper::getTitle( TitleHelper::MAIN_TITLE, xModel ).is();
421 bHasSubTitle = TitleHelper::getTitle( TitleHelper::SUB_TITLE, xModel ).is();
________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V05UPxvVjWch-2Bd2MGckcRZBnDJeNb0HijxaS4JNJPxk3kpyAm2AYqo71yXmnOxB72ibeUH-2F-2F1Lhi9AZq3dRu-2F4-3D_g-2BrHdvqzaBa155F-2F8AmPhpJzY63UzWDisJV95WUBpGhqFw1ICExHG8aMaV2EoFpy9Bt6qqM9a5w-2FF0bnTRcHcDEYKe8k8CFx5Vq-2B5iNTTFHFc1ufETyPQ4O-2FkUKnNJCrBsL3I6wlzAUyBHL1-2FyJbEV-2BnUYyUVKba38TtVAvSbEHwlJ8KWdBhfoeZmRReY4l0qpUbgzChUcEivIDLCceBFfw7oA11SOiDuNo10q3SA4U-3D
To manage Coverity Scan email notifications for "libreoffice at lists.freedesktop.org", click https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V05UPxvVjWch-2Bd2MGckcRbVDbis712qZDP-2FA8y06Nq4k1FZJSDV-2FTHi5VQof9xGafB4oBwGYxuHHknceo2QLpCrZ44Ciy7AqBR2QyX6OCB5N5X-2B1MAElavPQhH6nLwozJzqOkt2k8uOkYf2ZoppNa9QVe0T3fEZVQ7Kky1tOkLz_g-2BrHdvqzaBa155F-2F8AmPhpJzY63UzWDisJV95WUBpGhqFw1ICExHG8aMaV2EoFpy9Bt6qqM9a5w-2FF0bnTRcHcN345kgpxIPFrXPJsYxYluhNL-2FymW1gkmBDR54ftUgYONxOboL-2B-2BSs7D-2BJF-2BWZODX-2F32QQvs1bgQuZ2-2FIj2YNVWizKuN7gAjUczScbl-2BW9gujOZJGot2-2Fi-2BzWNynUPdh4NNLl6V-2FnhRkrIWV44tAddo-3D
More information about the LibreOffice
mailing list