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


** CID 1489785:  Null pointer dereferences  (FORWARD_NULL)


________________________________________________________________________________________________________
*** CID 1489785:  Null pointer dereferences  (FORWARD_NULL)
/sc/source/ui/drawfunc/fudraw.cxx: 717 in FuDraw::ForcePointer(const MouseEvent *)()
711         else if ( !bAlt && (pObj = pView->PickObj(aPnt, pView->getHitTolLog(), pPV, SdrSearchOptions::PICKMACRO)) )
712         {
713             //  could be suppressed with ALT
714             SdrObjMacroHitRec aHitRec;  //! something missing ????
715             rViewShell.SetActivePointer( pObj->GetMacroPointer(aHitRec) );
716         }
>>>     CID 1489785:  Null pointer dereferences  (FORWARD_NULL)
>>>     Passing null pointer "pObj" to "getHyperlink", which dereferences it.
717         else if ( !bAlt && pInfo && (!pInfo->GetMacro().isEmpty() || !pObj->getHyperlink().isEmpty()) )
718             pWindow->SetPointer( PointerStyle::RefHand );
719         else if ( IsDetectiveHit( aPnt ) )
720             rViewShell.SetActivePointer( PointerStyle::Detective );
721         else
722         {

** CID 1489784:  Null pointer dereferences  (FORWARD_NULL)
/sc/source/ui/drawfunc/drawsh2.cxx: 239 in ScDrawShell::GetDrawFuncState(SfxItemSet &)()


________________________________________________________________________________________________________
*** CID 1489784:  Null pointer dereferences  (FORWARD_NULL)
/sc/source/ui/drawfunc/drawsh2.cxx: 239 in ScDrawShell::GetDrawFuncState(SfxItemSet &)()
233             {
234                 rSet.DisableItem( SID_EDIT_HYPERLINK );
235                 rSet.DisableItem( SID_OPEN_HYPERLINK );
236                 rSet.DisableItem( SID_REMOVE_HYPERLINK );
237                 rSet.DisableItem( SID_COPY_HYPERLINK_LOCATION );
238             }
>>>     CID 1489784:  Null pointer dereferences  (FORWARD_NULL)
>>>     Passing null pointer "pObj" to "GetLayer", which dereferences it. (The dereference happens because this is a virtual function call.)
239             SdrLayerID nLayerID = pObj->GetLayer();
240             if ( nLayerID != SC_LAYER_INTERN )
241                 bCanRename = true;                          // #i51351# anything except internal objects can be renamed
242     
243             // #91929#; don't show original size entry if not possible
244             sal_uInt16 nObjType = pObj->GetObjIdentifier();


________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://u15810271.ct.sendgrid.net/ls/click?upn=HRESupC-2F2Czv4BOaCWWCy7my0P0qcxCbhZ31OYv50ypSs1kiFPuCn2xFdlMIFBirii0zZ9j2-2F9F2XPBcBm2BNgi9duPy3v-2FzgFDd2LJ-2BDKI-3D79FV_OTq2XUZbbipYjyLSo6GRo-2FpVxQ9OzkDINu9UTS-2FQhSdO0F0jQniitrGlNxDIzPJiiQ7y-2FGsEsWVN-2FyCvUrEN-2FsNO2mXu7IIRuqP9syuSjbjitGYe-2FEYx4ap0VhSq6OeOrUA-2FSMhktDcLeS09UbmsNDaN7hyf22lGS-2BaEdnT4K0MBwpQp-2Bnx2ofN46yjqAH8re2euGwzQzHsdEpeLbmYiGwP5K5gKPfieCV-2BIK4tX-2Bkg-3D