[PATCH] lib: fix memory leak of field processor objects

Tomi Ollila tomi.ollila at iki.fi
Fri Jun 10 04:13:26 PDT 2016


On Fri, Jun 10 2016, David Bremner <david at tethera.net> wrote:

> The field processor objects need to be deallocated explicitly just like
> the range processors (or a talloc destructor defined).
> ---
>  lib/database.cc | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/lib/database.cc b/lib/database.cc
> index 2b2d821..afafe88 100644
> --- a/lib/database.cc
> +++ b/lib/database.cc
> @@ -1107,6 +1107,13 @@ notmuch_database_close (notmuch_database_t *notmuch)
>      delete notmuch->last_mod_range_processor;
>      notmuch->last_mod_range_processor = NULL;
>  
> +#if HAVE_XAPIAN_FIELD_PROCESSOR
> +    delete notmuch->date_field_processor;
> +    notmuch->date_field_processor = NULL;
> +    delete notmuch->query_field_processor;
> +    notmuch->query_field_processor = NULL;
> +#endif
> +

... ok, it was this I marked trivial and not the one I wrote in
id:m2r3c5ti1c.fsf at guru.guru-group.fi (the same thought about triviality may
apply, though)


>      return status;
>  }
>  
> -- 
> 2.8.1


More information about the notmuch mailing list