New Defects reported by Coverity Scan for LibreOffice
scan-admin at coverity.com
scan-admin at coverity.com
Thu Jun 2 13:11:15 UTC 2016
Hi,
Please find the latest report on new defect(s) introduced to LibreOffice found with Coverity Scan.
9 new defect(s) introduced to LibreOffice found with Coverity Scan.
5 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 9 of 9 defect(s)
** CID 1362486: Null pointer dereferences (FORWARD_NULL)
/dbaccess/source/ui/dlg/dbwiz.cxx: 92 in dbaui::ODbTypeWizDialog::ODbTypeWizDialog(vcl::Window *, SfxItemSet *, const com::sun::star::uno::Reference<com::sun::star::uno::XComponentContext> &, const com::sun::star::uno::Any &)()
________________________________________________________________________________________________________
*** CID 1362486: Null pointer dereferences (FORWARD_NULL)
/dbaccess/source/ui/dlg/dbwiz.cxx: 92 in dbaui::ODbTypeWizDialog::ODbTypeWizDialog(vcl::Window *, SfxItemSet *, const com::sun::star::uno::Reference<com::sun::star::uno::XComponentContext> &, const com::sun::star::uno::Any &)()
86 m_pPrevPage->SetHelpId(HID_DBWIZ_PREVIOUS);
87 m_pNextPage->SetHelpId(HID_DBWIZ_NEXT);
88 m_pCancel->SetHelpId(HID_DBWIZ_CANCEL);
89 m_pFinish->SetHelpId(HID_DBWIZ_FINISH);
90 // no local resources needed anymore
91
>>> CID 1362486: Null pointer dereferences (FORWARD_NULL)
>>> Assigning: "pCollectionItem" = "dynamic_cast <dbaui::DbuTypeCollectionItem const *>(_pItems->GetItem(5, true))".
92 const DbuTypeCollectionItem* pCollectionItem = dynamic_cast<const DbuTypeCollectionItem*>( _pItems->GetItem(DSID_TYPECOLLECTION) );
93 m_pCollection = pCollectionItem->getCollection();
94
95 ActivatePage();
96 setTitleBase(ModuleRes(STR_DATABASE_TYPE_CHANGE));
97 }
** CID 1362485: Null pointer dereferences (FORWARD_NULL)
________________________________________________________________________________________________________
*** CID 1362485: Null pointer dereferences (FORWARD_NULL)
/starmath/qa/cppunit/test_nodetotextvisitors.cxx: 613 in <unnamed>::Test::testUnaryInMixedNumberAsNumerator()()
607
608 SmCursor aCursor(pTree, xDocShRef);
609 ScopedVclPtrInstance< VirtualDevice > pOutputDevice;
610
611 // move forward (more than) enough places to be at the end
612 for (size_t i = 0; i < 3; ++i)
>>> CID 1362485: Null pointer dereferences (FORWARD_NULL)
>>> "Move" dereferences null "aCursor.mpPosition".
613 aCursor.Move(pOutputDevice, MoveRight);
614
615 // Select the whole Unary Horizontal Node
616 aCursor.Move(pOutputDevice, MoveLeft, false);
617 aCursor.Move(pOutputDevice, MoveLeft, false);
618
** CID 1362484: Null pointer dereferences (FORWARD_NULL)
/sw/source/core/doc/tblafmt.cxx: 698 in SwTableAutoFormat::GetBoxFormat(unsigned char)()
________________________________________________________________________________________________________
*** CID 1362484: Null pointer dereferences (FORWARD_NULL)
/sw/source/core/doc/tblafmt.cxx: 698 in SwTableAutoFormat::GetBoxFormat(unsigned char)()
692
693 SwBoxAutoFormat& SwTableAutoFormat::GetBoxFormat( sal_uInt8 nPos )
694 {
695 SAL_WARN_IF(!(nPos < 16), "sw.core", "GetBoxFormat wrong area");
696
697 SwBoxAutoFormat* pFormat = aBoxAutoFormat[ nPos ];
>>> CID 1362484: Null pointer dereferences (FORWARD_NULL)
>>> Comparing "pFormat" to null implies that "pFormat" might be null.
698 if( !pFormat )
699 {
700 // If default doesn't exist yet:
701 if( !pDfltBoxAutoFormat )
702 pDfltBoxAutoFormat = new SwBoxAutoFormat();
703 *pFormat = *pDfltBoxAutoFormat;
** CID 1362483: Null pointer dereferences (FORWARD_NULL)
________________________________________________________________________________________________________
*** CID 1362483: Null pointer dereferences (FORWARD_NULL)
/starmath/qa/cppunit/test_cursor.cxx: 102 in <unnamed>::Test::testCopySelectPaste()()
96
97 SmCursor aCursor(xTree.get(), xDocShRef);
98 ScopedVclPtrInstance<VirtualDevice> pOutputDevice;
99
100 // go to the right end
101 for (int i=0;i<5;i++)
>>> CID 1362483: Null pointer dereferences (FORWARD_NULL)
>>> "Move" dereferences null "aCursor.mpPosition".
102 aCursor.Move(pOutputDevice, MoveRight);
103 // select "b + c" and then copy
104 aCursor.Move(pOutputDevice, MoveLeft, false);
105 aCursor.Move(pOutputDevice, MoveLeft, false);
106 aCursor.Move(pOutputDevice, MoveLeft, false);
107 aCursor.Copy();
** CID 1362482: Null pointer dereferences (FORWARD_NULL)
________________________________________________________________________________________________________
*** CID 1362482: Null pointer dereferences (FORWARD_NULL)
/starmath/qa/cppunit/test_cursor.cxx: 152 in <unnamed>::Test::testCutSelectPaste()()
146
147 SmCursor aCursor(xTree.get(), xDocShRef);
148 ScopedVclPtrInstance<VirtualDevice> pOutputDevice;
149
150 // go to the right end
151 for (int i=0;i<5;i++)
>>> CID 1362482: Null pointer dereferences (FORWARD_NULL)
>>> "Move" dereferences null "aCursor.mpPosition".
152 aCursor.Move(pOutputDevice, MoveRight);
153 // select "b + c" and then cut
154 aCursor.Move(pOutputDevice, MoveLeft, false);
155 aCursor.Move(pOutputDevice, MoveLeft, false);
156 aCursor.Move(pOutputDevice, MoveLeft, false);
157 aCursor.Cut();
** CID 1362481: Null pointer dereferences (FORWARD_NULL)
________________________________________________________________________________________________________
*** CID 1362481: Null pointer dereferences (FORWARD_NULL)
/starmath/qa/cppunit/test_cursor.cxx: 78 in <unnamed>::Test::testCopyPaste()()
72 xTree->Prepare(xDocShRef->GetFormat(), *xDocShRef);
73
74 SmCursor aCursor(xTree.get(), xDocShRef);
75 ScopedVclPtrInstance<VirtualDevice> pOutputDevice;
76
77 // go to the position at "*"
>>> CID 1362481: Null pointer dereferences (FORWARD_NULL)
>>> "Move" dereferences null "aCursor.mpPosition".
78 aCursor.Move(pOutputDevice, MoveRight);
79 // select "* b" and then copy
80 aCursor.Move(pOutputDevice, MoveRight, false);
81 aCursor.Move(pOutputDevice, MoveRight, false);
82 aCursor.Copy();
83 // go to the right end and then paste
** CID 1362480: Null pointer dereferences (FORWARD_NULL)
________________________________________________________________________________________________________
*** CID 1362480: Null pointer dereferences (FORWARD_NULL)
/starmath/qa/cppunit/test_cursor.cxx: 128 in <unnamed>::Test::testCutPaste()()
122 xTree->Prepare(xDocShRef->GetFormat(), *xDocShRef);
123
124 SmCursor aCursor(xTree.get(), xDocShRef);
125 ScopedVclPtrInstance<VirtualDevice> pOutputDevice;
126
127 // go to the position at "*"
>>> CID 1362480: Null pointer dereferences (FORWARD_NULL)
>>> "Move" dereferences null "aCursor.mpPosition".
128 aCursor.Move(pOutputDevice, MoveRight);
129 // select "* b" and then cut
130 aCursor.Move(pOutputDevice, MoveRight, false);
131 aCursor.Move(pOutputDevice, MoveRight, false);
132 aCursor.Cut();
133 // go to the left end and then paste
** CID 1362479: Null pointer dereferences (FORWARD_NULL)
________________________________________________________________________________________________________
*** CID 1362479: Null pointer dereferences (FORWARD_NULL)
/starmath/qa/cppunit/test_nodetotextvisitors.cxx: 529 in <unnamed>::Test::testBinomInBinHor()()
523 SmCursor aCursor(pTree, xDocShRef);
524 ScopedVclPtrInstance< VirtualDevice > pOutputDevice;
525
526 // move forward (more than) enough places to be at the end
527 int i;
528 for (i = 0; i < 8; ++i)
>>> CID 1362479: Null pointer dereferences (FORWARD_NULL)
>>> "Move" dereferences null "aCursor.mpPosition".
529 aCursor.Move(pOutputDevice, MoveRight);
530
531 // tack +d on the end, which will put the binom into an SmBinHorNode
532 aCursor.InsertElement(PlusElement);
533 aCursor.InsertText("d");
534
** CID 1362478: Null pointer dereferences (FORWARD_NULL)
________________________________________________________________________________________________________
*** CID 1362478: Null pointer dereferences (FORWARD_NULL)
/starmath/qa/cppunit/test_nodetotextvisitors.cxx: 557 in <unnamed>::Test::testBinVerInUnary()()
551 SmCursor aCursor(pTree, xDocShRef);
552 ScopedVclPtrInstance< VirtualDevice > pOutputDevice;
553
554 // move forward (more than) enough places to be at the end
555 int i;
556 for (i = 0; i < 3; ++i)
>>> CID 1362478: Null pointer dereferences (FORWARD_NULL)
>>> "Move" dereferences null "aCursor.mpPosition".
557 aCursor.Move(pOutputDevice, MoveRight);
558
559 // select the operand
560 aCursor.Move(pOutputDevice, MoveLeft, false);
561 // set up a fraction
562 aCursor.InsertFraction();
________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://scan.coverity.com/projects/libreoffice?tab=overview
To manage Coverity Scan email notifications for "libreoffice at lists.freedesktop.org", click https://scan.coverity.com/subscriptions/edit?email=libreoffice%40lists.freedesktop.org&token=d6481d718a775246b2340f282ebe5939
More information about the LibreOffice
mailing list