From 519a0c0fab0487468d83c2ff67f4ab98bb479b86 Mon Sep 17 00:00:00 2001
From: Amos Jeffries <squid3@treenet.co.nz>
Date: Thu, 22 Dec 2011 00:30:04 -0700
Subject: [PATCH] Bug 3447: assertion failed: CommCalls.h:150: "dp"

Detatch FD/connection close handlers from reliance on commCloseCbParams
some handlers use FdeCbParams and others are possible.

If the callback handler needs FD the call creator must save it in the
params at call construction time.
---
 src/comm.cc | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/comm.cc b/src/comm.cc
index 069962b65f..7a364f9a49 100644
--- a/src/comm.cc
+++ b/src/comm.cc
@@ -936,10 +936,6 @@ commCallCloseHandlers(int fd)
         // If call is not canceled schedule it for execution else ignore it
         if (!call->canceled()) {
             debugs(5, 5, "commCallCloseHandlers: ch->handler=" << call);
-            // XXX: this should not be needed. Params can be set by the call creator
-            typedef CommCloseCbParams Params;
-            Params &params = GetCommParams<Params>(call);
-            params.fd = fd;
             ScheduleCallHere(call);
         }
     }
-- 
2.47.2