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.

1 new defect(s) introduced to LibreOffice found with Coverity Scan.
2 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 1 of 1 defect(s)


** CID 1515457:  Integer handling issues  (SIGN_EXTENSION)
/sw/source/core/text/porrst.cxx: 802 in SwBookmarkPortion::Paint(const SwTextPaintInfo &) const()


________________________________________________________________________________________________________
*** CID 1515457:  Integer handling issues  (SIGN_EXTENSION)
/sw/source/core/text/porrst.cxx: 802 in SwBookmarkPortion::Paint(const SwTextPaintInfo &) const()
796                 aNewPos.AdjustX(mnHalfCharWidth * -2 * (m_oColors.size() - 1));
797         }
798         else if ( m_nStart != 0 && m_nEnd != 0 )
799             // both end and start boundary marks: adjust them around the bookmark position
800             // |te|xt|
801             //  ]] [[
>>>     CID 1515457:  Integer handling issues  (SIGN_EXTENSION)
>>>     Suspicious implicit sign extension: "this->mnHalfCharWidth" with type "sal_uInt16" (16 bits, unsigned) is promoted in "this->mnHalfCharWidth * -(2 * this->m_nEnd - 1 + this->m_nPoint)" to type "int" (32 bits, signed), then sign-extended to type "long" (64 bits, signed).  If "this->mnHalfCharWidth * -(2 * this->m_nEnd - 1 + this->m_nPoint)" is greater than 0x7FFFFFFF, the upper bits of the result will all be 1.
802             aNewPos.AdjustX(mnHalfCharWidth * -(2 * m_nEnd - 1 + m_nPoint) );
803     
804         const_cast< SwTextPaintInfo& >( rInf ).SetPos( aNewPos );
805     
806         for ( const auto& it : m_oColors )
807         {


________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://u15810271.ct.sendgrid.net/ls/click?upn=HRESupC-2F2Czv4BOaCWWCy7my0P0qcxCbhZ31OYv50ypSs1kiFPuCn2xFdlMIFBirii0zZ9j2-2F9F2XPBcBm2BNgi9duPy3v-2FzgFDd2LJ-2BDKI-3DQqW8_OTq2XUZbbipYjyLSo6GRo-2FpVxQ9OzkDINu9UTS-2FQhSdO0F0jQniitrGlNxDIzPJiRih-2B0lZoCTGsDo62R3pac-2B55-2FH2vQr-2Bb-2FLbsiYBLRIjRt8HDJq3IHmb25TU7q2Pvz0yqiTq7ecVmISnSBq0zTC-2FmZtIu9bmYYODYLqSVVinGSB3SfjsBhLhUa7fcjCzCw9-2BD6N9D7ZziFSJZjk8liMyVeHQyt-2Bj9R4Mx1nB-2Fzls-3D