[cvs] bogofilter/src bogomain.c, 1.10, 1.11 datastore_db_trans.c, 1.18, 1.19 datastore_sqlite.c, 1.17, 1.18 datastore_tdb.c, 1.45, 1.46

[Matthias Andree]

Matthias Andree <matthias.andree at gmx.de> writes:

>> Modified Files:
>> 	bogomain.c datastore_db_trans.c datastore_sqlite.c 
>> 	datastore_tdb.c 
>> Log Message:
>> Additionall free/xfree cleanup.
>
> Spraying xfree() everywhere free() is with a watering can isn't a good
> idea. free() is appropriate when we know the memory is definitely
> allocated, for instance, after malloc(), and perhaps in code that is
> executed only once during the lifetime of a program run.

In fact, datastore_sqlite.c does not need a single xfree() - everything
that we free() is _guaranteed_ to not be NULL. Changes reverted to free().

-- 
Matthias Andree
_______________________________________________
Bogofilter-dev mailing list
Bogofilter-dev at bogofilter.org
http://www.bogofilter.org/mailman/listinfo/bogofilter-dev