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.

3 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 3 of 3 defect(s)


** CID 1635785:    (INVALIDATE_ITERATOR)
/vcl/source/uitest/uiobject.cxx: 778 in EditUIObject::execute(const rtl::OUString &, const std::map<rtl::OUString, rtl::OUString, std::less<rtl::OUString>, std::allocator<std::pair<const rtl::OUString, rtl::OUString>>> &)()
/vcl/source/uitest/uiobject.cxx: 777 in EditUIObject::execute(const rtl::OUString &, const std::map<rtl::OUString, rtl::OUString, std::less<rtl::OUString>, std::allocator<std::pair<const rtl::OUString, rtl::OUString>>> &)()


________________________________________________________________________________________________________
*** CID 1635785:    (INVALIDATE_ITERATOR)
/vcl/source/uitest/uiobject.cxx: 778 in EditUIObject::execute(const rtl::OUString &, const std::map<rtl::OUString, rtl::OUString, std::less<rtl::OUString>, std::allocator<std::pair<const rtl::OUString, rtl::OUString>>> &)()
772         else if (rAction == "SELECT")
773         {
774             if (rParameters.contains(u"FROM"_ustr) &&
775                     rParameters.contains(u"TO"_ustr))
776             {
777                 tools::Long nMin = rParameters.find(u"FROM"_ustr)->second.toInt32();
>>>     CID 1635785:    (INVALIDATE_ITERATOR)
>>>     Dereferencing iterator "rParameters->find(rtl::OUString(operator "" _ustr()))" though it is already past the end of its container.
778                 tools::Long nMax = rParameters.find(u"TO"_ustr)->second.toInt32();
779                 Selection aSelection(nMin, nMax);
780                 mxEdit->SetSelection(aSelection);
781             }
782         }
783         else if (rAction == "CLEAR")
/vcl/source/uitest/uiobject.cxx: 777 in EditUIObject::execute(const rtl::OUString &, const std::map<rtl::OUString, rtl::OUString, std::less<rtl::OUString>, std::allocator<std::pair<const rtl::OUString, rtl::OUString>>> &)()
771         }
772         else if (rAction == "SELECT")
773         {
774             if (rParameters.contains(u"FROM"_ustr) &&
775                     rParameters.contains(u"TO"_ustr))
776             {
>>>     CID 1635785:    (INVALIDATE_ITERATOR)
>>>     Dereferencing iterator "rParameters->find(rtl::OUString(operator "" _ustr()))" though it is already past the end of its container.
777                 tools::Long nMin = rParameters.find(u"FROM"_ustr)->second.toInt32();
778                 tools::Long nMax = rParameters.find(u"TO"_ustr)->second.toInt32();
779                 Selection aSelection(nMin, nMax);
780                 mxEdit->SetSelection(aSelection);
781             }
782         }

** CID 1635784:  API usage errors  (INVALIDATE_ITERATOR)
/vcl/source/uitest/uiobject.cxx: 362 in WindowUIObject::execute(const rtl::OUString &, const std::map<rtl::OUString, rtl::OUString, std::less<rtl::OUString>, std::allocator<std::pair<const rtl::OUString, rtl::OUString>>> &)()


________________________________________________________________________________________________________
*** CID 1635784:  API usage errors  (INVALIDATE_ITERATOR)
/vcl/source/uitest/uiobject.cxx: 362 in WindowUIObject::execute(const rtl::OUString &, const std::map<rtl::OUString, rtl::OUString, std::less<rtl::OUString>, std::allocator<std::pair<const rtl::OUString, rtl::OUString>>> &)()
356                     mxWindow->KeyInput(keyEvent);
357                 }
358             }
359             else if (rParameters.contains(u"KEYCODE"_ustr))
360             {
361                 auto itr = rParameters.find(u"KEYCODE"_ustr);
>>>     CID 1635784:  API usage errors  (INVALIDATE_ITERATOR)
>>>     Dereferencing iterator "itr" though it is already past the end of its container.
362                 const OUString rText = itr->second;
363                 auto aKeyEvents = generate_key_events_from_keycode(rText);
364                 for (auto const& keyEvent : aKeyEvents)
365                 {
366                     mxWindow->KeyInput(keyEvent);
367                 }

** CID 1635783:    (INVALIDATE_ITERATOR)
/vcl/source/uitest/uiobject.cxx: 1211 in ComboBoxUIObject::execute(const rtl::OUString &, const std::map<rtl::OUString, rtl::OUString, std::less<rtl::OUString>, std::allocator<std::pair<const rtl::OUString, rtl::OUString>>> &)()
/vcl/source/uitest/uiobject.cxx: 1205 in ComboBoxUIObject::execute(const rtl::OUString &, const std::map<rtl::OUString, rtl::OUString, std::less<rtl::OUString>, std::allocator<std::pair<const rtl::OUString, rtl::OUString>>> &)()


________________________________________________________________________________________________________
*** CID 1635783:    (INVALIDATE_ITERATOR)
/vcl/source/uitest/uiobject.cxx: 1211 in ComboBoxUIObject::execute(const rtl::OUString &, const std::map<rtl::OUString, rtl::OUString, std::less<rtl::OUString>, std::allocator<std::pair<const rtl::OUString, rtl::OUString>>> &)()
1205                 OUString aVal = itr->second;
1206                 sal_Int32 nPos = aVal.toInt32();
1207                 mxComboBox->SelectEntryPos(nPos);
1208             }
1209             else if(rParameters.contains(u"TEXT"_ustr)){
1210                 auto itr = rParameters.find(u"TEXT"_ustr);
>>>     CID 1635783:    (INVALIDATE_ITERATOR)
>>>     Dereferencing iterator "itr" though it is already past the end of its container.
1211                 OUString aVal = itr->second;
1212                 sal_Int32 nPos = mxComboBox->GetEntryPos(aVal);
1213                 mxComboBox->SelectEntryPos(nPos);
1214             }
1215             mxComboBox->Select();
1216         }
/vcl/source/uitest/uiobject.cxx: 1205 in ComboBoxUIObject::execute(const rtl::OUString &, const std::map<rtl::OUString, rtl::OUString, std::less<rtl::OUString>, std::allocator<std::pair<const rtl::OUString, rtl::OUString>>> &)()
1199     {
1200         if (rAction == "SELECT")
1201         {
1202             if (rParameters.contains(u"POS"_ustr))
1203             {
1204                 auto itr = rParameters.find(u"POS"_ustr);
>>>     CID 1635783:    (INVALIDATE_ITERATOR)
>>>     Dereferencing iterator "itr" though it is already past the end of its container.
1205                 OUString aVal = itr->second;
1206                 sal_Int32 nPos = aVal.toInt32();
1207                 mxComboBox->SelectEntryPos(nPos);
1208             }
1209             else if(rParameters.contains(u"TEXT"_ustr)){
1210                 auto itr = rParameters.find(u"TEXT"_ustr);


________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://u15810271.ct.sendgrid.net/ls/click?upn=u001.AxU2LYlgjL6eX23u9ErQy-2BKADyCpvUKOL6EWmZljiu6VvXBlQRUbS683tC8265rGNPXqJ1ffcoLZCnTuJFQbNcTEkb4XaEQkzovKhJ5DB3c-3DRahl_A9M4dSy7guk8NP6DcfgslOyvJRzavztVIKj6nRqYjYpWom7SJFyX0y710bz0kUGtlU4lQUwaGiHQXOwErg3BJosWDIOg83ld01U8I9HASEg14fwp03-2F3jnKw2pYEde2jzKp9tV-2FHn8A9hYsO-2FPLH602HAIlLy0IKFpGc-2FlrTF2HLoTWvDWE8lmG-2FHV2UmnCIx8wUlNmHNPOKX6qWIOgn4OIOHAVps96JyJMhTVgIPlw-3D