diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2016-10-12 23:20:35 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2016-10-13 05:55:12 +0000 |
commit | 14a7ac2033273fdddfb9748d5fa1e1c0f25b64ca (patch) | |
tree | 01b32f0d6ef1cecd2c1c4e3fc4f7aee8d9b01164 | |
parent | a9afa89e953f0f32acf26b143717e7d067cbc75a (diff) |
SOCKET_USE_AUTODIAL is dead code
...ever since 010f71130b0d55079d01fbda38d45901de98b133 "INTEGRATION: CWS sal01:
#105958# Remove usage of wininet.dll autodial functions" wrapped its central
code in
#ifdef SOCKET_USE_AUTODIAL
instead of removing it properly.
Change-Id: I4c077c71ce8eda3a3f8b83286178d0d7907a6e02
Reviewed-on: https://gerrit.libreoffice.org/29742
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
-rw-r--r-- | sal/osl/w32/socket.cxx | 255 | ||||
-rw-r--r-- | sal/osl/w32/sockimpl.h | 31 |
2 files changed, 55 insertions, 231 deletions
diff --git a/sal/osl/w32/socket.cxx b/sal/osl/w32/socket.cxx index c927f4231a08..1b2bc6d1eac9 100644 --- a/sal/osl/w32/socket.cxx +++ b/sal/osl/w32/socket.cxx @@ -262,142 +262,14 @@ static oslSocketError osl_SocketErrorFromNative(int nativeType) #define ERROR_FROM_NATIVE(y) osl_SocketErrorFromNative(y) /*****************************************************************************/ -/* oslSocketDialupImpl */ -/*****************************************************************************/ -static oslSocketDialupImpl *pDialupImpl = nullptr; - -static oslSocketDialupImpl* osl_createSocketDialupImpl() -{ - oslSocketDialupImpl *pImpl; - pImpl = static_cast<oslSocketDialupImpl*>(rtl_allocateZeroMemory( sizeof (oslSocketDialupImpl))); - - InitializeCriticalSection (&pImpl->m_hMutex); - - return (pImpl); -} - -static void osl_initSocketDialupImpl (oslSocketDialupImpl *pImpl) -{ -#ifdef SOCKET_USE_AUTODIAL - if (pImpl) - { - HINSTANCE hModule; - - EnterCriticalSection (&pImpl->m_hMutex); - - hModule = LoadLibrary (INTERNET_MODULE_NAME); - if (!(hModule <= (HINSTANCE)HINSTANCE_ERROR)) - { - pImpl->m_pfnAttemptConnect = (INTERNETATTEMPTCONNECT) - (GetProcAddress (hModule, "InternetAttemptConnect")); - pImpl->m_pfnAutodial = (INTERNETAUTODIAL) - (GetProcAddress (hModule, "InternetAutodial")); - pImpl->m_pfnAutodialHangup = (INTERNETAUTODIALHANGUP) - (GetProcAddress (hModule, "InternetAutodialHangup")); - pImpl->m_pfnGetConnectedState = (INTERNETGETCONNECTEDSTATE) - (GetProcAddress (hModule, "InternetGetConnectedState")); - pImpl->m_hModule = hModule; - } - - LeaveCriticalSection (&pImpl->m_hMutex); - } -#else - (void)pImpl; -#endif -} - -static void osl_destroySocketDialupImpl (oslSocketDialupImpl *pImpl) -{ - if (pImpl) - { - EnterCriticalSection (&pImpl->m_hMutex); - - if (pImpl->m_dwFlags & INTERNET_CONNECTION_HANGUP) - { - if (pImpl->m_pfnAutodialHangup) - { - (pImpl->m_pfnAutodialHangup)(0); - pImpl->m_dwFlags &= ~INTERNET_CONNECTION_HANGUP; - } - } - - if (pImpl->m_hModule) - FreeLibrary (pImpl->m_hModule); - - LeaveCriticalSection (&pImpl->m_hMutex); - DeleteCriticalSection (&pImpl->m_hMutex); - - rtl_freeMemory (pImpl); - } -} - -static bool osl_querySocketDialupImpl() -{ - bool result; - - if (pDialupImpl == nullptr) - { - pDialupImpl = osl_createSocketDialupImpl(); - osl_initSocketDialupImpl (pDialupImpl); - } - - EnterCriticalSection (&pDialupImpl->m_hMutex); - - result = true; - if (pDialupImpl->m_pfnGetConnectedState) - { - DWORD dwFlags = 0; - - result = (pDialupImpl->m_pfnGetConnectedState)(&dwFlags, 0); - pDialupImpl->m_dwFlags |= dwFlags; - } - - LeaveCriticalSection (&pDialupImpl->m_hMutex); - return result; -} - -static bool osl_attemptSocketDialupImpl() -{ - bool result; - - if (pDialupImpl == nullptr) - { - pDialupImpl = osl_createSocketDialupImpl(); - osl_initSocketDialupImpl (pDialupImpl); - } - - EnterCriticalSection (&pDialupImpl->m_hMutex); - - result = osl_querySocketDialupImpl(); - if (!result) - { - result = true; - if (pDialupImpl->m_pfnAutodial) - { - result = (pDialupImpl->m_pfnAutodial)(0, 0); - if (result) - pDialupImpl->m_dwFlags |= INTERNET_CONNECTION_HANGUP; - else - WSASetLastError (WSAENETDOWN); - } - } - - LeaveCriticalSection (&pDialupImpl->m_hMutex); - return result; -} - -/*****************************************************************************/ /* oslSocketImpl */ /*****************************************************************************/ -static sal_uInt32 g_nSocketImpl = 0; - #if OSL_DEBUG_LEVEL > 0 static sal_uInt32 g_nSocketAddr = 0; struct LeakWarning { ~LeakWarning() { - SAL_WARN_IF( g_nSocketImpl, "sal.osl", "sal_socket: " << g_nSocketImpl << " socket instances leak" ); SAL_WARN_IF( g_nSocketAddr, "sal.osl", "sal_socket: " << g_nSocketAddr << " socket address instances leak" ); } }; @@ -409,9 +281,6 @@ oslSocket osl_createSocketImpl_(SOCKET Socket) oslSocket pSockImpl = static_cast<oslSocket>(rtl_allocateZeroMemory( sizeof(struct oslSocketImpl))); pSockImpl->m_Socket = Socket; pSockImpl->m_nRefCount = 1; - - g_nSocketImpl++; - return (pSockImpl); } @@ -419,11 +288,6 @@ void osl_destroySocketImpl_(oslSocketImpl *pImpl) { if (pImpl) { - if (--g_nSocketImpl == 0) - { - osl_destroySocketDialupImpl (pDialupImpl); - pDialupImpl = nullptr; - } rtl_freeMemory (pImpl); } } @@ -797,49 +661,46 @@ oslHostAddr SAL_CALL osl_createHostAddrByName(rtl_uString *strHostname) if ((strHostname == nullptr) || (strHostname->length == 0)) return nullptr; - if (osl_attemptSocketDialupImpl()) - { #if _WIN32_WINNT < _WIN32_WINNT_VISTA - struct hostent *he; - rtl_String *Hostname= NULL; + struct hostent *he; + rtl_String *Hostname= NULL; - rtl_uString2String( - &Hostname, strHostname->buffer, strHostname->length, - RTL_TEXTENCODING_UTF8, OUSTRING_TO_OSTRING_CVTFLAGS); + rtl_uString2String( + &Hostname, strHostname->buffer, strHostname->length, + RTL_TEXTENCODING_UTF8, OUSTRING_TO_OSTRING_CVTFLAGS); - he= gethostbyname (Hostname->buffer); + he= gethostbyname (Hostname->buffer); - rtl_string_release (Hostname); - return __osl_hostentToHostAddr (he); + rtl_string_release (Hostname); + return __osl_hostentToHostAddr (he); #else - PADDRINFOW pAddrInfo = nullptr; - int ret = GetAddrInfoW( - strHostname->buffer, nullptr, nullptr, & pAddrInfo); - if (0 == ret) + PADDRINFOW pAddrInfo = nullptr; + int ret = GetAddrInfoW( + strHostname->buffer, nullptr, nullptr, & pAddrInfo); + if (0 == ret) + { + oslHostAddr pRet = nullptr; + for (PADDRINFOW pIter = pAddrInfo; pIter; pIter = pIter->ai_next) { - oslHostAddr pRet = nullptr; - for (PADDRINFOW pIter = pAddrInfo; pIter; pIter = pIter->ai_next) + if (AF_INET == pIter->ai_family) { - if (AF_INET == pIter->ai_family) - { - pRet = static_cast<oslHostAddr>( - rtl_allocateZeroMemory(sizeof(struct oslHostAddrImpl))); - rtl_uString_newFromStr(&pRet->pHostName, pIter->ai_canonname); - pRet->pSockAddr = osl_createSocketAddr_(); - memcpy(& pRet->pSockAddr->m_sockaddr, - pIter->ai_addr, pIter->ai_addrlen); - break; // ignore other results - } + pRet = static_cast<oslHostAddr>( + rtl_allocateZeroMemory(sizeof(struct oslHostAddrImpl))); + rtl_uString_newFromStr(&pRet->pHostName, pIter->ai_canonname); + pRet->pSockAddr = osl_createSocketAddr_(); + memcpy(& pRet->pSockAddr->m_sockaddr, + pIter->ai_addr, pIter->ai_addrlen); + break; // ignore other results } - FreeAddrInfoW(pAddrInfo); - return pRet; - } - else - { - SAL_INFO("sal.osl", "GetAddrInfoW failed: " << WSAGetLastError()); } -#endif // _WIN32_WINNT + FreeAddrInfoW(pAddrInfo); + return pRet; } + else + { + SAL_INFO("sal.osl", "GetAddrInfoW failed: " << WSAGetLastError()); + } +#endif // _WIN32_WINNT return nullptr; } @@ -858,37 +719,34 @@ oslHostAddr SAL_CALL osl_createHostAddrByAddr(const oslSocketAddr pAddr) if (sin->sin_addr.s_addr == htonl(INADDR_ANY)) return nullptr; - if (osl_attemptSocketDialupImpl()) - { #if _WIN32_WINNT < _WIN32_WINNT_VISTA - struct hostent *he; - he= gethostbyaddr ((const sal_Char *)&(sin->sin_addr), - sizeof (sin->sin_addr), - sin->sin_family); - return __osl_hostentToHostAddr (he); + struct hostent *he; + he= gethostbyaddr ((const sal_Char *)&(sin->sin_addr), + sizeof (sin->sin_addr), + sin->sin_family); + return __osl_hostentToHostAddr (he); #else - WCHAR buf[NI_MAXHOST]; - int ret = GetNameInfoW( - & pAddr->m_sockaddr, sizeof(struct sockaddr), - buf, NI_MAXHOST, - nullptr, 0, 0); - if (0 == ret) - { - oslHostAddr pRet = static_cast<oslHostAddr>( - rtl_allocateZeroMemory(sizeof(struct oslHostAddrImpl))); - rtl_uString_newFromStr(&pRet->pHostName, - reinterpret_cast<sal_Unicode*>(&buf)); - pRet->pSockAddr = osl_createSocketAddr_(); - memcpy(& pRet->pSockAddr->m_sockaddr, - & pAddr->m_sockaddr, sizeof(struct sockaddr)); - return pRet; - } - else - { - SAL_INFO("sal.osl", "GetNameInfoW failed: " << WSAGetLastError()); - } -#endif // _WIN32_WINNT + WCHAR buf[NI_MAXHOST]; + int ret = GetNameInfoW( + & pAddr->m_sockaddr, sizeof(struct sockaddr), + buf, NI_MAXHOST, + nullptr, 0, 0); + if (0 == ret) + { + oslHostAddr pRet = static_cast<oslHostAddr>( + rtl_allocateZeroMemory(sizeof(struct oslHostAddrImpl))); + rtl_uString_newFromStr(&pRet->pHostName, + reinterpret_cast<sal_Unicode*>(&buf)); + pRet->pSockAddr = osl_createSocketAddr_(); + memcpy(& pRet->pSockAddr->m_sockaddr, + & pAddr->m_sockaddr, sizeof(struct sockaddr)); + return pRet; } + else + { + SAL_INFO("sal.osl", "GetNameInfoW failed: " << WSAGetLastError()); + } +#endif // _WIN32_WINNT } return nullptr; @@ -1289,9 +1147,6 @@ oslSocketResult SAL_CALL osl_connectSocketTo ( if (pAddr == nullptr) /* EDESTADDRREQ */ return osl_Socket_Error; - if (!osl_attemptSocketDialupImpl()) /* ENETDOWN */ - return osl_Socket_Error; - if (pTimeout == nullptr) { if(connect(pSocket->m_Socket, diff --git a/sal/osl/w32/sockimpl.h b/sal/osl/w32/sockimpl.h index fd1cd3daecab..1e3898e51fdb 100644 --- a/sal/osl/w32/sockimpl.h +++ b/sal/osl/w32/sockimpl.h @@ -47,37 +47,6 @@ struct oslSocketAddrImpl oslSocket osl_createSocketImpl_(SOCKET Socket); void osl_destroySocketImpl_(oslSocket pImpl); -/*****************************************************************************/ -/* oslSocketDialupImpl */ -/*****************************************************************************/ -#define INTERNET_MODULE_NAME "wininet.dll" - -#define INTERNET_CONNECTION_HANGUP 0x80000000L - -typedef DWORD (WINAPI *INTERNETATTEMPTCONNECT) ( - DWORD dwReserved); -typedef BOOL (WINAPI *INTERNETAUTODIAL) ( - DWORD dwFlags, DWORD dwReserved); -typedef BOOL (WINAPI *INTERNETAUTODIALHANGUP) ( - DWORD dwReserved); -typedef BOOL (WINAPI *INTERNETGETCONNECTEDSTATE) ( - LPDWORD lpdwFlags, DWORD dwReserved); - -typedef struct osl_socket_dialup_impl_st -{ - CRITICAL_SECTION m_hMutex; - HINSTANCE m_hModule; - INTERNETATTEMPTCONNECT m_pfnAttemptConnect; - INTERNETAUTODIAL m_pfnAutodial; - INTERNETAUTODIALHANGUP m_pfnAutodialHangup; - INTERNETGETCONNECTEDSTATE m_pfnGetConnectedState; - DWORD m_dwFlags; -} oslSocketDialupImpl; - -/*****************************************************************************/ -/* The End */ -/*****************************************************************************/ - #ifdef __cplusplus } #endif |