Hi Alfred,

>   pop://<USER>:<PASSWORD>;auth=+APOP@<HOST>
> 
> is interpreted as having the user <USER>:<PASSWORD>.

Thanks for noticing.  Please, try the attached patch.

Regards,
Sergey

Index: mailbox/url.c
===================================================================
RCS file: /cvsroot/mailutils/mailutils/mailbox/url.c,v
retrieving revision 1.32
diff -p -u -r1.32 url.c
--- mailbox/url.c	16 Aug 2008 08:21:32 -0000	1.32
+++ mailbox/url.c	14 Oct 2008 13:42:37 -0000
@@ -260,12 +260,17 @@ url_parse0 (mu_url_t u, char *name)
 	      /* Try to split the user into a:
 		 <user>:<password>
 		 or
-		 <user>;AUTH=<auth>
+		 <user>:<password>;AUTH=<auth>
 	      */
 
 	      for (; *name; name++)
 		{
-		  if (*name == ';')
+		  if (*name == ':')
+		    {
+		      *name++ = 0;
+		      u->passwd = name;
+		    }
+		  else if (*name == ';')
 		    {
 		      /* Make sure it's the auth token. */
 		      if (strncasecmp (name + 1, "auth=", 5) == 0)
@@ -276,12 +281,6 @@ url_parse0 (mu_url_t u, char *name)
 			  break;
 			}
 		    }
-		  if (*name == ':')
-		    {
-		      *name++ = 0;
-		      u->passwd = name;
-		      break;
-		    }
 		}
 	    }
 
_______________________________________________
Bug-mailutils mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/bug-mailutils

Reply via email to