From: AMA <>
Date: Fri, 17 Feb 2012 12:04:22 +0000 (+0000)
Subject: 0022934: Wrong delete operator in IGESSelect_SelectFromDrawing.cxx / IGESSelect_Selec... 
X-Git-Tag: V6_5_3_beta1~86
X-Git-Url: http://git.dev.opencascade.org/gitweb/?a=commitdiff_plain;h=65c017004c2452a713070cea8f7bc5c45aa0cb01;p=occt-copy.git

0022934: Wrong delete operator in IGESSelect_SelectFromDrawing.cxx / IGESSelect_SelectFromSingleView.cxx
---

diff --git a/src/IGESSelect/IGESSelect_SelectFromDrawing.cxx b/src/IGESSelect/IGESSelect_SelectFromDrawing.cxx
index d2fdccde4b..1cd8858544 100755
--- a/src/IGESSelect/IGESSelect_SelectFromDrawing.cxx
+++ b/src/IGESSelect/IGESSelect_SelectFromDrawing.cxx
@@ -15,9 +15,7 @@
   Interface_EntityIterator draws = InputResult(G);
   if (draws.NbEntities() == 0) return list;
   Standard_Integer nb = G.Size();
-  char* nums = new char[nb+1];
   Standard_Integer i; // svv Jan11 2000 : porting on DEC
-  for (i = 1; i <= nb; i ++) nums[i] = 0;
 
 //  Pour chaque Drawing : prendre d une part l integralite de son contenu,
 //  (c-a-d avec le "Frame"), d autre part les entites attachees a ses vues
@@ -28,11 +26,6 @@
     list.GetOneItem (igesent);
     Interface_EntityIterator someviews = G.Shareds (draws.Value());
     list.AddList (someviews.Content());
-    for (someviews.Start(); someviews.More(); someviews.Next()) {
-      DeclareAndCast(IGESData_ViewKindEntity,igesview,someviews.Value());
-      Standard_Integer nv = G.EntityNumber(igesview);
-      if (nv > 0 && nv <= nb) nums[nv] = 1;
-    }
   }
   for (i = 1; i <= nb; i ++) {
 //    if (!G.IsPresent(i)) continue;
@@ -41,7 +34,6 @@
     Standard_Integer nv = G.EntityNumber (igesent->View());
     if (nv > 0 && nv <= nb) list.GetOneItem(igesent);
   }
-  delete nums;
   return list;
 }
 
diff --git a/src/IGESSelect/IGESSelect_SelectFromSingleView.cxx b/src/IGESSelect/IGESSelect_SelectFromSingleView.cxx
index db554e2e70..04fb9dd52b 100755
--- a/src/IGESSelect/IGESSelect_SelectFromSingleView.cxx
+++ b/src/IGESSelect/IGESSelect_SelectFromSingleView.cxx
@@ -14,13 +14,7 @@
   Interface_EntityIterator views = InputResult(G);
   if (views.NbEntities() == 0) return list;
   Standard_Integer nb = G.Size();
-  char* nums = new char[nb+1];
   Standard_Integer i; // svv Jan11 2000 : porting on DEC
-  for (i = 1; i <= nb; i ++) nums[i] = 0;
-  for (views.Start(); views.More(); views.Next()) {
-    Standard_Integer nv = G.EntityNumber(views.Value());
-    if (nv > 0 && nv <= nb) nums[nv] = 1;
-  }
   for (i = 1; i <= nb; i ++) {
 //    if (!G.IsPresent(i)) continue;
     DeclareAndCast(IGESData_IGESEntity,igesent,G.Entity(i));
@@ -28,7 +22,6 @@
     Standard_Integer nv = G.EntityNumber (igesent->View());
     if (nv > 0 && nv <= nb) list.GetOneItem(igesent);
   }
-  delete nums;
   return list;
 }