2005-11-14 [wwp] 1.9.100cvs12
authorTristan Chabredier <wwp@claws-mail.org>
Mon, 14 Nov 2005 08:58:18 +0000 (08:58 +0000)
committerTristan Chabredier <wwp@claws-mail.org>
Mon, 14 Nov 2005 08:58:18 +0000 (08:58 +0000)
* src/folder.c
prevent from crashing when copying/moving message(s) from and
to the same folder

ChangeLog
PATCHSETS
configure.ac
src/folder.c

index f7da936..b2d22ff 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2005-11-14 [wwp]       1.9.100cvs12
+
+       * src/folder.c
+               prevent from crashing when copying/moving message(s) from and
+               to the same folder
+
 2005-11-14 [paul]      1.9.100cvs11
 
        * src/plugins/clamav/Makefile.am
index b616e3a..fd33dbe 100644 (file)
--- a/PATCHSETS
+++ b/PATCHSETS
 ( cvs diff -u -r 1.1.4.24 -r 1.1.4.25 src/etpan/imap-thread.c;  ) > 1.9.100cvs9.patchset
 ( cvs diff -u -r 1.654.2.1039 -r 1.654.2.1040 configure.ac;  cvs diff -u -r 1.5.2.2 -r 1.5.2.3 src/plugins/spamassassin/Makefile.am;  cvs diff -u -r 1.18.2.11 -r 1.18.2.12 src/plugins/spamassassin/spamassassin.c;  cvs diff -u -r 1.4.2.3 -r 1.4.2.4 src/plugins/spamassassin/spamassassin.h;  cvs diff -u -r 1.23.2.14 -r 1.23.2.15 src/plugins/spamassassin/spamassassin_gtk.c;  ) > 1.9.100cvs10.patchset
 ( cvs diff -u -r 1.4.2.3 -r 1.4.2.4 src/plugins/clamav/Makefile.am;  cvs diff -u -r 1.13.2.11 -r 1.13.2.12 src/plugins/clamav/clamav_plugin.c;  cvs diff -u -r 1.3.2.2 -r 1.3.2.3 src/plugins/clamav/clamav_plugin.h;  cvs diff -u -r 1.9.2.11 -r 1.9.2.12 src/plugins/clamav/clamav_plugin_gtk.c;  ) > 1.9.100cvs11.patchset
+( cvs diff -u -r 1.213.2.68 -r 1.213.2.69 src/folder.c;  ) > 1.9.100cvs12.patchset
index cb42722..297bc1b 100644 (file)
@@ -11,7 +11,7 @@ MINOR_VERSION=9
 MICRO_VERSION=100
 INTERFACE_AGE=0
 BINARY_AGE=0
-EXTRA_VERSION=11
+EXTRA_VERSION=12
 EXTRA_RELEASE=
 EXTRA_GTK2_VERSION=
 
index 352f1ba..d88beaf 100644 (file)
@@ -2744,8 +2744,20 @@ static gint do_copy_msgs(FolderItem *dest, GSList *msglist, gboolean remove_sour
                        return -1;
                }
        } else {
-               for (l = msglist ; l != NULL ; l = g_slist_next(l)) {
-                       MsgInfo * msginfo = (MsgInfo *) l->data;
+               MsgInfo * msginfo;
+               l = msglist;
+
+               /* immediately stop if src and dest folders are identical */
+               if (l != NULL) {
+                       msginfo = (MsgInfo *) l->data;
+                       if (msginfo != NULL && msginfo->folder == dest) {
+                               g_relation_destroy(relation);
+                               return -1;
+                       }
+               }
+
+               for (; l != NULL ; l = g_slist_next(l)) {
+                       msginfo = (MsgInfo *) l->data;
 
                        num = folder->klass->copy_msg(folder, dest, msginfo);
                        if (num > 0)