[Xorp-cvs] [greearb/xorp.ct] e9f50d: libxorp/heap: Fix long-standing memory corruption...

noreply at github.com noreply at github.com
Mon Aug 16 12:44:00 PDT 2010


Branch: refs/heads/master
Home:   http://github.com/greearb/xorp.ct

Commit: e9f50df436d634c819a5e8eb2fb9e4084d02fe37
    http://github.com/greearb/xorp.ct/commit/e9f50df436d634c819a5e8eb2fb9e4084d02fe37
Author: Ben Greear <greearb at candelatech.com>
Date:   2010-08-16 (Mon, 16 Aug 2010)

Changed paths:
  M xorp/libxorp/heap.cc
  M xorp/libxorp/heap.hh
  M xorp/libxorp/timer.cc

Log Message:
-----------
libxorp/heap:  Fix long-standing memory corruption.

We used to leak Heap objects on purpose because destroying
them caused crashes.  It seems the crash was either because
of un-initialized memory or (more likely I think), using
memset() on this in the constructor.  Either way, this
patch appears to fix the problem and we run clean under
valgrind (as far as Heaps are concerned, at least).

Signed-off-by: Ben Greear <greearb at candelatech.com>




More information about the Xorp-cvs mailing list