diff options
author | des <des@FreeBSD.org> | 2000-07-13 09:13:58 +0000 |
---|---|---|
committer | des <des@FreeBSD.org> | 2000-07-13 09:13:58 +0000 |
commit | 797b7f6ddd1a3cf9f0003f851c886513d0e6e08e (patch) | |
tree | 08c1f4385b1b9740c10992f8b586588b386540be | |
parent | 9ced9463554050d5ab35e1fb0fdc6ff38ca2daa5 (diff) | |
download | FreeBSD-src-797b7f6ddd1a3cf9f0003f851c886513d0e6e08e.zip FreeBSD-src-797b7f6ddd1a3cf9f0003f851c886513d0e6e08e.tar.gz |
Fix a bug (misplaced continue) that caused redirects to fail. Lots of code
moved around, but the acutal functional changes are small.
Add support for site-internal redirects (where the Location: header gives a
path instead of an absolute URI)
Pointed out by: kuriyama
-rw-r--r-- | lib/libfetch/http.c | 45 |
1 files changed, 29 insertions, 16 deletions
diff --git a/lib/libfetch/http.c b/lib/libfetch/http.c index dfe1314..30647ae 100644 --- a/lib/libfetch/http.c +++ b/lib/libfetch/http.c @@ -732,15 +732,20 @@ _http_request(struct url *URL, char *op, struct url_stat *us, char *flags) n = noredirect ? 1 : MAX_REDIRECT; - us->size = -1; - us->atime = us->mtime = 0; + /* just to appease compiler warnings */ + code = HTTP_PROTOCOL_ERROR; chunked = 0; offset = 0; fd = -1; - code = HTTP_PROTOCOL_ERROR; /* just to appease a compiler warning */ for (url = URL, i = 0; i < n; ++i) { + new = NULL; + us->size = -1; + us->atime = us->mtime = 0; + chunked = 0; need_auth = 0; + offset = 0; + fd = -1; retry: /* connect to server or proxy */ if ((fd = _http_connect(url, &proxy, flags)) == -1) @@ -858,27 +863,28 @@ _http_request(struct url *URL, char *op, struct url_stat *us, char *flags) case hdr_location: if (!HTTP_REDIRECT(code)) break; + if (new) + free(new); if (verbose) _fetch_info("%d redirect to %s", code, p); - if ((new = fetchParseURL(p)) == NULL) - /* invalid location */ + if (*p == '/') + /* absolute path */ + new = fetchMakeURL(url->scheme, url->host, url->port, p, + url->user, url->pwd); + else + new = fetchParseURL(p); + if (new == NULL) { + /* XXX should set an error code */ + DEBUG(fprintf(stderr, "failed to parse new URL\n")); goto ouch; + } if (!*new->user && !*new->pwd) { strcpy(new->user, url->user); strcpy(new->pwd, url->pwd); } new->offset = url->offset; new->length = url->length; - close(fd); - us->size = -1; - us->atime = us->mtime = 0; - chunked = 0; - offset = 0; - fd = -1; - if (url != URL) - fetchFreeURL(url); - url = new; - continue; + break; case hdr_transfer_encoding: /* XXX weak test*/ chunked = (strcasecmp(p, "chunked") == 0); @@ -891,8 +897,15 @@ _http_request(struct url *URL, char *op, struct url_stat *us, char *flags) } } while (h > hdr_end); - if (code == HTTP_OK || code == HTTP_PARTIAL) + /* we either have a hit, or a redirect with no Location: header */ + if (code == HTTP_OK || code == HTTP_PARTIAL || !new) break; + + /* we have a redirect */ + close(fd); + if (url != URL) + fetchFreeURL(url); + url = new; } /* no success */ |