migrate to MemBuffer class from New() to remove leak

src/pefile.cpp

@@ -1731,7 +1731,11 @@ void PeFile::Resource::build(const upx_rnode *node, unsigned &bpos,
 
 upx_byte* PeFile::Resource::build()
 {
-    newstart = New(upx_byte, dirsize());
+    mem.dealloc();
+    newstart = nullptr;
+    if (dirsize()) {
+      mem.alloc(dirsize());
+      newstart = static_cast<upx_byte *>(mem.getVoidPtr());
       unsigned bpos = 0,spos = dsize;
       build(root,bpos,spos,0);
 
@@ -1739,6 +1743,7 @@ upx_byte *PeFile::Resource::build()
       // up to 2 bytes to make valgrind happy
       while (spos < dirsize())
           newstart[spos++] = 0;
+    }
 
     return newstart;
 }
@@ -1825,9 +1830,10 @@ void PeFile::processResources(Resource *res,unsigned newaddr)
     while (res->next())
         if (res->newoffs())
             res->newoffs() += newaddr;
+    if (res->dirsize()) {
       upx_byte *p = res->build();
       memcpy(oresources,p,res->dirsize());
-    delete [] p;
+    }
 }
 
 static bool match(unsigned itype, const unsigned char *ntype,
@@ -2757,12 +2763,13 @@ void PeFile::rebuildResources(upx_byte *& extrainfo, unsigned lastvaddr)
                 icondir_count = 0;
             }
         }
+    if (res.dirsize()) {
       upx_byte *p = res.build();
       OCHECK(obuf + ODADDR(PEDIR_RESOURCE) - rvamin, 16);
       // write back when the original is zeroed
       if (get_le32(obuf + ODADDR(PEDIR_RESOURCE) - rvamin + 12) == 0)
         omemcpy(obuf + ODADDR(PEDIR_RESOURCE) - rvamin, p, res.dirsize());
-    delete [] p;
+    }
 }
 
 template <typename LEXX, typename ord_mask_t>

src/pefile.h

@@ -29,6 +29,8 @@
 #ifndef __UPX_PEFILE_H
 #define __UPX_PEFILE_H 1
 
+#include "mem.h"
+
 
 /*************************************************************************
 // general/pe handling
@@ -341,6 +343,7 @@ protected:
         struct upx_rbranch;
         struct upx_rleaf;
 
+        MemBuffer mem;
         const upx_byte *start;
         upx_byte   *newstart;
         upx_rnode  *root;