Author: sammccall Date: Tue Nov 7 06:45:31 2017 New Revision: 317580 URL: http://llvm.org/viewvc/llvm-project?rev=317580&view=rev Log: [clangd] don't crash on invalid JSON-RPC ID
Modified: clang-tools-extra/trunk/clangd/JSONRPCDispatcher.cpp Modified: clang-tools-extra/trunk/clangd/JSONRPCDispatcher.cpp URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clangd/JSONRPCDispatcher.cpp?rev=317580&r1=317579&r2=317580&view=diff ============================================================================== --- clang-tools-extra/trunk/clangd/JSONRPCDispatcher.cpp (original) +++ clang-tools-extra/trunk/clangd/JSONRPCDispatcher.cpp Tue Nov 7 06:45:31 2017 @@ -12,6 +12,7 @@ #include "ProtocolHandlers.h" #include "Trace.h" #include "llvm/ADT/SmallString.h" +#include "llvm/ADT/StringExtras.h" #include "llvm/Support/SourceMgr.h" #include "llvm/Support/YAMLParser.h" #include <istream> @@ -149,7 +150,8 @@ bool JSONRPCDispatcher::call(StringRef C ID.emplace(V.str()); } else { double D; - if (!V.getAsDouble(D)) + // FIXME: this is locale-sensitive. + if (llvm::to_float(V, D)) ID.emplace(D); } } _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits