Mirror/zeek
1
0
Fork 0
mirror of https://github.com/zeek/zeek.git synced 2025-10-10 18:48:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

Memory leak fixes.

Browse source 
Both related to Val lists constructed as arguments to events that were
not freed because the event function was never called (e.g. no handlers).

Addresses #574
This commit is contained in:
Jon Siwek 2011-09-15 16:08:48 -05:00
parent d656e2a8c4
commit 8697615b81
2 changed files with 8 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/Func.cc
View file

@ -276,6 +276,8 @@ Val* BroFunc::Call(val_list* args, Frame* parent) const
{ {
// Can only happen for events. // Can only happen for events.
assert(IsEvent()); assert(IsEvent());
loop_over_list(*args, i)
Unref((*args)[i]);
return 0 ; return 0 ;
} }

6
src/Reporter.cc
View file

@ -303,6 +303,12 @@ void Reporter::DoLog(const char* prefix, EventHandlerPtr event, FILE* out, Conne
s += "\n"; s += "\n";
fprintf(out, "%s", s.c_str()); fprintf(out, "%s", s.c_str());
if ( addl )
{
loop_over_list(*addl, i)
Unref((*addl)[i]);
}
} }
if ( alloced ) if ( alloced )