notmuch/bindings/python-cffi
Anton Khirnov 1317579079 python/notmuch2: do not destroy messages owned by a query
Any messages retrieved from a query - either directly via
search_messages() or indirectly via thread objects - are owned by that
query. Retrieving the same message (i.e. corresponding to the same
message ID / database object) several times will always yield the same
C object.

The caller is allowed to destroy message objects owned by a query before
the query itself - which can save memory for long-lived queries.
However, that message must then never be retrieved again from that
query.

The python-notmuch2 bindings will currently destroy every message object
in Message._destroy(), which will lead to an invalid free if the same
message is then retrieved again. E.g. the following python program leads
to libtalloc abort()ing:

import notmuch2
db   = notmuch2.Database(mode = notmuch2.Database.MODE.READ_ONLY)
t    = next(db.threads('*'))
msgs = list(zip(t.toplevel(), t.toplevel()))
msgs = list(zip(t.toplevel(), t.toplevel()))

Fix this issue by creating a subclass of Message, which is used for
"standalone" message which have to be freed by the caller. Message class
is then used only for messages descended from a query, which do not need
to be freed by the caller.
2020-06-16 08:02:02 -03:00
..
notmuch2 python/notmuch2: do not destroy messages owned by a query 2020-06-16 08:02:02 -03:00
tests python config access: fix style and KeyError bug 2020-06-15 21:50:03 -03:00
MANIFEST.in Introduce CFFI-based python bindings 2019-12-03 08:12:30 -04:00
setup.py Rename package to notmuch2 2019-12-03 08:12:30 -04:00
tox.ini Update tox.ini for python3.8 and fix pypy3.6 2020-06-15 11:25:39 -03:00