diff options
| author | Milan Crha <mcrha@redhat.com> | 2014-03-26 23:48:03 +0800 |
|---|---|---|
| committer | Milan Crha <mcrha@redhat.com> | 2014-03-26 23:48:03 +0800 |
| commit | 733462ca84044f3fe956b4578870ab7698242fd6 (patch) | |
| tree | 2c9777fea09c0fc00177d230561584e27ac8a239 /mail | |
| parent | a2a29549268e8d75d0cc8c0f8de4d0542b1889b2 (diff) | |
| download | gsoc2013-evolution-733462ca84044f3fe956b4578870ab7698242fd6.tar gsoc2013-evolution-733462ca84044f3fe956b4578870ab7698242fd6.tar.gz gsoc2013-evolution-733462ca84044f3fe956b4578870ab7698242fd6.tar.bz2 gsoc2013-evolution-733462ca84044f3fe956b4578870ab7698242fd6.tar.lz gsoc2013-evolution-733462ca84044f3fe956b4578870ab7698242fd6.tar.xz gsoc2013-evolution-733462ca84044f3fe956b4578870ab7698242fd6.tar.zst gsoc2013-evolution-733462ca84044f3fe956b4578870ab7698242fd6.zip | |
Fix two memory leaks
a) ETableColumnSpecification: the 'title' property is always overwritten
during the spec file loading, thus no need to allocate it here
b) the 'uids' are leaked on message move, because the GPtrArray is
supposed to be always unreffed
Diffstat (limited to 'mail')
| -rw-r--r-- | mail/e-mail-reader.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/mail/e-mail-reader.c b/mail/e-mail-reader.c index e1c477ed12..b7d380ac8d 100644 --- a/mail/e-mail-reader.c +++ b/mail/e-mail-reader.c @@ -868,20 +868,16 @@ action_mail_move_cb (GtkAction *action, g_free (default_xfer_messages_uri); default_xfer_messages_uri = g_strdup (uri); - if (uri != NULL) { + if (uri != NULL) mail_transfer_messages ( session, folder, uids, TRUE, uri, 0, NULL, NULL); - uids = NULL; - } exit: - if (uids != NULL) - g_ptr_array_unref (uids); - gtk_widget_destroy (dialog); g_clear_object (&folder); + g_ptr_array_unref (uids); } static void |