zorba-coders team mailing list archive

[Chris Hillery <chillery+launchpad@xxxxxxxxx>]

I believe there's a memory leak. valgrind reports a large block (around 73k) of memory lost at line 847 in sqlite_module.cpp for most of the tests. This is the call to sqlite3_open_v2(). After a little debugging, I see that sqlite3_close() is being called in ConnMap::destroy(). However, according to http://www.sqlite.org/c3ref/close.html , calling sqlite3_close() defers all memory cleanup if there are any outstanding prepared statement objects.

Prepared statements are cleaned up with the call to sqlite3_finalize() in StmtMap::destroy(); however, this function does not get called when executing one of the test queries. Hence, the entire database connection is leaked.
-- 
https://code.launchpad.net/~zorba-coders/zorba/zorba-for-sqlite/+merge/139108
Your team Zorba Coders is subscribed to branch lp:zorba.