d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Don't forget to set file descriptor to -1 after closing it, since the code

Browse Source 
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.
This commit is contained in:
Dag-Erling Smørgrav 2000-07-17 21:09:11 +00:00
parent 5cd33c40d7
commit c789727701
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/libfetch/http.c
View File

@ -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;