New Defects reported by Coverity Scan for LibreOffice

[scan-admin at coverity.com]

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.


New defect(s) Reported-by: Coverity Scan
Showing 2 of 2 defect(s)


** CID 1594402:  Null pointer dereferences  (FORWARD_NULL)


________________________________________________________________________________________________________
*** CID 1594402:  Null pointer dereferences  (FORWARD_NULL)
/sc/source/ui/unoobj/docuno.cxx: 2372 in lcl_SetMediaScreen(const com::sun::star::uno::Reference<com::sun::star::drawing::XShape> &, const OutputDevice *, tools::Rectangle &, int)()
2366                                        : OUString::Concat(sTitle) + OUString::Concat("\n")
2367                                              + OUString::Concat(sDescription));
2368     
2369             OUString const mimeType(xPropSet->getPropertyValue("MediaMimeType").get<OUString>());
2370             SdrObject* pSdrObj(SdrObject::getSdrObjectFromXShape(xMediaShape));
2371             vcl::PDFExtOutDevData* pPDF = dynamic_cast<vcl::PDFExtOutDevData*>(pDev->GetExtOutDevData());
>>>     CID 1594402:  Null pointer dereferences  (FORWARD_NULL)
>>>     Passing null pointer "pPDF" to "CreateScreen", which dereferences it.
2372             sal_Int32 nScreenId = pPDF->CreateScreen(aRect, altText, mimeType, nPageNumb, pSdrObj);
2373             if (sMediaURL.startsWith("vnd.sun.star.Package:"))
2374             {
2375                 // Embedded media
2376                 OUString aTempFileURL;
2377                 xPropSet->getPropertyValue("PrivateTempFileURL") >>= aTempFileURL;

** CID 1594401:  Null pointer dereferences  (FORWARD_NULL)


________________________________________________________________________________________________________
*** CID 1594401:  Null pointer dereferences  (FORWARD_NULL)
/sfx2/source/sidebar/ControllerItem.cxx: 72 in sfx2::sidebar::ControllerItem::RequestUpdate()()
66         const SfxItemState eState (GetBindings().QueryState(GetId(), pState));
67         if (GetId() == SID_ATTR_METRIC && comphelper::LibreOfficeKit::isActive())
68         {
69             MeasurementSystem eSystem
70                 = LocaleDataWrapper(comphelper::LibreOfficeKit::getLocale()).getMeasurementSystemEnum();
71             FieldUnit eUnit = MeasurementSystem::Metric == eSystem ? FieldUnit::CM : FieldUnit::INCH;
>>>     CID 1594401:  Null pointer dereferences  (FORWARD_NULL)
>>>     Passing null pointer "pState.get()" to "SetValue", which dereferences it.
72             static_cast<SfxUInt16Item*>(pState.get())->SetValue(static_cast<sal_uInt16>(eUnit));
73         }
74         mrItemUpdateReceiver.NotifyItemUpdate(GetId(), eState, pState.get());
75     }
76     
77     ControllerItem::ItemUpdateReceiverInterface::~ItemUpdateReceiverInterface()


________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://u15810271.ct.sendgrid.net/ls/click?upn=u001.AxU2LYlgjL6eX23u9ErQy-2BKADyCpvUKOL6EWmZljiu6VvXBlQRUbS683tC8265rGNPXqJ1ffcoLZCnTuJFQbNcTEkb4XaEQkzovKhJ5DB3c-3DQmg-_A9M4dSy7guk8NP6DcfgslOyvJRzavztVIKj6nRqYjYpWom7SJFyX0y710bz0kUGtuM6dpEYHAKPAFOJnFFVTAqyXcfeEbV-2BOW4lbsmS1OR0nkjjh7njRe7s5QFDFx7brLxb5QyVkD5FJ6Cbd7-2Bgo8L3HkTQubHucgEUCgCYMVFNZbqXF2c2vU7innBMhSfrv9VCmBEXIFp5fBqc-2B4Gq1xnoJqgsa0QHSbGHvJGqgj30NSky9ULHy9BfyemEoKG3F