Fixes for a few memory leaks. #136

jen140 · Jan 19, 2016

Found a few memory leaks with cppcheck, after reading code I've added a few free's. In case of libstemmer, i'm not sure if the sb_stemmer_delete() can be used, because of the second check (stemmer->close == 0).

Have a nice day.

swh · Apr 20, 2016

Sorry for the delay, I've been super-busy!

Just merged it in, and I'm about to test on OS X.

swh merged commit e867ee0 into 4store:master Apr 20, 2016

jen140	`fixed a few memory leaks in backend.c`	`17bc3a5`
jen140	`fixed memory leaks in tree.c`	`993d368`
jen140	`fixed memory leaks in tlist.c`	`0d36f43`
jen140	`fixed memory leaks in tbchain.c`	`5f5b26b`
jen140	`fixed memory leaks in rhash.c`	`de48cfd`
jen140	`fixed memory leaks in ptable.c`	`a733019`
jen140	`fixed memory leaks in mhash.c`	`429083b`
jen140	`fixed memory leaks in metadata.c`	`9d4ccb9`
jen140	`fixed memory leaks in list.c`	`d767453`
jen140	`fixed memory leaks in chain.c`	`943deea`
jen140	`fixed a memory leak in libstemmer.c`	`a3e2c28`
jen140	`fixed a memory leak in results.c`	`2ed09a4`
jen140	`fixed a memory leak in umac.c`	`76bc16a`
jen140	`fixed a memory leak in libstemmer_utf8.c`	`2596f01`

4store/4store

Fixes for a few memory leaks. #136

jen140 commented Jan 19, 2016

jen140 added some commits Jan 19, 2016

swh merged commit `e867ee0` into 4store:master Apr 20, 2016

swh commented Apr 20, 2016

4store/4store

Join GitHub today

Fixes for a few memory leaks. #136

Conversation

jen140 commented Jan 19, 2016

jen140 added some commits Jan 19, 2016

swh merged commit e867ee0 into 4store:master Apr 20, 2016

swh commented Apr 20, 2016

swh merged commit `e867ee0` into 4store:master Apr 20, 2016