Don't crash when we have a .a file that contains an object with a 16 character name. Any calls to std::string::erase must be bounds checked.

<rdar://problem/22260988>

llvm-svn: 244984

1 files changed, 3 insertions, 1 deletions

diff --git a/lldb/source/Plugins/ObjectContainer/BSD-Archive/ObjectContainerBSDArchive.cpp b/lldb/source/Plugins/ObjectContainer/BSD-Archive/ObjectContainerBSDArchive.cpp
index 8209d23c0ae..d9d399c5eff 100644
--- a/lldb/source/Plugins/ObjectContainer/BSD-Archive/ObjectContainerBSDArchive.cpp
+++ b/lldb/source/Plugins/ObjectContainer/BSD-Archive/ObjectContainerBSDArchive.cpp
@@ -105,7 +105,9 @@ ObjectContainerBSDArchive::Object::Extract (const DataExtractor& data, lldb::off
     {
         // Strip off any spaces (if the object file name contains spaces it
         // will use the extended format above).
-        str.erase (str.find(' '));
+        const size_t space_pos = str.find(' ');
+        if (space_pos != std::string::npos)
+            str.erase (space_pos);
         ar_name.SetCString(str.c_str());
     }