clear memory in initialization for ns-3-simu

4 views
Skip to first unread message

theh...@gmail.com

unread,
Jun 22, 2009, 11:50:38 AM6/22/09
to theh...@gmail.com, ns-3-r...@googlegroups.com
Reviewers: ,

Message:
Hi,

This patch fixes the memory initialization problem of KingsleyAlloc
constructor of src/process-manager/alloc.cc. Patch is against to
ns-3-simu repository.

Description:
This changeset is created agiainst
http://code.nsnam.org/mathieu/ns-3-simu/
rev 4583.

These modification intend to support zebra (quagga) integration in ns-3.

Please review this at http://codereview.appspot.com/82060

Affected files:
M src/process-manager/alloc.cc


Index: src/process-manager/alloc.cc
===================================================================
--- a/src/process-manager/alloc.cc
+++ b/src/process-manager/alloc.cc
@@ -54,6 +54,7 @@
: m_defaultMmapSize (1<<15)
{
NS_LOG_FUNCTION (this);
+ memset(m_buckets, 0, sizeof(m_buckets));
}
KingsleyAlloc::~KingsleyAlloc ()
{


Faker Moatamri

unread,
Jul 1, 2009, 5:25:00 AM7/1/09
to ns-3-r...@googlegroups.com, ns-dev...@isi.edu
Hi all,
I have been reviewing the wimax code and I commited today the comments
that I did:
http://codereview.appspot.com/75057/show
Even though I didn't finish reviewing all the code, I commited the
comments on the already reviewed code to enable the WiMax developers to
view my comments and to do the relevant changes.
Best regards
Faker Moatamri

Mathieu Lacage

unread,
Sep 4, 2009, 9:02:02 AM9/4/09
to ns-3-r...@googlegroups.com
merged in http://code.nsnam.org/mathieu/ns-3-simu.

thank you very much !
Mathieu
--
Mathieu Lacage <mathieu...@gmail.com>
Reply all
Reply to author
Forward
0 new messages