New Defects reported by Coverity Scan for LibreOffice

[scan-admin@xxxxxxxxxxxx]

Hi,

Please find the latest report on new defect(s) introduced to LibreOffice found with Coverity Scan.

4 new defect(s) introduced to LibreOffice found with Coverity Scan.
52 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 4 of 4 defect(s)


** CID 1610243:  Uninitialized members  (UNINIT_CTOR)
/sw/source/uibase/wrtsh/select.cxx: 754 in SwWrtShell::SetInsMode(bool)::[lambda(int) (instance 1)]::lambda(const [lambda(int) (instance 1)]&)()


________________________________________________________________________________________________________
*** CID 1610243:  Uninitialized members  (UNINIT_CTOR)
/sw/source/uibase/wrtsh/select.cxx: 754 in SwWrtShell::SetInsMode(bool)::[lambda(int) (instance 1)]::lambda(const [lambda(int) (instance 1)]&)()
748     void SwWrtShell::SetInsMode( bool bOn )
749     {
750         const bool bDoAsk = officecfg::Office::Common::Misc::QuerySetInsMode::get();
751         if (!bOn && bDoAsk)
752         {
753             auto xDialog = std::make_shared<QuerySetInsModeDialog>(GetView().GetFrameWeld());
>>>     CID 1610243:  Uninitialized members  (UNINIT_CTOR)
>>>     Non-static class member "this" is not initialized in this constructor nor in any functions that it calls.
754             weld::DialogController::runAsync(xDialog, [this, bOn, xDialog](sal_Int32 nResult){
755     
756                 std::shared_ptr<comphelper::ConfigurationChanges> xChanges(
757                     comphelper::ConfigurationChanges::create());
758                 officecfg::Office::Common::Misc::QuerySetInsMode::set(!xDialog->GetDoNotShowAgain(), xChanges);
759                 xChanges->commit();

** CID 1610242:  Uninitialized members  (UNINIT_CTOR)
/sw/source/uibase/wrtsh/select.cxx: 754 in SwWrtShell::SetInsMode(bool)::[lambda(int) (instance 1)]::lambda([lambda(int) (instance 1)]&&)()


________________________________________________________________________________________________________
*** CID 1610242:  Uninitialized members  (UNINIT_CTOR)
/sw/source/uibase/wrtsh/select.cxx: 754 in SwWrtShell::SetInsMode(bool)::[lambda(int) (instance 1)]::lambda([lambda(int) (instance 1)]&&)()
748     void SwWrtShell::SetInsMode( bool bOn )
749     {
750         const bool bDoAsk = officecfg::Office::Common::Misc::QuerySetInsMode::get();
751         if (!bOn && bDoAsk)
752         {
753             auto xDialog = std::make_shared<QuerySetInsModeDialog>(GetView().GetFrameWeld());
>>>     CID 1610242:  Uninitialized members  (UNINIT_CTOR)
>>>     Non-static class member "this" is not initialized in this constructor nor in any functions that it calls.
754             weld::DialogController::runAsync(xDialog, [this, bOn, xDialog](sal_Int32 nResult){
755     
756                 std::shared_ptr<comphelper::ConfigurationChanges> xChanges(
757                     comphelper::ConfigurationChanges::create());
758                 officecfg::Office::Common::Misc::QuerySetInsMode::set(!xDialog->GetDoNotShowAgain(), xChanges);
759                 xChanges->commit();

** CID 1610241:  Integer handling issues  (NEGATIVE_RETURNS)


________________________________________________________________________________________________________
*** CID 1610241:  Integer handling issues  (NEGATIVE_RETURNS)
/vcl/source/control/ivctrl.cxx: 539 in VerticalTabControl::RemovePage(std::basic_string_view<char16_t, std::char_traits<char16_t>>)()
533         for (auto it = maPageList.begin(), end = maPageList.end(); it != end; ++it)
534         {
535             VerticalTabPageData* pData = it->get();
536             if (pData->sId == rPageId)
537             {
538                 sal_Int32 nEntryListPos = m_xChooser->GetEntryListPos(pData->pEntry);
>>>     CID 1610241:  Integer handling issues  (NEGATIVE_RETURNS)
>>>     "nEntryListPos" is passed to a parameter that cannot be negative.
539                 m_xChooser->RemoveEntry(nEntryListPos);
540                 m_xChooser->ArrangeIcons();
541                 maPageList.erase(it);
542                 break;
543             }
544         }

** CID 1610240:  Null pointer dereferences  (FORWARD_NULL)
/desktop/source/lib/init.cxx: 3195 in lo_extractDocumentStructureRequest(_LibreOfficeKit *, const char *)()


________________________________________________________________________________________________________
*** CID 1610240:  Null pointer dereferences  (FORWARD_NULL)
/desktop/source/lib/init.cxx: 3195 in lo_extractDocumentStructureRequest(_LibreOfficeKit *, const char *)()
3189                     //if it is a writer document..
3190                     uno::Reference<lang::XServiceInfo> xDocument(xComp, uno::UNO_QUERY_THROW);
3191                     if (xDocument->supportsService(u"com.sun.star.text.TextDocument"_ustr) || xDocument->supportsService(u"com.sun.star.text.WebDocument"_ustr))
3192                     {
3193                         tools::JsonWriter aJson;
3194                         {
>>>     CID 1610240:  Null pointer dereferences  (FORWARD_NULL)
>>>     Passing null pointer "pDoc" to "getCommandValues", which dereferences it. (The dereference happens because this is a virtual function call.)
3195                             pDoc->getCommandValues(aJson, ".uno:ExtractDocumentStructure");
3196                             //auto aNode = aJson.startNode("Controls");
3197                             //extractLinks(xLTS->getLinks(), false, aJson);
3198                         }
3199                         return convertOString(aJson.finishAndGetAsOString());
3200                     }


________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://u15810271.ct.sendgrid.net/ls/click?upn=u001.AxU2LYlgjL6eX23u9ErQy-2BKADyCpvUKOL6EWmZljiu6VvXBlQRUbS683tC8265rGNPXqJ1ffcoLZCnTuJFQbNcTEkb4XaEQkzovKhJ5DB3c-3DlJUh_A9M4dSy7guk8NP6DcfgslOyvJRzavztVIKj6nRqYjYpWom7SJFyX0y710bz0kUGtAZtC52NswyE8sbZtF2824btlZixS78TWEI625KrotNmYK9-2FNDCHerqPP-2FRvERXq59uAjn5SCruixNgGvuAIGEn-2BidzsYyuKqP8rU2-2ByGa2YlIKOA3YijSCtga2E-2BMel9-2BKRsQcTmu3sbjZicQCfV-2FCiJ-2But-2Fwcyxq9L4EXAklYw-3D