]> git.notmuchmail.org Git - notmuch/commit
Avoid giving GMime a NULL MIME-stream filter.
authorCarl Worth <cworth@cworth.org>
Sat, 5 Jun 2010 15:40:26 +0000 (08:40 -0700)
committerCarl Worth <cworth@cworth.org>
Sat, 5 Jun 2010 15:40:26 +0000 (08:40 -0700)
commit9c7668bdb51e4739eecda83b6452ef668ed0efa3
tree4a8139cf7179186245a732f0df9b2ba1fc753a5a
parent42e146a3a20c1ca2e1a9d6fd2d5e5e9d03a06641
Avoid giving GMime a NULL MIME-stream filter.

Micah Anderson reported an issue where a message failed to display in
the emacs interface, (it instead gave an error, "json-read-string: Bad
string format").

Micah tracked this down to the json output from "notmuch show" being
interrupted by a GMime error message:

gmime-CRITICAL **: g_mime_stream_filter_add: assertion
`GMIME_IS_FILTER (filter)

I tracked this down further to notmuch passing a NULL value to
g_mime_stream_filter_add. And this was due to calling
g_mime_filter_charset_new with a value of "unknown-8bit".

So we add a test message withe a Conten-Type of "text/plain;
charset=unknown-8bit" from Micah's message. Then we fix "notmuch show"
to test for NULL before calling g_mime_stream_filter_add. Bug fixed.
notmuch-show.c
test/notmuch-test