Skip to content

Commit

Permalink
Don't free lua state in lua_parser. (#632)
Browse files Browse the repository at this point in the history
The lua_state is being freed/closed on the falco side, so it shouldn't
be closed here.
  • Loading branch information
mstemm committed Jul 19, 2016
1 parent 99245c1 commit 821d4a5
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions userspace/libsinsp/lua_parser.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -64,13 +64,10 @@ sinsp_filter* lua_parser::get_filter(bool reset_filter)
}
lua_parser::~lua_parser()
{
if(m_ls)
{
lua_close(m_ls);
m_ls = NULL;
}
delete m_filter;
// The lua state is not considered owned by this object, so
// not freeing it.

delete m_filter;
}


0 comments on commit 821d4a5

Please sign in to comment.