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.

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


** CID 1594675:    (DEADCODE)
/sc/source/ui/view/tabview.cxx: 1160 in ScTabView::ScrollHdl(ScrollAdaptor *)()
/sc/source/ui/view/tabview.cxx: 1158 in ScTabView::ScrollHdl(ScrollAdaptor *)()


________________________________________________________________________________________________________
*** CID 1594675:    (DEADCODE)
/sc/source/ui/view/tabview.cxx: 1160 in ScTabView::ScrollHdl(ScrollAdaptor *)()
1154                     aHelpStr = ScResId(STR_ROW) +
1155                                " " + OUString::number(nScrollPos + 1);
1156     
1157                     // show quicktext always inside sheet area
1158                     aRect.SetLeft( bLayoutRTL ? (aPos.X() + aSize.Width() + 8) : (aPos.X() - 8) );
1159                     aRect.SetTop( aMousePos.Y() );
>>>     CID 1594675:    (DEADCODE)
>>>     Execution cannot reach the expression "QuickHelpFlags::Left" inside this statement: "<temporary> = (bLayoutRTL ?...".
1160                     nAlign       = (bLayoutRTL ? QuickHelpFlags::Left : QuickHelpFlags::Right) | QuickHelpFlags::VCenter;
1161                 }
1162                 aRect.SetRight( aRect.Left() );
1163                 aRect.SetBottom( aRect.Top() );
1164     
1165                 Help::ShowQuickHelp(pScroll->GetParent(), aRect, aHelpStr, nAlign);
/sc/source/ui/view/tabview.cxx: 1158 in ScTabView::ScrollHdl(ScrollAdaptor *)()
1152                 else
1153                 {
1154                     aHelpStr = ScResId(STR_ROW) +
1155                                " " + OUString::number(nScrollPos + 1);
1156     
1157                     // show quicktext always inside sheet area
>>>     CID 1594675:    (DEADCODE)
>>>     Execution cannot reach the expression "aPos.X() + aSize.Width() + 8L" inside this statement: "aRect.SetLeft((bLayoutRTL ?...".
1158                     aRect.SetLeft( bLayoutRTL ? (aPos.X() + aSize.Width() + 8) : (aPos.X() - 8) );
1159                     aRect.SetTop( aMousePos.Y() );
1160                     nAlign       = (bLayoutRTL ? QuickHelpFlags::Left : QuickHelpFlags::Right) | QuickHelpFlags::VCenter;
1161                 }
1162                 aRect.SetRight( aRect.Left() );
1163                 aRect.SetBottom( aRect.Top() );

** CID 1594674:  API usage errors  (INVALIDATE_ITERATOR)
/sc/source/core/data/SolverSettings.cxx: 789 in sc::SolverSettings::TabHasSolverModel()()


________________________________________________________________________________________________________
*** CID 1594674:  API usage errors  (INVALIDATE_ITERATOR)
/sc/source/core/data/SolverSettings.cxx: 789 in sc::SolverSettings::TabHasSolverModel()()
783        so before it is called, no solver-related named ranges exist.
784     */
785     bool SolverSettings::TabHasSolverModel()
786     {
787         // Check if the named range for the objective value exists in the sheet
788         const auto iter = m_mNamedRanges.find(SP_OBJ_CELL);
>>>     CID 1594674:  API usage errors  (INVALIDATE_ITERATOR)
>>>     Dereferencing iterator "iter" though it is already past the end of its container.
789         OUString sRange = iter->second;
790         ScRangeData* pRangeData
791             = m_pRangeName->findByUpperName(ScGlobal::getCharClass().uppercase(sRange));
792         if (pRangeData)
793             return true;
794         return false;
795     }
796     


________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://u15810271.ct.sendgrid.net/ls/click?upn=u001.AxU2LYlgjL6eX23u9ErQy-2BKADyCpvUKOL6EWmZljiu6VvXBlQRUbS683tC8265rGNPXqJ1ffcoLZCnTuJFQbNcTEkb4XaEQkzovKhJ5DB3c-3DBAIx_A9M4dSy7guk8NP6DcfgslOyvJRzavztVIKj6nRqYjYpWom7SJFyX0y710bz0kUGtERQty1Y5JpkDXubi7MsokQaRv4jsHtrS-2BixY5vNs-2FGbRZsRBY0-2Ba4Ds142E1HxrBQy3-2FfJLFeOubpB02YnnKddIcygoNaLijEdfdc1WG4Y32XWZ2k46a8UyxKtYrRJWZowog86JcdLAHzoDzhbfpYGyqwawJaDre8AYwyGfd4rM-3D