From 9e42d615d8102289f59981b6332a387ed4c92781 Mon Sep 17 00:00:00 2001 From: des Date: Mon, 17 Jul 2000 21:09:11 +0000 Subject: 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. --- lib/libfetch/http.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lib/libfetch') 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; -- cgit v1.1