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

Fix potential mem leak in remote function/event unserialization.

Browse source 
I say potential because a code path to get in the required state is
not obvious (if one even exists).
This commit is contained in:
Jon Siwek 2014-05-28 14:55:24 -05:00
parent ed7273ccf1
commit 8383828b02
3 changed files with 14 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

11
CHANGES
View file

@ -1,4 +1,15 @@
2.3-beta-4 | 2014-05-28 14:55:24 -0500
* Fix potential mem leak in remote function/event unserialization.
(Jon Siwek)
* Fix reference counting bug in table coercion expressions (Jon Siwek)
* Fix an "unused value" warning. (Jon Siwek)
* Remove a duplicate unit test baseline dir. (Jon Siwek)
2.3-beta | 2014-05-19 16:36:50 -0500 2.3-beta | 2014-05-19 16:36:50 -0500
* Release 2.3-beta * Release 2.3-beta

2
VERSION
View file

@ -1 +1 @@
2.3-beta 2.3-beta-4

2
src/RemoteSerializer.cc
View file

@ -2833,6 +2833,7 @@ void RemoteSerializer::GotEvent(const char* name, double time,
if ( ! current_peer ) if ( ! current_peer )
{ {
Error("unserialized event from unknown peer"); Error("unserialized event from unknown peer");
delete_vals(args);
return; return;
} }
@ -2882,6 +2883,7 @@ void RemoteSerializer::GotFunctionCall(const char* name, double time,
if ( ! current_peer ) if ( ! current_peer )
{ {
Error("unserialized function from unknown peer"); Error("unserialized function from unknown peer");
delete_vals(args);
return; return;
} }