You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Ben Laurie <be...@gonzo.ben.algroup.co.uk> on 1995/10/14 12:01:39 UTC

Whoops!

Just noticed a bug in the 07_SocketMemLeak patch... non-fatal, I guess this
should go in the 0.8.15 patchlist (if someone could put it there, please)

From: ben@algroup.co.uk (Ben Laurie)
Subject: Use pfclose() on timeout
Affects: http_main.c
Changelog: Use pfclose() instead of fclose() on a timeout.

*** ../../apache_0.8.14/src/http_main.c     Tue Sep 19 17:05:01 1995
--- http_main.c Sat Oct 14 11:54:41 1995
***************
*** 255,262 ****

        log_transaction(timeout_req);

!       fclose (timeout_req->connection->client);
!       fclose (timeout_req->connection->request_in);

        if (!standalone) exit(0);
  #ifdef NEXT
--- 255,262 ----

        log_transaction(timeout_req);

!       pfclose (timeout_req->connection->pool,timeout_req->connection->client);

!       pfclose (timeout_req->connection->pool,timeout_req->connection->request_in);

        if (!standalone) exit(0);
  #ifdef NEXT

Cheers,

Ben.

-- 
Ben Laurie                  Phone: +44 (181) 994 6435
Freelance Consultant        Fax:   +44 (181) 994 6472
and Technical Director      Email: ben@algroup.co.uk
A.L. Digital Ltd,
London, England.