From bfcbde1c97971211d2ecdf5c5f8033eb138658d1 Mon Sep 17 00:00:00 2001
From: leak <leak@bitmx.net>
Date: Sun, 22 Jun 2014 16:29:49 +0200
Subject: Various cleanups and fixes due to feedback

---
 src/server/shared/Configuration/Config.cpp   | 10 +++++-----
 src/server/shared/Cryptography/BigNumber.cpp |  4 ++--
 src/server/shared/Cryptography/BigNumber.h   |  2 +-
 src/server/shared/Utilities/Util.cpp         |  7 ++-----
 4 files changed, 10 insertions(+), 13 deletions(-)

(limited to 'src/server/shared')

diff --git a/src/server/shared/Configuration/Config.cpp b/src/server/shared/Configuration/Config.cpp
index 95336457428..9e0e57eb198 100644
--- a/src/server/shared/Configuration/Config.cpp
+++ b/src/server/shared/Configuration/Config.cpp
@@ -35,13 +35,13 @@ bool ConfigMgr::LoadInitial(char const* file)
 
     try
     {
-        ptree temp;
-        boost::property_tree::ini_parser::read_ini(file, temp);
+        ptree fullTree;
+        boost::property_tree::ini_parser::read_ini(file, fullTree);
 
-
-        for (auto bla : temp)
+        // Since we're using only one section per config file, we skip the section and have direct property access
+        for (auto section : fullTree)
         {
-            _config = bla.second;
+            _config = section.second;
             break;
         }
     }
diff --git a/src/server/shared/Cryptography/BigNumber.cpp b/src/server/shared/Cryptography/BigNumber.cpp
index ed355b5f63c..c5e0635c5ec 100644
--- a/src/server/shared/Cryptography/BigNumber.cpp
+++ b/src/server/shared/Cryptography/BigNumber.cpp
@@ -168,7 +168,7 @@ bool BigNumber::isZero() const
     return BN_is_zero(_bn);
 }
 
-std::unique_ptr<uint8> BigNumber::AsByteArray(int32 minSize, bool littleEndian)
+std::unique_ptr<uint8[]> BigNumber::AsByteArray(int32 minSize, bool littleEndian)
 {
     int length = (minSize >= GetNumBytes()) ? minSize : GetNumBytes();
 
@@ -184,7 +184,7 @@ std::unique_ptr<uint8> BigNumber::AsByteArray(int32 minSize, bool littleEndian)
     if (littleEndian)
         std::reverse(array, array + length);
 
-    std::unique_ptr<uint8> ret(array);
+    std::unique_ptr<uint8[]> ret(array);
     return ret;
 }
 
diff --git a/src/server/shared/Cryptography/BigNumber.h b/src/server/shared/Cryptography/BigNumber.h
index d0a09dca6c4..848b3da3e2d 100644
--- a/src/server/shared/Cryptography/BigNumber.h
+++ b/src/server/shared/Cryptography/BigNumber.h
@@ -88,7 +88,7 @@ class BigNumber
 
         uint32 AsDword();
 
-        std::unique_ptr<uint8> AsByteArray(int32 minSize = 0, bool littleEndian = true);
+        std::unique_ptr<uint8[]> AsByteArray(int32 minSize = 0, bool littleEndian = true);
 
         char * AsHexStr() const;
         char * AsDecStr() const;
diff --git a/src/server/shared/Utilities/Util.cpp b/src/server/shared/Utilities/Util.cpp
index f80730af05d..1290a7dbae4 100644
--- a/src/server/shared/Utilities/Util.cpp
+++ b/src/server/shared/Utilities/Util.cpp
@@ -139,16 +139,13 @@ void stripLineInvisibleChars(std::string &str)
 
 }
 
+#if (defined(WIN32) || defined(_WIN32) || defined(__WIN32__))
 struct tm* localtime_r(const time_t* time, struct tm *result)
 {
-#if (defined(WIN32) || defined(_WIN32) || defined(__WIN32__))
     localtime_s(result, time);
     return result;
-#else
-    return localtime_r(&time, &result); // POSIX
-#endif
 }
-
+#endif
 
 std::string secsToTimeString(uint64 timeInSecs, bool shortText, bool hoursOnly)
 {
-- 
cgit v1.2.3