[Lldb-commits] [lldb] r190654 - Change OptionValueFileSpec::SetValueFromCString to strip off whitespace,

Jason Molenda jmolenda at apple.com
Thu Sep 12 19:33:16 PDT 2013


Author: jmolenda
Date: Thu Sep 12 21:33:15 2013
New Revision: 190654

URL: http://llvm.org/viewvc/llvm-project?rev=190654&view=rev
Log:
Change OptionValueFileSpec::SetValueFromCString to strip off whitespace,
single-quote and double-quotemarks from around file paths specified to
settings like target.expr-prefix or target.process.python-os-plugin-path.
<rdar://problem/14970457> 

Modified:
    lldb/trunk/source/Interpreter/OptionValueFileSpec.cpp

Modified: lldb/trunk/source/Interpreter/OptionValueFileSpec.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Interpreter/OptionValueFileSpec.cpp?rev=190654&r1=190653&r2=190654&view=diff
==============================================================================
--- lldb/trunk/source/Interpreter/OptionValueFileSpec.cpp (original)
+++ lldb/trunk/source/Interpreter/OptionValueFileSpec.cpp Thu Sep 12 21:33:15 2013
@@ -84,8 +84,21 @@ OptionValueFileSpec::SetValueFromCString
     case eVarSetOperationAssign:
         if (value_cstr && value_cstr[0])
         {
+            // The setting value may have whitespace, double-quotes, or single-quotes around the file
+            // path to indicate that internal spaces are not word breaks.  Strip off any ws & quotes
+            // from the start and end of the file path - we aren't doing any word // breaking here so 
+            // the quoting is unnecessary.  NB this will cause a problem if someone tries to specify
+            // a file path that legitimately begins or ends with a " or ' character, or whitespace.
+            std::string filepath(value_cstr);
+            auto prefix_chars_to_trim = filepath.find_first_not_of ("\"' \t");
+            if (prefix_chars_to_trim != std::string::npos && prefix_chars_to_trim > 0)
+                filepath.erase(0, prefix_chars_to_trim);
+            auto suffix_chars_to_trim = filepath.find_last_not_of ("\"' \t");
+            if (suffix_chars_to_trim != std::string::npos && suffix_chars_to_trim < filepath.size())
+                filepath.erase (suffix_chars_to_trim + 1);
+
             m_value_was_set = true;
-            m_current_value.SetFile(value_cstr, true);
+            m_current_value.SetFile(filepath.c_str(), true);
         }
         else
         {





More information about the lldb-commits mailing list