diff options
author | des <des@FreeBSD.org> | 2000-07-17 21:09:11 +0000 |
---|---|---|
committer | des <des@FreeBSD.org> | 2000-07-17 21:09:11 +0000 |
commit | 9e42d615d8102289f59981b6332a387ed4c92781 (patch) | |
tree | b7a07d8f231faccd144176986e527c8f5a092e5a /lib/libfetch | |
parent | 147a437a42224aa1f7bc1915d14ec6297becdbf0 (diff) | |
download | FreeBSD-src-9e42d615d8102289f59981b6332a387ed4c92781.zip FreeBSD-src-9e42d615d8102289f59981b6332a387ed4c92781.tar.gz |
Don't forget to set file descriptor to -1 after closing it, since the code
outside the loop inspects it to determine whether or not we succeeded in
retrieving the requested document. This fixes a bug where fetchGetHTTP()
would return a FILE with an invalid file descriptor if it hit the redirect
limit without locating the requested document.
Diffstat (limited to 'lib/libfetch')
-rw-r--r-- | lib/libfetch/http.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/libfetch/http.c b/lib/libfetch/http.c index c9b2802..a384e04 100644 --- a/lib/libfetch/http.c +++ b/lib/libfetch/http.c @@ -777,7 +777,6 @@ _http_request(struct url *URL, char *op, struct url_stat *us, char *flags) chunked = 0; need_auth = 0; offset = 0; - fd = -1; retry: /* connect to server or proxy */ if ((fd = _http_connect(url, &proxy, flags)) == -1) @@ -935,6 +934,7 @@ _http_request(struct url *URL, char *op, struct url_stat *us, char *flags) /* we have a redirect */ close(fd); + fd = -1; if (url != URL) fetchFreeURL(url); url = new; |