From owner-cvs-all Mon Jul 17 15:31:25 2000 Delivered-To: cvs-all@freebsd.org Received: from freefall.freebsd.org (freefall.FreeBSD.ORG [204.216.27.21]) by hub.freebsd.org (Postfix) with ESMTP id DE73E37B6AC; Mon, 17 Jul 2000 15:31:18 -0700 (PDT) (envelope-from kris@FreeBSD.org) Received: from localhost (kris@localhost) by freefall.freebsd.org (8.9.3/8.9.2) with ESMTP id PAA70783; Mon, 17 Jul 2000 15:31:17 -0700 (PDT) (envelope-from kris@FreeBSD.org) X-Authentication-Warning: freefall.freebsd.org: kris owned process doing -bs Date: Mon, 17 Jul 2000 15:31:17 -0700 (PDT) From: Kris Kennaway To: Dag-Erling Smorgrav Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/lib/libfetch http.c In-Reply-To: <200007172109.OAA59539@freefall.freebsd.org> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG On Mon, 17 Jul 2000, Dag-Erling Smorgrav wrote: > des 2000/07/17 14:09:12 PDT > > Modified files: > lib/libfetch http.c > Log: > 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. Given the number of bugfixes going into fetch and libfetch, I have to wonder whether the MFC was premature - don't forget the code freeze is in 3 days. Kris -- In God we Trust -- all others must submit an X.509 certificate. -- Charles Forsythe To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message