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.
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 1539071:  Integer handling issues  (DIVIDE_BY_ZERO)
/accessibility/source/extended/accessibletablistboxtable.cxx: 328 in accessibility::AccessibleTabListBoxTable::getSelectedAccessibleChild(long)()


________________________________________________________________________________________________________
*** CID 1539071:  Integer handling issues  (DIVIDE_BY_ZERO)
/accessibility/source/extended/accessibletablistboxtable.cxx: 328 in accessibility::AccessibleTabListBoxTable::getSelectedAccessibleChild(long)()
322     
323             sal_Int32 nRows = implGetSelRowCount();
324             if ( nRows == 0 )
325                 throw IndexOutOfBoundsException();
326     
327             const sal_Int32 nColCount = implGetColumnCount();
>>>     CID 1539071:  Integer handling issues  (DIVIDE_BY_ZERO)
>>>     In expression "nSelectedChildIndex / nColCount", division by expression "nColCount" which may be zero has undefined behavior.
328             const sal_Int32 nRow = implGetSelRow(nSelectedChildIndex / nColCount);
329             const sal_Int32 nColumn = nSelectedChildIndex % nColCount;
330             return getAccessibleCellAt( nRow, nColumn );
331         }
332     
333         void SAL_CALL AccessibleTabListBoxTable::deselectAccessibleChild( sal_Int64 nSelectedChildIndex )

** CID 1539070:  Null pointer dereferences  (REVERSE_INULL)
/sd/source/ui/func/futext.cxx: 725 in sd::FuText::MouseButtonUp(const MouseEvent &)()


________________________________________________________________________________________________________
*** CID 1539070:  Null pointer dereferences  (REVERSE_INULL)
/sd/source/ui/func/futext.cxx: 725 in sd::FuText::MouseButtonUp(const MouseEvent &)()
719         }
720     
721         ForcePointer(&rMEvt);
722         mpWindow->ReleaseMouse();
723         sal_uInt16 nDrgLog1 = sal_uInt16 ( mpWindow->PixelToLogic(Size(mpView->GetDragThresholdPixels(),0)).Width() );
724     
>>>     CID 1539070:  Null pointer dereferences  (REVERSE_INULL)
>>>     Null-checking "this->mpView" suggests that it may be null, but it has already been dereferenced on all paths leading to the check.
725         if ( mpView && !mpView->AreObjectsMarked() &&
726              std::abs(aMDPos.X() - aPnt.X()) < nDrgLog1 &&
727              std::abs(aMDPos.Y() - aPnt.Y()) < nDrgLog1 &&
728              !rMEvt.IsShift() && !rMEvt.IsMod2() )
729         {
730             SdrPageView* pPV2 = mpView->GetSdrPageView();


________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://u15810271.ct.sendgrid.net/ls/click?upn=HRESupC-2F2Czv4BOaCWWCy7my0P0qcxCbhZ31OYv50ypSs1kiFPuCn2xFdlMIFBirii0zZ9j2-2F9F2XPBcBm2BNgi9duPy3v-2FzgFDd2LJ-2BDKI-3DmZOW_OTq2XUZbbipYjyLSo6GRo-2FpVxQ9OzkDINu9UTS-2FQhSdO0F0jQniitrGlNxDIzPJizAj7K4U7LzoHJpnORrww8GbnFbzCRczyOubZ3ivnZuxJoEh8jLeAyHSSX7-2FzL2YgzK-2FC5SE-2FcHMcgtucfLH8dF2GvzvGEcFPnle3I-2FlMrflZ7bB6rXCS798C2HzDXonQaMD-2F5Z56xbUYVGAjGGiJpfZIow8lM4QcZvz7kfUQYL4-3D