mirror of
https://git.notmuchmail.org/git/notmuch
synced 2024-12-22 17:34:54 +01:00
lib: call g_mime_init() from notmuch_database_open()
As reported in id:"CAEbOPGyuHnz4BPtDutnTPUHcP3eYcRCRkXhYoJR43RUMw671+g@mail.gmail.com" sometimes gmime tries to access a NULL pointer, e.g. g_mime_iconv_open() tries to access iconv_cache that is NULL if g_mime_init() is not called. This causes notmuch to segfault when calling gmime functions. Calling g_mime_init() initializes iconv_cache and others variables needed by gmime, making sure they are initialized when notmuch calls gmime functions.
This commit is contained in:
parent
bee7cb9472
commit
f80d4f9f6d
1 changed files with 9 additions and 0 deletions
|
@ -28,6 +28,8 @@
|
|||
#include <glib.h> /* g_free, GPtrArray, GHashTable */
|
||||
#include <glib-object.h> /* g_type_init */
|
||||
|
||||
#include <gmime/gmime.h> /* g_mime_init */
|
||||
|
||||
using namespace std;
|
||||
|
||||
#define ARRAY_SIZE(arr) (sizeof (arr) / sizeof (arr[0]))
|
||||
|
@ -585,6 +587,7 @@ notmuch_database_open (const char *path,
|
|||
struct stat st;
|
||||
int err;
|
||||
unsigned int i, version;
|
||||
static int initialized = 0;
|
||||
|
||||
if (asprintf (¬much_path, "%s/%s", path, ".notmuch") == -1) {
|
||||
notmuch_path = NULL;
|
||||
|
@ -608,6 +611,12 @@ notmuch_database_open (const char *path,
|
|||
/* Initialize the GLib type system and threads */
|
||||
g_type_init ();
|
||||
|
||||
/* Initialize gmime */
|
||||
if (! initialized) {
|
||||
g_mime_init (0);
|
||||
initialized = 1;
|
||||
}
|
||||
|
||||
notmuch = talloc (NULL, notmuch_database_t);
|
||||
notmuch->exception_reported = FALSE;
|
||||
notmuch->path = talloc_strdup (notmuch, path);
|
||||
|
|
Loading…
Reference in a new issue