2011-01-21 [colin] 3.7.8cvs41
authorColin Leroy <colin@colino.net>
Fri, 21 Jan 2011 10:46:02 +0000 (10:46 +0000)
committerColin Leroy <colin@colino.net>
Fri, 21 Jan 2011 10:46:02 +0000 (10:46 +0000)
* src/procmime.c
Fix bug #2050, Clawsmail sends return receipt every seconds
We forgot to close the file when parsing
disposition-notification fields, which made unlinking of the
corresponding emails in tempfolder/processing impossible in
Windows. Causing them to be re-fetched every time.

ChangeLog
PATCHSETS
configure.ac
src/procmime.c

index e8c2b66..7c52b61 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2011-01-21 [colin]     3.7.8cvs41
+
+       * src/procmime.c
+               Fix bug #2050, Clawsmail sends return receipt every seconds
+               We forgot to close the file when parsing 
+               disposition-notification fields, which made unlinking of the
+               corresponding emails in tempfolder/processing impossible in
+               Windows. Causing them to be re-fetched every time.
+
 2011-01-21 [colin]     3.7.8cvs40
 
        * claws-mail.desktop
index dd64d20..6953c05 100644 (file)
--- a/PATCHSETS
+++ b/PATCHSETS
 ( cvs diff -u -r 1.1.2.2 -r 1.1.2.3 claws-mail.desktop;  ) > 3.7.8cvs38.patchset
 ( cvs diff -u -r 1.18.2.63 -r 1.18.2.64 src/plugins/spamassassin/spamassassin.c;  ) > 3.7.8cvs39.patchset
 ( cvs diff -u -r 1.1.2.3 -r 1.1.2.4 claws-mail.desktop;  ) > 3.7.8cvs40.patchset
+( cvs diff -u -r 1.49.2.136 -r 1.49.2.137 src/procmime.c;  ) > 3.7.8cvs41.patchset
index 11eb8f6..5b60cd9 100644 (file)
@@ -12,7 +12,7 @@ MINOR_VERSION=7
 MICRO_VERSION=8
 INTERFACE_AGE=0
 BINARY_AGE=0
-EXTRA_VERSION=40
+EXTRA_VERSION=41
 EXTRA_RELEASE=
 EXTRA_GTK2_VERSION=
 
index 4b13271..7783f99 100644 (file)
@@ -1464,8 +1464,10 @@ static void procmime_parse_disposition_notification(MimeInfo *mimeinfo,
        } else {
                procheader_get_header_fields(fp, hentry);
        }
+    
+        fclose(fp);
 
-       if (!hentry[0].body || !hentry[1].body) {
+       if (!hentry[0].body || !hentry[1].body) {
                debug_print("MsgId %s, Disp %s\n",
                        hentry[0].body ? hentry[0].body:"(nil)",
                        hentry[1].body ? hentry[1].body:"(nil)");