michael/mlua
michael
/
mlua
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Enable stack leak panic universally 

This will potentially panic on Drop of a `Lua` instance, which may be an abort
if this is a double panic, but that is more desirable than such a bug being
hidden.
This commit is contained in:
kyren 2018-03-08 12:36:03 -05:00
parent d06890afc6
commit 431f84012a
1 changed files with 2 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/lua.rs
View File

@ -52,12 +52,8 @@ impl Drop for Lua {
fn drop(&mut self) { fn drop(&mut self) {
unsafe { unsafe {
if !self.ephemeral { if !self.ephemeral {
if cfg!(test) {
let top = ffi::lua_gettop(self.state); let top = ffi::lua_gettop(self.state);
if top != 0 { rlua_assert!(top == 0, "stack leak detected, stack top is {}", top);
rlua_abort!("Lua stack leak detected, stack top is {}", top);
}
}
let extra_data = *(ffi::lua_getextraspace(self.state) as *mut *mut ExtraData); let extra_data = *(ffi::lua_getextraspace(self.state) as *mut *mut ExtraData);
*(*extra_data).registry_unref_list.lock().unwrap() = None; *(*extra_data).registry_unref_list.lock().unwrap() = None;