Bug 3274: assertion failed: CommCalls.h:144: "dp"

The  ConnOpener::earlyAbort method shopuld defined as comm close handler
which means that it should have as argument CommCloseCbParams not
CommConnectCbParams

diff --git a/src/comm/ConnOpener.cc b/src/comm/ConnOpener.cc
index e5417ce093e1a4b8be6537af89488bcb4006121d..84f30c3b2a844816b4ec07c1a088d942b30f5121 100644 (file)
--- a/src/comm/ConnOpener.cc
+++ b/src/comm/ConnOpener.cc
@@ -156,7 +156,7 @@ Comm::ConnOpener::start()
         }
     }
 
-    typedef CommCbMemFunT<Comm::ConnOpener, CommConnectCbParams> abortDialer;
+    typedef CommCbMemFunT<Comm::ConnOpener, CommCloseCbParams> abortDialer;
     calls_.earlyAbort_ = JobCallback(5, 4, abortDialer, this, Comm::ConnOpener::earlyAbort);
     comm_add_close_handler(conn_->fd, calls_.earlyAbort_);
 
@@ -279,7 +279,7 @@ Comm::ConnOpener::lookupLocalAddress()
  * Handles the case(s) when a partially setup connection gets closed early.
  */
 void
-Comm::ConnOpener::earlyAbort(const CommConnectCbParams &io)
+Comm::ConnOpener::earlyAbort(const CommCloseCbParams &io)
 {
     debugs(5, 3, HERE << io.conn);
     doneConnecting(COMM_ERR_CLOSING, io.xerrno); // NP: is closing or shutdown better?

diff --git a/src/comm/ConnOpener.h b/src/comm/ConnOpener.h
index 06ace783dc3b25c89523114b7df6addbeb6fb012..0874605b5b1f0a0ce11a8e20f700371f7e17d590 100644 (file)
--- a/src/comm/ConnOpener.h
+++ b/src/comm/ConnOpener.h
@@ -34,7 +34,7 @@ private:
     ConnOpener(const ConnOpener &);
     ConnOpener & operator =(const ConnOpener &c);
 
-    void earlyAbort(const CommConnectCbParams &);
+    void earlyAbort(const CommCloseCbParams &);
     void timeout(const CommTimeoutCbParams &);
     void doneConnecting(comm_err_t status, int xerrno);
     static void InProgressConnectRetry(int fd, void *data);