include/vcl/BitmapTools.hxx |    5 +++++
 1 file changed, 5 insertions(+)

New commits:
commit 96e1170b7c88942f500233f3ff220209ef1e80f3
Author:     Caolán McNamara <[email protected]>
AuthorDate: Sun Aug 29 16:58:11 2021 +0100
Commit:     Michael Stahl <[email protected]>
CommitDate: Wed Nov 17 10:27:40 2021 +0100

    ofz#37796 limit to numeric_limits<int>::max
    
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/121230
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <[email protected]>
    (cherry picked from commit 72da4c623baf60eb2b7073697cd36ffb3022847d)
    
    Change-Id: I6e09226fad1e566ba2758d0084042b603b84d221
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/125327
    Tested-by: Michael Stahl <[email protected]>
    Reviewed-by: Michael Stahl <[email protected]>

diff --git a/include/vcl/BitmapTools.hxx b/include/vcl/BitmapTools.hxx
index 3652bb849bf4..7c076a451a69 100644
--- a/include/vcl/BitmapTools.hxx
+++ b/include/vcl/BitmapTools.hxx
@@ -20,6 +20,7 @@
 #include <basegfx/range/b2drectangle.hxx>
 #include <o3tl/safeint.hxx>
 #include <array>
+#include <limits>
 
 class SvStream;
 namespace basegfx { class B2DHomMatrix; }
@@ -51,6 +52,10 @@ public:
           mnBitCount(nBitCount)
     {
         assert(nBitCount == 24 || nBitCount == 32);
+        if (rSize.getWidth() > std::numeric_limits<sal_Int32>::max() || 
rSize.getWidth() < 0)
+            throw std::bad_alloc();
+        if (rSize.getHeight() > std::numeric_limits<sal_Int32>::max() || 
rSize.getHeight() < 0)
+            throw std::bad_alloc();
         sal_Int32 nRowSize, nDataSize;
         if (o3tl::checked_multiply<sal_Int32>(rSize.getWidth(), nBitCount/8, 
nRowSize) ||
             o3tl::checked_multiply<sal_Int32>(nRowSize, rSize.getHeight(), 
nDataSize) ||

Reply via email to