From 855c1bbc6859971137d13f9ae878910fba036d56 Mon Sep 17 00:00:00 2001
From: R4SAS <r4sas@i2pmail.org>
Date: Wed, 4 Oct 2017 06:07:39 +0300
Subject: [PATCH] warnings workaround

---
 src/init.cpp |  6 ++--
 src/net.cpp  | 90 +++++++++++++++++++++++++++-------------------------
 2 files changed, 51 insertions(+), 45 deletions(-)

diff --git a/src/init.cpp b/src/init.cpp
index 0a97c25..8faad99 100644
--- a/src/init.cpp
+++ b/src/init.cpp
@@ -136,7 +136,9 @@ void Shutdown()
     boost::filesystem::remove(GetPidFile());
     UnregisterWallet(pwalletMain);
     if (pwalletMain)
+    {
         delete pwalletMain;
+    }
 
 	I2PSession::Instance ().Stop ();
 
@@ -826,7 +828,7 @@ bool AppInit2(boost::thread_group& threadGroup)
 #endif
 
     CService addrProxy;
-    bool fProxy = false;
+    //bool fProxy = false; // is usable?
     if (mapArgs.count("-proxy")) {
         addrProxy = CService(mapArgs["-proxy"], 9050);
         if (!addrProxy.IsValid())
@@ -841,7 +843,7 @@ bool AppInit2(boost::thread_group& threadGroup)
 #endif
             SetNameProxy(addrProxy, nSocksVersion);
         }
-        fProxy = true;
+        //fProxy = true; // is usable?
     }
 
     // see Step 2: parameter interactions for more information about these
diff --git a/src/net.cpp b/src/net.cpp
index 3b28f75..db3d8ff 100644
--- a/src/net.cpp
+++ b/src/net.cpp
@@ -990,57 +990,61 @@ void ThreadSocketHandler()
         // Accept new connections
         //
         if (!IsI2POnly())
-        BOOST_FOREACH(SOCKET hListenSocket, vhListenSocket)
-        if (hListenSocket != INVALID_SOCKET && FD_ISSET(hListenSocket, &fdsetRecv))
         {
+            BOOST_FOREACH(SOCKET hListenSocket, vhListenSocket)
+            {
+                if (hListenSocket != INVALID_SOCKET && FD_ISSET(hListenSocket, &fdsetRecv))
+                {
 #ifdef USE_IPV6
-            struct sockaddr_storage sockaddr;
+                    struct sockaddr_storage sockaddr;
 #else
-            struct sockaddr sockaddr;
+                    struct sockaddr sockaddr;
 #endif
-            socklen_t len = sizeof(sockaddr);
-            SOCKET hSocket = accept(hListenSocket, (struct sockaddr*)&sockaddr, &len);
-            CAddress addr;
-            int nInbound = 0;
+                    socklen_t len = sizeof(sockaddr);
+                    SOCKET hSocket = accept(hListenSocket, (struct sockaddr*)&sockaddr, &len);
+                    CAddress addr;
+                    int nInbound = 0;
 
-            if (hSocket != INVALID_SOCKET)
-                if (!addr.SetSockAddr((const struct sockaddr*)&sockaddr))
-                    printf("Warning: Unknown socket family\n");
+                    if (hSocket != INVALID_SOCKET)
+                        if (!addr.SetSockAddr((const struct sockaddr*)&sockaddr))
+                            printf("Warning: Unknown socket family\n");
 
-            {
-                LOCK(cs_vNodes);
-                BOOST_FOREACH(CNode* pnode, vNodes)
-                    if (pnode->fInbound)
-                        nInbound++;
-            }
+                    {
+                        LOCK(cs_vNodes);
+                        BOOST_FOREACH(CNode* pnode, vNodes)
+                            if (pnode->fInbound)
+                                nInbound++;
+                    }
 
-            if (hSocket == INVALID_SOCKET)
-            {
-                int nErr = WSAGetLastError();
-                if (nErr != WSAEWOULDBLOCK)
-                    printf("socket error accept failed: %d\n", nErr);
-            }
-            else if (nInbound >= nMaxConnections - MAX_OUTBOUND_CONNECTIONS)
-            {
-                {
-                    LOCK(cs_setservAddNodeAddresses);
-                    if (!setservAddNodeAddresses.count(addr))
+                    if (hSocket == INVALID_SOCKET)
+                    {
+                        int nErr = WSAGetLastError();
+                        if (nErr != WSAEWOULDBLOCK)
+                            printf("socket error accept failed: %d\n", nErr);
+                    }
+                    else if (nInbound >= nMaxConnections - MAX_OUTBOUND_CONNECTIONS)
+                    {
+                        {
+                            LOCK(cs_setservAddNodeAddresses);
+                            if (!setservAddNodeAddresses.count(addr))
+                                closesocket(hSocket);
+                        }
+                    }
+                    else if (CNode::IsBanned(addr))
+                    {
+                        printf("connection from %s dropped (banned)\n", addr.ToString().c_str());
                         closesocket(hSocket);
-                }
-            }
-            else if (CNode::IsBanned(addr))
-            {
-                printf("connection from %s dropped (banned)\n", addr.ToString().c_str());
-                closesocket(hSocket);
-            }
-            else
-            {
-                printf("accepted connection %s\n", addr.ToString().c_str());
-                CNode* pnode = new CNode(hSocket, addr, "", true);
-                pnode->AddRef();
-                {
-                    LOCK(cs_vNodes);
-                    vNodes.push_back(pnode);
+                    }
+                    else
+                    {
+                        printf("accepted connection %s\n", addr.ToString().c_str());
+                        CNode* pnode = new CNode(hSocket, addr, "", true);
+                        pnode->AddRef();
+                        {
+                            LOCK(cs_vNodes);
+                            vNodes.push_back(pnode);
+                        }
+                    }
                 }
             }
         }