[PATCH] D138830: [llvm] Check for overflows when computing load-command's addresses.

[Thorsten via Phabricator via llvm-commits]

tschuett added inline comments.


================
Comment at: llvm/lib/Object/MachOObjectFile.cpp:234
+  // done somewhere else (earlier).
+  uint32_t LoadCmdsEnd = llvm::SaturatingAdd<uintptr_t>(
+      reinterpret_cast<uintptr_t>(Obj.getData().data()), HeaderSize,
----------------
I am known to be stupid. You do saturating add with `uintptr_t` and claim errors on `uint32_t`?

Would it help to make the asserts into real errors? I like to be informed about malformed files in release builds.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D138830/new/

https://reviews.llvm.org/D138830