[Libreoffice-commits] core.git: vcl/source

[Caolán McNamara (via logerrit)]

 vcl/source/gdi/pdfwriter_impl.cxx |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 4ec8787b41fa366182cc97e165943d6af130f304
Author:     Caolán McNamara <caol...@redhat.com>
AuthorDate: Tue May 2 10:47:53 2023 +0100
Commit:     Caolán McNamara <caol...@redhat.com>
CommitDate: Tue May 2 13:53:50 2023 +0200

    ofz#58551 Conditional jump or move depends on uninitialised value
    
    Change-Id: Iee5d82fff5edf14dd4f8adeabc305e96dd0ddd8d
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/151260
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caol...@redhat.com>

diff --git a/vcl/source/gdi/pdfwriter_impl.cxx 
b/vcl/source/gdi/pdfwriter_impl.cxx
index ce70dedc8b21..7d51e3d916d1 100644
--- a/vcl/source/gdi/pdfwriter_impl.cxx
+++ b/vcl/source/gdi/pdfwriter_impl.cxx
@@ -2872,7 +2872,7 @@ sal_Int32 PDFWriterImpl::createToUnicodeCMap( sal_uInt8 
const * pEncoding,
 {
     int nMapped = 0;
     for (auto n = 0u; n < nGlyphs; ++n)
-        if( pCodeUnits[pEncToUnicodeIndex[n]] && pCodeUnitsPerGlyph[n] )
+        if (pCodeUnitsPerGlyph[n] && pCodeUnits[pEncToUnicodeIndex[n]])
             nMapped++;
 
     if( nMapped == 0 )
@@ -2900,7 +2900,7 @@ sal_Int32 PDFWriterImpl::createToUnicodeCMap( sal_uInt8 
const * pEncoding,
     int nCount = 0;
     for (auto n = 0u; n < nGlyphs; ++n)
     {
-        if( pCodeUnits[pEncToUnicodeIndex[n]] && pCodeUnitsPerGlyph[n] )
+        if (pCodeUnitsPerGlyph[n] && pCodeUnits[pEncToUnicodeIndex[n]])
         {
             if( (nCount % 100) == 0 )
             {