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 1636891:  Memory - illegal accesses  (UNINIT)


________________________________________________________________________________________________________
*** CID 1636891:  Memory - illegal accesses  (UNINIT)
/sax/source/fastparser/fastparser.cxx: 545 in <unnamed>::Entity::transferUsedEvents()()
539     {
540         std::unique_lock aGuard(maEventProtector);
541         if (!maUsedEvents.empty())
542         {
543             mxProducedEvents = std::move(maUsedEvents.front());
544             maUsedEvents.pop();
>>>     CID 1636891:  Memory - illegal accesses  (UNINIT)
>>>     Using uninitialized value "aGuard._M_owns" when calling "unlock".
545             aGuard.unlock(); // unlock
546             mnProducedEventsSize = 0;
547         }
548     }
549     
550     EventList& Entity::getEventList()

** CID 1636890:  Integer handling issues  (DIVIDE_BY_ZERO)
/sc/source/core/data/table2.cxx: 4397 in ScTable::GetRowForHeightPixels(int, long &, long, double) const()


________________________________________________________________________________________________________
*** CID 1636890:  Integer handling issues  (DIVIDE_BY_ZERO)
/sc/source/core/data/table2.cxx: 4397 in ScTable::GetRowForHeightPixels(int, long &, long, double) const()
4391             tools::Long nCommonPixels = nRowHeightPx * nCommonRows;
4392     
4393             // is the target height inside the common span ?
4394             if (nSumPx + nCommonPixels > nHeightPx)
4395             {
4396                 // calculate how many rows to skip inside the common span
>>>     CID 1636890:  Integer handling issues  (DIVIDE_BY_ZERO)
>>>     In expression "(nHeightPx - nSumPx) / nRowHeightPx", division by expression "nRowHeightPx" which may be zero has undefined behavior.
4397                 SCROW nRowsInside = (nHeightPx - nSumPx) / nRowHeightPx;
4398                 nRow += nRowsInside;
4399     
4400                 assert(aHiddenRange.mnRow1 <= nRow && aHiddenRange.mnRow2 >= nRow && "the current hidden-row span should overlap the current row");
4401                 assert(aRowHeightRange.mnRow1 <= nRow && aRowHeightRange.mnRow2 >= nRow && "the current height span should overlap the current row");
4402     


________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://u15810271.ct.sendgrid.net/ls/click?upn=u001.AxU2LYlgjL6eX23u9ErQy-2BKADyCpvUKOL6EWmZljiu6VvXBlQRUbS683tC8265rGNPXqJ1ffcoLZCnTuJFQbNcTEkb4XaEQkzovKhJ5DB3c-3DI8PN_A9M4dSy7guk8NP6DcfgslOyvJRzavztVIKj6nRqYjYpWom7SJFyX0y710bz0kUGtwsnZuY7NxUySzrvmi-2FHxskEzU95pBa9eKVggrEgzt5WFE4BzNQNIigT7yqZJq2c-2FkbxT5PgeWk8QcwI00e0pMBQJDvbXynwaT-2BuCxrA-2FcwQZELDx-2BkktJ6oxpBFs5h-2B-2B-2BwJcGgl5CFI60EsYBgQER4zNCFyJGldp3qWmtZpU6Qw-3D