From e85ed684ea66295339a84522f956811f5116ca2c Mon Sep 17 00:00:00 2001 From: psychon Date: Tue, 28 Oct 2008 17:21:53 +0000 Subject: [PATCH] Some member vars of CWebAdminMod should really be local vars to OnLoad() git-svn-id: https://znc.svn.sourceforge.net/svnroot/znc/trunk@1265 726aef4b-f618-498e-8847-2d620e286838 --- modules/webadmin.cpp | 17 ++++++++--------- 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/modules/webadmin.cpp b/modules/webadmin.cpp index a0de0b7b..8c1fb83a 100644 --- a/modules/webadmin.cpp +++ b/modules/webadmin.cpp @@ -111,7 +111,6 @@ protected: class CWebAdminMod : public CGlobalModule { public: GLOBALMODCONSTRUCTOR(CWebAdminMod) { - m_uPort = 8080; m_sSkinName = GetNV("SkinName"); } @@ -121,9 +120,11 @@ public: virtual bool OnLoad(const CString& sArgStr, CString& sMessage) { bool bSSL = false; bool bIPv6 = false; + unsigned short uPort = 8080; CString sArgs(sArgStr); CString sOpt; CString sPort; + CString sListenHost; if (sArgs.Left(1) == "-") { sOpt = sArgs.Token(0); @@ -140,7 +141,7 @@ public: } if (sArgs.find(" ") != CString::npos) { - m_sListenHost = sArgs.Token(0); + sListenHost = sArgs.Token(0); sPort = sArgs.Token(1); } else { sPort = sArgs.Token(0); @@ -156,7 +157,7 @@ public: } if (!sPort.empty()) { - m_uPort = sPort.ToUShort(); + uPort = sPort.ToUShort(); } CWebAdminSock* pListenSock = new CWebAdminSock(this); @@ -167,11 +168,11 @@ public: #endif errno = 0; - bool b = m_pManager->ListenHost(m_uPort, "WebAdmin::Listener", m_sListenHost, bSSL, SOMAXCONN, pListenSock, 0, bIPv6); + bool b = m_pManager->ListenHost(uPort, "WebAdmin::Listener", sListenHost, bSSL, SOMAXCONN, pListenSock, 0, bIPv6); if (!b) { - sMessage = "Could not bind to port " + CString(m_uPort); - if (!m_sListenHost.empty()) - sMessage += " on vhost [" + m_sListenHost + "]"; + sMessage = "Could not bind to port " + CString(uPort); + if (!sListenHost.empty()) + sMessage += " on vhost [" + sListenHost + "]"; if (errno != 0) sMessage += ": " + CString(strerror(errno)); } @@ -207,9 +208,7 @@ public: } private: - unsigned short m_uPort; CString m_sSkinName; - CString m_sListenHost; map m_suSwitchCounters; };