From a922b6c0703026021126c8833cd4d42a27126e11 Mon Sep 17 00:00:00 2001
From: Andrei Karas <akaras@inbox.ru>
Date: Sun, 5 Feb 2017 21:21:49 +0300
Subject: Fix incorrect buffer access in chatutils and in stringutils unit
 tests.

---
 src/utils/chatutils.cpp | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

(limited to 'src/utils/chatutils.cpp')

diff --git a/src/utils/chatutils.cpp b/src/utils/chatutils.cpp
index 3ad6c8a1f..d80e8d761 100644
--- a/src/utils/chatutils.cpp
+++ b/src/utils/chatutils.cpp
@@ -146,12 +146,16 @@ void replaceVars(std::string &str)
             if (*it != localPlayer->getName())
                 newStr.append(*it).append(",");
         }
-        if (newStr[newStr.size() - 1] == ',')
-            newStr = newStr.substr(0, newStr.size() - 1);
         if (!newStr.empty())
+        {
+            if (newStr[newStr.size() - 1] == ',')
+                newStr = newStr.substr(0, newStr.size() - 1);
             replaceAll(str, "<PEOPLE>", newStr);
+        }
         else
+        {
             replaceAll(str, "<PEOPLE>", "");
+        }
     }
     if (str.find("<PARTY>") != std::string::npos)
     {
@@ -167,12 +171,16 @@ void replaceVars(std::string &str)
                 if (*it != localPlayer->getName())
                     newStr.append(*it).append(",");
             }
-            if (newStr[newStr.size() - 1] == ',')
-                newStr = newStr.substr(0, newStr.size() - 1);
             if (!newStr.empty())
+            {
+                if (newStr[newStr.size() - 1] == ',')
+                    newStr = newStr.substr(0, newStr.size() - 1);
                 replaceAll(str, "<PARTY>", newStr);
+            }
             else
+            {
                 replaceAll(str, "<PARTY>", "");
+            }
         }
         else
         {
-- 
cgit v1.2.3-70-g09d2