[Liblas-commits] hg-main-tree: dumb fix for memleak

[liblas-commits at liblas.org]

details:   http://hg.libpc.orghg-main-tree/rev/483284c82936
changeset: 1089:483284c82936
user:      Michael P. Gerlek <mpg at flaxen.com>
date:      Mon Aug 15 07:34:03 2011 -0700
description:
dumb fix for memleak

diffstat:

 src/drivers/las/LasHeaderReader.cpp |  7 +++++--
 1 files changed, 5 insertions(+), 2 deletions(-)

diffs (25 lines):

diff -r 796b179c92df -r 483284c82936 src/drivers/las/LasHeaderReader.cpp
--- a/src/drivers/las/LasHeaderReader.cpp	Mon Aug 15 07:12:43 2011 -0700
+++ b/src/drivers/las/LasHeaderReader.cpp	Mon Aug 15 07:34:03 2011 -0700
@@ -342,9 +342,10 @@
     std::string description = "";
 
     {
-        boost::scoped_ptr<boost::uint8_t> buf1( new boost::uint8_t[pdal::drivers::las::VariableLengthRecord::s_headerLength]);
+        // BUG: this should be a scoped ptr
+        boost::uint8_t* buf1 = new boost::uint8_t[pdal::drivers::las::VariableLengthRecord::s_headerLength];
         Utils::read_n(buf1, m_istream, pdal::drivers::las::VariableLengthRecord::s_headerLength);
-        boost::uint8_t* p1 = buf1.get();
+        boost::uint8_t* p1 = buf1;
 
         reserved = Utils::read_field<boost::uint16_t>(p1);
         boost::ignore_unused_variable_warning(reserved);
@@ -359,6 +360,8 @@
         boost::uint8_t description_data[32];
         Utils::read_array_field(p1, description_data, 32);
         description = VariableLengthRecord::bytes2string(description_data, 32);
+
+        delete[] buf1;
     }
 
     boost::uint8_t* data = new boost::uint8_t[recordLenAfterHeader];