Fixed multiple embeddings of square brackets so that it doesn't crash

the client. This commit makes it so that all brackets except for the
innermost brackets are ignored.

Signed-off-by: Ira Rice <irarice@gmail.com>

1 files changed, 7 insertions, 0 deletions

diff --git a/src/being.cpp b/src/being.cpp
index c5a98a2d..8ed66c7a 100644
--- a/src/being.cpp
+++ b/src/being.cpp
@@ -187,6 +187,13 @@ void Being::setSpeech(const std::string &text, Uint32 time)
 
     while (start != std::string::npos && end != std::string::npos)
     {
+        // Catch multiple embeds and ignore them so it doesn't crash the client.
+        while ((mSpeech.find('[', start + 1) != std::string::npos) && 
+               (mSpeech.find('[', start + 1) < end))
+        {
+            start = mSpeech.find('[', start + 1);
+        }
+
         std::string::size_type position = mSpeech.find('|');
         mSpeech.erase(end, 1);
         mSpeech.erase(start, (position - start) + 1);