[fetchmail-devel] [PATCH] imap trail broken

[Sunil S. <sh...@bo...>]

Hi,

Changes in imap_trail() in r4396 are incorrect and can cause segfault.
This patch should fix this.

===============================================================================
Index: fetchmail/imap.c
===================================================================
--- fetchmail/imap.c	(revision 4410)
+++ fetchmail/imap.c	(working copy)
@@ -1055,8 +1055,9 @@
 
 	/* UW IMAP returns "OK FETCH", Cyrus returns "OK Completed" */
 	if (strncmp(buf, tag, strlen(tag)) == 0) {
-	    t = buf + strspn(t, " \t");
-	    if (strncmp(t, "OK", 2))
+	    t = buf + strlen(tag);
+	    t += strspn(t, " \t");
+	    if (strncmp(t, "OK", 2) == 0)
 		break;
 	}
     }
===============================================================================

-- 
Sunil Shetye.