Protect the result

src/net/AresHandler.cxx

@@ -16,8 +16,10 @@
 /* system implementation headers */
 #include <cerrno>
 #include <cstring>
+#include <mutex>
 
 bool AresHandler::globallyInited = false;
+std::mutex callback_mutex;
 
 AresHandler::AresHandler(int _index)
     : index(_index), status(None)
@@ -135,6 +137,9 @@ void AresHandler::callback(int callbackStatus, struct hostent *hostent)
 {
     if (callbackStatus == ARES_EDESTRUCTION)
         return;
+
+    const std::lock_guard<std::mutex> lock(callback_mutex);
+
     if (callbackStatus != ARES_SUCCESS)
     {
         logDebugMessage(1,"Player [%d] failed to resolve: error %d\n", index,
@@ -156,23 +161,27 @@ void AresHandler::callback(int callbackStatus, struct hostent *hostent)
 
 const char *AresHandler::getHostname()
 {
+    const std::lock_guard<std::mutex> lock(callback_mutex);
+
     return hostName.c_str();
 }
 
 AresHandler::ResolutionStatus AresHandler::getHostAddress(struct in_addr
         *clientAddr)
 {
+    const std::lock_guard<std::mutex> lock(callback_mutex);
+
     if (status == HbNSucceeded)
         memcpy(clientAddr, &hostAddress, sizeof(hostAddress));
     return status;
 }
 
-void AresHandler::setFd(fd_set *read_set, fd_set *write_set, int &maxFile)
+void AresHandler::setFd(fd_set *, fd_set *, int &)
 {
     return;
 }
 
-void AresHandler::process(fd_set *read_set, fd_set *write_set)
+void AresHandler::process(fd_set *, fd_set *)
 {
     return;
 }