[clang-tools-extra] [clangd] Fix SIGSEGV crash when receiving a textDocument/didOpen request with the URI pointing to a directory (PR #177834)

[Baranov Victor via cfe-commits]

================
@@ -1027,6 +1031,10 @@ flattenSymbolHierarchy(llvm::ArrayRef<DocumentSymbol> 
Symbols,
 void ClangdLSPServer::onDocumentSymbol(const DocumentSymbolParams &Params,
                                        Callback<llvm::json::Value> Reply) {
   URIForFile FileURI = Params.textDocument.uri;
+  if (llvm::sys::fs::is_directory(FileURI.file())) {
+    return Reply(llvm::make_error<LSPError>("URI is a directory",
+                                            ErrorCode::InvalidParams));
+  }
----------------
vbvictor wrote:

```suggestion
  if (llvm::sys::fs::is_directory(FileURI.file()))
    return Reply(llvm::make_error<LSPError>("URI is a directory",
                                            ErrorCode::InvalidParams));
```

https://github.com/llvm/llvm-project/pull/177834
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits