summaryrefslogtreecommitdiffabout
authorSergey Poznyakoff <gray@gnu.org.ua>2010-09-24 11:46:45 (GMT)
committer Sergey Poznyakoff <gray@gnu.org.ua>2010-09-24 11:47:09 (GMT)
commitdd13dedfe0580e391e2a082ff9d1ac842292a4eb (patch) (side-by-side diff)
tree86b069ee5df16dcefea963819597f557e32499e2
parent81cd471c86dafffd89d517e5b9dcf2be3c16b1e5 (diff)
downloadmailutils-dd13dedfe0580e391e2a082ff9d1ac842292a4eb.tar.gz
mailutils-dd13dedfe0580e391e2a082ff9d1ac842292a4eb.tar.bz2
Fix hex (%XX) expansion in URLs.
* mailbox/url.c (url_parse0): Take three arguments. Return in the third one a boolean indicating whether to expand %XX notations in the URL. Do not decode absolute file names and pipes. (mu_url_parse): Use this value to decide. * movemail/movemail.c (main): In emacs mode: force UNIX mbox format as a default. Send debugging output to stderr.
Diffstat (more/less context) (show whitespace changes)
-rw-r--r--mailbox/url.c13
-rw-r--r--movemail/movemail.c8
2 files changed, 17 insertions, 4 deletions
diff --git a/mailbox/url.c b/mailbox/url.c
index 1b833bf..f46fbc1 100644
--- a/mailbox/url.c
+++ b/mailbox/url.c
@@ -40,7 +40,7 @@
#define AC2(a,b) a ## b
#define AC4(a,b,c,d) a ## b ## c ## d
-static int url_parse0 (mu_url_t, char *, size_t *poff);
+static int url_parse0 (mu_url_t, char *, size_t *poff, int *decode);
static int
parse_query (const char *query,
@@ -286,6 +286,7 @@ mu_url_parse (mu_url_t url)
struct _mu_url u;
size_t pstart;
mu_secret_t newsec;
+ int want_decode;
if (!url || !url->name)
return EINVAL;
@@ -301,7 +302,7 @@ mu_url_parse (mu_url_t url)
if (!n)
return ENOMEM;
- err = url_parse0 (&u, n, &pstart);
+ err = url_parse0 (&u, n, &pstart, &want_decode);
if (!err)
{
@@ -332,7 +333,8 @@ mu_url_parse (mu_url_t url)
*/
#define UALLOC(X) \
- if (u.X && u.X[0] && (url->X = mu_url_decode(u.X)) == 0) \
+ if (u.X && u.X[0] && \
+ !(url->X = (want_decode ? mu_url_decode (u.X) : strdup (u.X)))) \
{ \
err = ENOMEM; \
goto CLEANUP; \
@@ -420,7 +422,7 @@ Is this required to be % quoted, though? I hope so!
*/
static int
-url_parse0 (mu_url_t u, char *name, size_t *poff)
+url_parse0 (mu_url_t u, char *name, size_t *poff, int *decode)
{
char *start = name;
char *p; /* pointer into name */
@@ -432,11 +434,13 @@ url_parse0 (mu_url_t u, char *name, size_t *poff)
if (name[0] == '/')
{
u->scheme = "file";
+ *decode = 0;
}
else if (name[0] == '|')
{
int rc;
u->scheme = "prog";
+ *decode = 0;
rc = mu_argcv_get (name + 1, NULL, NULL, &u->qargc, &u->qargv);
if (rc == 0)
{
@@ -448,6 +452,7 @@ url_parse0 (mu_url_t u, char *name, size_t *poff)
}
else
{
+ *decode = 1;
/* Parse out the SCHEME. */
p = strchr (name, ':');
if (p == NULL)
diff --git a/movemail/movemail.c b/movemail/movemail.c
index afb83e8..4e7bdb9 100644
--- a/movemail/movemail.c
+++ b/movemail/movemail.c
@@ -760,6 +760,14 @@ main (int argc, char **argv)
return 1;
}
+ if (emacs_mode)
+ {
+ /* Undo the effect of configuration options that may affect
+ the interaction with Emacs. */
+ mu_registrar_set_default_record (mu_mbox_record);
+ mu_debug_default_printer = mu_debug_stderr_printer;
+ }
+
atexit (close_mailboxes);
source_name = argv[0];

Return to:

Send suggestions and report system problems to the System administrator.