diff options
| author |  Carl Worth <cworth@cworth.org> | 2009-11-17 08:50:14 -0800 |
|---|---|---|
| committer | Carl Worth <cworth@cworth.org> | 2009-11-17 08:50:14 -0800 |
| commit | 0dab6a2c1e8b59ac681f48417c0f06e88f73cd3e (patch) | |
| tree | 7df803a39e69c138be9572d601ff0b22c6951493 /lib | |
| parent | 6b135cae1e6e29a5ce8d8acb0cde2b50628ab8f1 (diff) | |
add_message: Avoid a memory leak when user holds on to message return.
When this function was originally written, the 'message' object was
always destroyed locally, so I thought it would be good to use a NULL
talloc context to make it more obvious if there was any leak.
Since then, however, this function has been changed to optionally
return the added message, and in that case we *don't* free the message
locally, so let's let the database be the talloc context.
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/database.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/lib/database.cc b/lib/database.cc index 8177c6b6..583a16ed 100644 --- a/lib/database.cc +++ b/lib/database.cc @@ -911,8 +911,7 @@ notmuch_database_add_message (notmuch_database_t *notmuch, * (which may or may not reference an existing document in the * database). */ - /* Use NULL for owner since we want to free this locally. */ - message = _notmuch_message_create_for_message_id (NULL, + message = _notmuch_message_create_for_message_id (database, notmuch, message_id, &private_status); |