hokein created this revision. hokein added a reviewer: kadircet. Herald added a subscriber: arphaman. Herald added a project: All. hokein requested review of this revision. Herald added subscribers: MaskRay, ilya-biryukov. Herald added a project: clang-tools-extra.
We encounter a few internal reports that we're dereference a nullptr. Unfortunately, no small reproduce testcase for this crash yet, but it makes the clangd more robost on broken code. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D143486 Files: clang-tools-extra/clangd/SemanticHighlighting.cpp Index: clang-tools-extra/clangd/SemanticHighlighting.cpp =================================================================== --- clang-tools-extra/clangd/SemanticHighlighting.cpp +++ clang-tools-extra/clangd/SemanticHighlighting.cpp @@ -521,8 +521,8 @@ return std::nullopt; const auto *Tok = TB.spelledTokenAt(Loc); - assert(Tok); - + if (!Tok) + return std::nullopt; return halfOpenToRange(SourceMgr, Tok->range(SourceMgr).toCharRange(SourceMgr)); }
Index: clang-tools-extra/clangd/SemanticHighlighting.cpp =================================================================== --- clang-tools-extra/clangd/SemanticHighlighting.cpp +++ clang-tools-extra/clangd/SemanticHighlighting.cpp @@ -521,8 +521,8 @@ return std::nullopt; const auto *Tok = TB.spelledTokenAt(Loc); - assert(Tok); - + if (!Tok) + return std::nullopt; return halfOpenToRange(SourceMgr, Tok->range(SourceMgr).toCharRange(SourceMgr)); }
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits