keplerproject/luasql
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fixed bug [#1834] Double free (sqlite3_finalize) in ls_sqlite3.c

Browse Source 
Only part of the included patch was applied, since the test for NULL was redundant (tks to Savin Zlobec and Marc Nijdam)
This commit is contained in:
carregal 2007-10-16 15:23:48 +00:00
parent 64eb574aa4
commit cadf41be20
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/ls_sqlite3.c
View File

@ -3,7 +3,7 @@
** Author: Tiago Dionizio, Eduardo Quintao
** See Copyright Notice in license.html
** $Id: ls_sqlite3.c,v 1.5 2007/09/12 00:26:10 carregal Exp $
** $Id: ls_sqlite3.c,v 1.6 2007/10/16 15:23:48 carregal Exp $
*/
#include <stdio.h>
@ -102,6 +102,7 @@ static int finalize(lua_State *L, cur_data *cur) {
lua_concat(L, 2);
return 2;
}
cur->sql_vm = NULL;
lua_pushnil(L);
return 1;
}
@ -184,7 +185,6 @@ static int cur_close(lua_State *L)
/* Nullify structure fields. */
cur->closed = 1;
sqlite3_finalize(cur->sql_vm);
/* Decrement cursor counter on connection object */
lua_rawgeti (L, LUA_REGISTRYINDEX, cur->conn);
conn = lua_touserdata (L, -1);