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.


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


** CID 1541152:  Control flow issues  (DEADCODE)
/sw/source/core/layout/paintfrm.cxx: 2899 in <unnamed>::SwTabFramePainter::InsertFollowTopBorder(const SwFrame &, const SvxBoxItem &, bool, long, long, long, bool)()


________________________________________________________________________________________________________
*** CID 1541152:  Control flow issues  (DEADCODE)
/sw/source/core/layout/paintfrm.cxx: 2899 in <unnamed>::SwTabFramePainter::InsertFollowTopBorder(const SwFrame &, const SvxBoxItem &, bool, long, long, long, bool)()
2893         }
2894     
2895         // This is then a first row in a follow table, without repeated headlines.
2896         auto pLastRow = dynamic_cast<const SwRowFrame*>(mrTabFrame.GetLastLower());
2897         if (!pLastRow && pLastRow == pThisRow)
2898         {
>>>     CID 1541152:  Control flow issues  (DEADCODE)
>>>     Execution cannot reach this statement: "return;".
2899             return;
2900         }
2901     
2902         const SwFrame* pLastCell = pLastRow->GetLower();
2903         for (int i = 0; i < nCol; ++i)
2904         {

** CID 1541151:  Null pointer dereferences  (FORWARD_NULL)


________________________________________________________________________________________________________
*** CID 1541151:  Null pointer dereferences  (FORWARD_NULL)
/sw/source/core/layout/paintfrm.cxx: 2902 in <unnamed>::SwTabFramePainter::InsertFollowTopBorder(const SwFrame &, const SvxBoxItem &, bool, long, long, long, bool)()
2896         auto pLastRow = dynamic_cast<const SwRowFrame*>(mrTabFrame.GetLastLower());
2897         if (!pLastRow && pLastRow == pThisRow)
2898         {
2899             return;
2900         }
2901     
>>>     CID 1541151:  Null pointer dereferences  (FORWARD_NULL)
>>>     Passing null pointer "pLastRow" to "GetLower", which dereferences it.
2902         const SwFrame* pLastCell = pLastRow->GetLower();
2903         for (int i = 0; i < nCol; ++i)
2904         {
2905             if (!pLastCell)
2906             {
2907                 break;


________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://u15810271.ct.sendgrid.net/ls/click?upn=HRESupC-2F2Czv4BOaCWWCy7my0P0qcxCbhZ31OYv50ypSs1kiFPuCn2xFdlMIFBirii0zZ9j2-2F9F2XPBcBm2BNgi9duPy3v-2FzgFDd2LJ-2BDKI-3DwgLn_OTq2XUZbbipYjyLSo6GRo-2FpVxQ9OzkDINu9UTS-2FQhSdO0F0jQniitrGlNxDIzPJiEY3BFVrLuW9iFjIZnXOsrXpSI528ZMHgx0KQIjS7mrlzhOPGVcvdGZWsIn3gYRsmP4IYBljOpsMbZxEVcU-2B0FvVBuWnQZZsYtleb6gNt28BFRjO08vzARDtIxBYDx2Iw8uYzy75AZQNB5Kii-2FcvzP6rnJDXN9E7dYsmjdLefzoM-3D