]> Pileus Git - ~andy/fetchmail/blobdiff - uid.c
Bug fixes and internationalization improvements.
[~andy/fetchmail] / uid.c
diff --git a/uid.c b/uid.c
index ac49a013b9fa3f2fff86febfec4eebebb50ad706..8163761b1dbc010907f563000e940c3955543c6c 100644 (file)
--- a/uid.c
+++ b/uid.c
@@ -6,6 +6,8 @@
 
 #include "config.h"
 
+#include <sys/stat.h>
+#include <errno.h>
 #include <stdio.h>
 #include <limits.h>
 #if defined(STDC_HEADERS)
@@ -17,6 +19,7 @@
 #endif
 
 #include "fetchmail.h"
+#include "i18n.h"
 
 /*
  * Machinery for handling UID lists live here.  This is mainly to support
@@ -33,7 +36,7 @@
  * considered seen in `oldsaved'.  These are messages that were left in
  * the mailbox and *not deleted* on previous queries (we don't need to
  * remember the UIDs of deleted messages because ... well, they're gone!)
- * This list is initially set up by initialized_saved_list() from the
+ * This list is initially set up by initialize_saved_list() from the
  * .fetchids file.
  *
  * Early in the query, during the execution of the protocol-specific 
@@ -60,7 +63,7 @@
  * be picked up by the next run.  If there are no un-expunged
  * messages, the file is deleted.
  *
- * Note: all comparisons are caseblind!
+ * Note: some comparisons (those used for DNS address lists) are caseblind!  
  */
 
 /* UIDs associated with un-queried hosts */
@@ -70,40 +73,147 @@ static struct idlist *scratchlist;
 void initialize_saved_lists(struct query *hostlist, const char *idfile)
 /* read file of saved IDs and attach to each host */
 {
-    int        st;
+    struct stat statbuf;
     FILE       *tmpfp;
     struct query *ctl;
 
     /* make sure lists are initially empty */
     for (ctl = hostlist; ctl; ctl = ctl->next)
-       ctl->oldsaved = ctl->newsaved = (struct idlist *)NULL;
+       ctl->skipped = ctl->oldsaved = ctl->newsaved = (struct idlist *)NULL;
+
+    errno = 0;
+
+    /*
+     * Croak if the uidl directory does not exist.
+     * This probably means an NFS mount failed and we can't
+     * see a uidl file that ought to be there.
+     * Question: is this a portable check? It's not clear
+     * that all implementations of lstat() will return ENOTDIR
+     * rather than plain ENOENT in this case...
+     */
+   if (lstat(idfile, &statbuf) < 0) {
+     if (errno == ENOTDIR) 
+    {
+      report(stderr, _("lstat: %s: %s\n"), idfile, strerror(errno));
+      exit(PS_IOERR);
+    }
+   }
 
     /* let's get stored message UIDs from previous queries */
-    if ((tmpfp = fopen(idfile, "r")) != (FILE *)NULL) {
-       char buf[POPBUFSIZE+1],host[HOSTLEN+1],user[USERNAMELEN+1],id[IDLEN+1];
+    if ((tmpfp = fopen(idfile, "r")) != (FILE *)NULL)
+    {
+       char buf[POPBUFSIZE+1];
+       char *host = NULL;      /* pacify -Wall */
+       char *user;
+       char *id;
+       char *atsign;   /* temp pointer used in parsing user and host */
+       char *delimp1;
+       char saveddelim1;
+       char *delimp2;
+       char saveddelim2 = '\0';        /* pacify -Wall */
 
        while (fgets(buf, POPBUFSIZE, tmpfp) != (char *)NULL)
        {
-           /* possible lossage here with very old versions of sscanf(3)... */
-           if ((st = sscanf(buf, "%[^@]@%s %s\n", user, host, id)) == 3)
+           /*
+            * At this point, we assume the bug has two fields -- a user@host 
+            * part, and an ID part. Either field may contain spurious @ signs.
+            * The previous version of this code presumed one could split at 
+            * the rightmost '@'.  This is not correct, as InterMail puts an 
+            * '@' in the UIDL.
+            */
+         
+           /* first, skip leading spaces */
+           user = buf + strspn(buf, " \t");
+
+           /*
+            * First, we split the buf into a userhost part and an id
+            * part ... but id doesn't necessarily start with a '<',
+            * espescially if the POP server returns an X-UIDL header
+            * instead of a Message-ID, as GMX's (www.gmx.net) POP3
+            * StreamProxy V1.0 does.
+            */
+           if ((id = strchr(user, ' ')) != NULL )
            {
-               for (ctl = hostlist; ctl; ctl = ctl->next)
-               {
-                   if (ctl->server.truename &&
-                       strcasecmp(host, ctl->server.truename) == 0
-                               && strcasecmp(user, ctl->remotename) == 0)
-                   {
-                       save_str(&ctl->oldsaved, id, UID_UNSEEN);
+               for (delimp1 = id; delimp1 >= user; delimp1--)
+                   if ((*delimp1 != ' ') && (*delimp1 != '\t'))
                        break;
+
+               /* 
+                * It should be safe to assume that id starts after
+                * the " " - after all, we're writing the " "
+                * ourselves in write_saved_lists() :-)
+                */
+               id = id + strspn(id, " ");
+
+               delimp1++; /* but what if there is only white space ?!? */
+               saveddelim1 = *delimp1; /* save char after token */
+               *delimp1 = '\0';                /* delimit token with \0 */
+               if (id != NULL) 
+               {
+                   /* now remove trailing white space chars from id */
+                   if ((delimp2 = strpbrk(id, " \t\n")) != NULL ) {
+                       saveddelim2 = *delimp2;
+                       *delimp2 = '\0';
+                   }
+                   atsign = strrchr(user, '@');
+                   if (atsign) {
+                       *atsign = '\0';
+                       host = atsign + 1;
+
+                   }
+                   for (ctl = hostlist; ctl; ctl = ctl->next) {
+                       if (ctl->server.truename &&
+                           strcasecmp(host, ctl->server.truename) == 0
+                           && strcasecmp(user, ctl->remotename) == 0) {
+       
+                           save_str(&ctl->oldsaved, id, UID_SEEN);
+                           break;
+                       }
+                   }
+                   /* if it's not in a host we're querying,
+                   ** save it anyway */
+                   if (ctl == (struct query *)NULL) {
+                               /* restore string */
+                       *delimp1 = saveddelim1;
+                       *atsign = '@';
+                       if (delimp2 != NULL) {
+                           *delimp2 = saveddelim2;
+                       }
+                       save_str(&scratchlist, buf, UID_SEEN);
                    }
                }
+           }
+       }
+       fclose(tmpfp);  /* not checking should be safe, mode was "r" */
+    }
+
+    if (outlevel >= O_DEBUG)
+    {
+       struct idlist   *idp;
+       int uidlcount = 0;
 
-               /* if it's not in a host we're querying, save it anyway */
-               if (ctl == (struct query *)NULL)
-                   save_str(&scratchlist, buf, UID_SEEN);
+       for (ctl = hostlist; ctl; ctl = ctl->next)
+           if (ctl->server.uidl)
+           {
+               report_build(stdout, _("Old UID list from %s:"), 
+                            ctl->server.pollname);
+               for (idp = ctl->oldsaved; idp; idp = idp->next)
+                   report_build(stdout, " %s", idp->id);
+               if (!idp)
+                   report_build(stdout, _(" <empty>"));
+               report_complete(stdout, "\n");
+               uidlcount++;
            }
+
+       if (uidlcount)
+       {
+           report_build(stdout, _("Scratch list of UIDs:"));
+           for (idp = scratchlist; idp; idp = idp->next)
+               report_build(stdout, " %s", idp->id);
+           if (!idp)
+               report_build(stdout, _(" <empty>"));
+           report_complete(stdout, "\n");
        }
-       fclose(tmpfp);
     }
 }
 #endif /* POP3_ENABLE */
@@ -170,15 +280,17 @@ void free_str_pair_list(struct idlist **idl)
 }
 #endif
 
-int str_in_list(struct idlist **idl, const char *str)
-/* is a given ID in the given list? (comparison is caseblind) */
+int str_in_list(struct idlist **idl, const char *str, const flag caseblind)
+/* is a given ID in the given list? (comparison may be caseblind) */
 {
     if (*idl == (struct idlist *)NULL || str == (char *) NULL)
        return(0);
-    else if (strcasecmp(str, (*idl)->id) == 0)
+    else if (!caseblind && strcmp(str, (*idl)->id) == 0)
+       return(1);
+    else if (caseblind && strcasecmp(str, (*idl)->id) == 0)
        return(1);
     else
-       return(str_in_list(&(*idl)->next, str));
+       return(str_in_list(&(*idl)->next, str, caseblind));
 }
 
 int str_nr_in_list( struct idlist **idl, const char *str )
@@ -189,7 +301,7 @@ int str_nr_in_list( struct idlist **idl, const char *str )
     if ( !str )
         return -1;
     for( walk = *idl, nr = 0; walk; nr ++, walk = walk->next )
-        if( strcasecmp( str, walk->id) == 0 )
+        if( strcmp( str, walk->id) == 0 )
            return nr;
     return -1;
 }
@@ -202,7 +314,7 @@ int str_nr_last_in_list( struct idlist **idl, const char *str)
     if ( !str )
         return -1;
     for( walk = *idl, nr = 0; walk; nr ++, walk = walk->next )
-        if( strcasecmp( str, walk->id) == 0 )
+        if( strcmp( str, walk->id) == 0 )
            ret = nr;
     return ret;
 }
@@ -215,7 +327,7 @@ void str_set_mark( struct idlist **idl, const char *str, const flag val)
     if (!str)
         return;
     for(walk = *idl, nr = 0; walk; nr ++, walk = walk->next)
-        if (strcasecmp(str, walk->id) == 0)
+        if (strcmp(str, walk->id) == 0)
            walk->val.status.mark = val;
 }
 
@@ -274,10 +386,28 @@ int delete_str(struct idlist **idl, int num)
     return(0);
 }
 
+struct idlist *copy_str_list(struct idlist *idl)
+/* copy the given UID list */
+{
+    struct idlist *newnode ;
+
+    if (idl == (struct idlist *)NULL)
+       return(NULL);
+    else
+    {
+       newnode = (struct idlist *)xmalloc(sizeof(struct idlist));
+       memcpy(newnode, idl, sizeof(struct idlist));
+       newnode->next = copy_str_list(idl->next);
+       return(newnode);
+    }
+}
+
 void append_str_list(struct idlist **idl, struct idlist **nidl)
 /* append nidl to idl (does not copy *) */
 {
-    if ((*idl) == (struct idlist *)NULL)
+    if ((*nidl) == (struct idlist *)NULL || *nidl == *idl)
+       return;
+    else if ((*idl) == (struct idlist *)NULL)
        *idl = *nidl;
     else if ((*idl)->next == (struct idlist *)NULL)
        (*idl)->next = *nidl;
@@ -296,12 +426,39 @@ void expunge_uids(struct query *ctl)
            idl->val.status.mark = UID_EXPUNGED;
 }
 
-void update_str_lists(struct query *ctl)
-/* perform end-of-query actions on UID lists */
+void uid_swap_lists(struct query *ctl) 
+/* finish a query */
 {
-    free_str_list(&ctl->oldsaved);
-    ctl->oldsaved = ctl->newsaved;
-    ctl->newsaved = (struct idlist *) NULL;
+    /* debugging code */
+    if (ctl->server.uidl && outlevel >= O_DEBUG)
+    {
+       struct idlist *idp;
+
+       report_build(stdout, _("New UID list from %s:"), ctl->server.pollname);
+       for (idp = ctl->newsaved; idp; idp = idp->next)
+           report_build(stdout, " %s = %d", idp->id, idp->val.status.mark);
+       if (!idp)
+           report_build(stdout, _(" <empty>"));
+       report_complete(stdout, "\n");
+    }
+
+    /*
+     * Don't swap UID lists unless we've actually seen UIDLs.
+     * This is necessary in order to keep UIDL information
+     * from being heedlessly deleted later on.
+     */
+    if (ctl->newsaved)
+    {
+       /* old state of mailbox may now be irrelevant */
+       if (outlevel >= O_DEBUG)
+           report(stdout, _("swapping UID lists\n"));
+       free_str_list(&ctl->oldsaved);
+       free_str_list(&scratchlist);
+       ctl->oldsaved = ctl->newsaved;
+       ctl->newsaved = (struct idlist *) NULL;
+    }
+    else if (outlevel >= O_DEBUG)
+       report(stdout, _("not swapping UID lists, no UIDs seen this query\n"));
 }
 
 void write_saved_lists(struct query *hostlist, const char *idfile)
@@ -315,14 +472,23 @@ void write_saved_lists(struct query *hostlist, const char *idfile)
     /* if all lists are empty, nuke the file */
     idcount = 0;
     for (ctl = hostlist; ctl; ctl = ctl->next) {
-       if (ctl->oldsaved)
-           idcount++;
+        for (idp = ctl->oldsaved; idp; idp = idp->next)
+            if (idp->val.status.mark == UID_SEEN
+                               || idp->val.status.mark == UID_DELETED)
+                idcount++;
     }
 
     /* either nuke the file or write updated last-seen IDs */
-    if (!idcount)
+    if (!idcount && !scratchlist)
+    {
+       if (outlevel >= O_DEBUG)
+           report(stdout, _("Deleting fetchids file.\n"));
        unlink(idfile);
+    }
     else
+    {
+       if (outlevel >= O_DEBUG)
+           report(stdout, _("Writing fetchids file.\n"));
        if ((tmpfp = fopen(idfile, "w")) != (FILE *)NULL) {
            for (ctl = hostlist; ctl; ctl = ctl->next) {
                for (idp = ctl->oldsaved; idp; idp = idp->next)
@@ -335,6 +501,7 @@ void write_saved_lists(struct query *hostlist, const char *idfile)
                fputs(idp->id, tmpfp);
            fclose(tmpfp);
        }
+    }
 }
 #endif /* POP3_ENABLE */