From 0f26ed547610df5ebf2a127b9c6dfbc7489064c4 Mon Sep 17 00:00:00 2001
From: kgv <kgv@opencascade.com>
Date: Sun, 10 Jan 2021 00:04:32 +0300
Subject: [PATCH] 0032044: Foundation Classes - NCollection_Array2::Resize()
 does not allocate pointer array

NCollection_Array2::Resize() has been fixed to set myDeletable before calling Allocate().
---
 src/NCollection/NCollection_Array2.hxx   |  6 +++---
 src/QANCollection/QANCollection_Test.cxx | 15 +++++++++++++--
 2 files changed, 16 insertions(+), 5 deletions(-)

diff --git a/src/NCollection/NCollection_Array2.hxx b/src/NCollection/NCollection_Array2.hxx
index 1aa317b2c8..4e6eb37c40 100644
--- a/src/NCollection/NCollection_Array2.hxx
+++ b/src/NCollection/NCollection_Array2.hxx
@@ -349,10 +349,11 @@ public:
     }
     delete[] aTableOld;
 
+    const Standard_Boolean wasDeletable = myDeletable;
+    myDeletable = Standard_True;
     Allocate();
     if (!theToCopyData)
     {
-      myDeletable = Standard_True;
       return;
     }
 
@@ -366,11 +367,10 @@ public:
       }
     }
 
-    if (myDeletable)
+    if (wasDeletable)
     {
       delete[] aStartOld;
     }
-    myDeletable = Standard_True;
   }
 
   //! Destructor - releases the memory
diff --git a/src/QANCollection/QANCollection_Test.cxx b/src/QANCollection/QANCollection_Test.cxx
index 3454695dac..63432cc57b 100644
--- a/src/QANCollection/QANCollection_Test.cxx
+++ b/src/QANCollection/QANCollection_Test.cxx
@@ -193,6 +193,8 @@ static void TestArray2  (QANCollection_Array2Func&     theA2)
   Handle(QANCollection_HArray2Func) aHa = new QANCollection_HArray2Func(aCArr);
   // Assign
   AssignCollection (aHa->ChangeArray2(), theA2);
+
+  delete[] rBlock;
 }
 
 // ===================== Test methods of List type ==========================
@@ -755,8 +757,17 @@ static Standard_Integer QANColTestArray2(Draw_Interpretor& di, Standard_Integer
     }
   }
 
-  QANCollection_Array2Func anArr2Copy2 = anArr2;
-  anArr2Copy2.Resize (LowerRow - 1, UpperRow - 1, LowerCol + 1, UpperCol + 1, false);
+  {
+    QANCollection_Array2Func anArr2Copy2 = anArr2;
+    anArr2Copy2.Resize (LowerRow - 1, UpperRow - 1, LowerCol + 1, UpperCol + 1, false);
+  }
+
+  {
+    // empty array resize
+    QANCollection_Array2Func anArr2Copy3;
+    anArr2Copy3.Resize (LowerRow, UpperRow, LowerCol, UpperCol, false);
+    anArr2Copy3 = anArr2;
+  }
 
   return 0;
 }