[clang] [clang][ASTImporter] fix assert fail due to offset overflow (PR #79084)

Ding Fei via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 29 07:46:31 PST 2024


================
@@ -9896,9 +9903,13 @@ Expected<FileID> ASTImporter::Import(FileID FromID, bool IsBuiltin) {
         // FIXME: The filename may be a virtual name that does probably not
         // point to a valid file and we get no Entry here. In this case try with
         // the memory buffer below.
-        if (Entry)
+        if (Entry) {
+          if (isBufferSizeOverflow(ToSM, Entry->getSize()))
+            return llvm::make_error<ASTImportError>(
+                ASTImportError::UnsupportedConstruct);
----------------
danix800 wrote:

`ASTImportError::Unknown` would crash CTU module too, `ASTImportError::UnsupportedConstruct` might be deliberately chosen so the analysis could continue in this case.

https://github.com/llvm/llvm-project/pull/79084


More information about the cfe-commits mailing list