diff options
author | Alex Tanskanen <aleta@cendio.com> | 2020-03-10 13:33:01 +0100 |
---|---|---|
committer | Alex Tanskanen <aleta@cendio.com> | 2020-03-17 14:46:20 +0100 |
commit | e52923415e55add9aa448ffcbb018a84fa39b742 (patch) | |
tree | 5ac63ed06a13c6a6a28e389ec051c4e82bfca2e4 /common | |
parent | 01ca961b0cf009a6f1e4603ad84b09d475b9f515 (diff) | |
download | tigervnc-e52923415e55add9aa448ffcbb018a84fa39b742.tar.gz tigervnc-e52923415e55add9aa448ffcbb018a84fa39b742.zip |
Throw GAIException() for getaddrinfo errors
Created a new subclass of Exception called GAIException() that will
handle error messages from getaddrinfo() instead of letting Exception()
handle it. GAIException() will make use of gai_strerror() to map the
error code to text. On Windows, gai_strerrorW() must be used if the text
is encoded with UTF-8.
Diffstat (limited to 'common')
-rw-r--r-- | common/network/TcpSocket.cxx | 9 | ||||
-rw-r--r-- | common/rdr/Exception.cxx | 18 | ||||
-rw-r--r-- | common/rdr/Exception.h | 9 |
3 files changed, 28 insertions, 8 deletions
diff --git a/common/network/TcpSocket.cxx b/common/network/TcpSocket.cxx index f5d92e25..07c8d2cd 100644 --- a/common/network/TcpSocket.cxx +++ b/common/network/TcpSocket.cxx @@ -133,8 +133,7 @@ TcpSocket::TcpSocket(const char *host, int port) hints.ai_next = NULL; if ((result = getaddrinfo(host, NULL, &hints, &ai)) != 0) { - throw Exception("unable to resolve host by name: %s", - gai_strerror(result)); + throw GAIException("unable to resolve host by name", result); } sock = -1; @@ -452,8 +451,7 @@ void network::createTcpListeners(std::list<SocketListener*> *listeners, snprintf (service, sizeof (service) - 1, "%d", port); service[sizeof (service) - 1] = '\0'; if ((result = getaddrinfo(addr, service, &hints, &ai)) != 0) - throw rdr::Exception("unable to resolve listening address: %s", - gai_strerror(result)); + throw GAIException("unable to resolve listening address", result); try { createTcpListeners(listeners, ai); @@ -645,8 +643,7 @@ TcpFilter::Pattern TcpFilter::parsePattern(const char* p) { } if ((result = getaddrinfo (p, NULL, &hints, &ai)) != 0) { - throw Exception("unable to resolve host by name: %s", - gai_strerror(result)); + throw GAIException("unable to resolve host by name", result); } memcpy (&pattern.address.u.sa, ai->ai_addr, ai->ai_addrlen); diff --git a/common/rdr/Exception.cxx b/common/rdr/Exception.cxx index e0592b29..9b6096b6 100644 --- a/common/rdr/Exception.cxx +++ b/common/rdr/Exception.cxx @@ -31,6 +31,9 @@ #include <tchar.h> #include <winsock2.h> #include <windows.h> +#include <ws2tcpip.h> +#else +#include <netdb.h> #endif #include <string.h> @@ -49,6 +52,21 @@ Exception::Exception(const char *format, ...) { va_end(ap); } +GAIException::GAIException(const char* s, int err) + : Exception("%s", s) +{ + strncat(str_, ": ", len-1-strlen(str_)); +#ifdef _WIN32 + wchar_t currStr[len-strlen(str_)]; + wcsncpy(currStr, gai_strerrorW(err), len-1-strlen(str_)); + WideCharToMultiByte(CP_UTF8, 0, currStr, -1, str_+strlen(str_), + len-1-strlen(str_), 0, 0); +#else + //FIXME: perhaps print the error number (NNNN) + strncat(str_, gai_strerror(err), len-1-strlen(str_)); +#endif +} + SystemException::SystemException(const char* s, int err_) : Exception("%s", s), err(err_) { diff --git a/common/rdr/Exception.h b/common/rdr/Exception.h index 69abbedb..eb3c8a9d 100644 --- a/common/rdr/Exception.h +++ b/common/rdr/Exception.h @@ -40,12 +40,17 @@ namespace rdr { struct SystemException : public Exception { int err; SystemException(const char* s, int err_); - }; + }; + + struct GAIException : public Exception { + int err; + GAIException(const char* s, int err_); + }; struct TimedOut : public Exception { TimedOut() : Exception("Timed out") {} }; - + struct EndOfStream : public Exception { EndOfStream() : Exception("End of stream") {} }; |