diff options
author | Andy Kaylor <andrew.kaylor@intel.com> | 2023-08-14 14:27:57 -0700 |
---|---|---|
committer | Andy Kaylor <andrew.kaylor@intel.com> | 2023-08-16 15:42:49 -0700 |
commit | 6664e80ace0845d34eecebe0d5da0b576cde39ef (patch) | |
tree | 41047dbf3c6238c7cde709887cc72838df9832d9 /clang/lib/ExtractAPI/Serialization/SymbolGraphSerializer.cpp | |
parent | e698695fbbf62e6676f8907665187f2d2c4d814b (diff) | |
download | llvm-6664e80ace0845d34eecebe0d5da0b576cde39ef.zip llvm-6664e80ace0845d34eecebe0d5da0b576cde39ef.tar.gz llvm-6664e80ace0845d34eecebe0d5da0b576cde39ef.tar.bz2 |
Fix integer overflow in ConcurrentHashtTableByPtr
This change fixes a case where there was a potential integer overflow
when multiplying two 32-bit values and assigning the result to a 64-bit
value.
The potential overflow has been present since https://reviews.llvm.org/D132455
was re-landed. The initial version of the patch defined MaxBucketSize and
NumberOfBuckets as size_t (which I guess would still be a problem for some
targets). When the patch was re-landed they were defined as uint32_t, making
the calculation of ExtHashMask subject to overflow.
Differential Revision: https://reviews.llvm.org/D158117
Diffstat (limited to 'clang/lib/ExtractAPI/Serialization/SymbolGraphSerializer.cpp')
0 files changed, 0 insertions, 0 deletions