diff options
| author | Gregory P. Smith <greg@krypto.org> | 2012-12-11 03:51:29 (GMT) |
|---|---|---|
| committer | Gregory P. Smith <greg@krypto.org> | 2012-12-11 03:51:29 (GMT) |
| commit | 27dc02e8c5b8e0d8355b6a12d3121891604f2e7d (patch) | |
| tree | 38fc0332b6dbbebbc5acc19eb476e5995b5d2406 /Include | |
| parent | e348c8d154cf6342c79d627ebfe89dfe9de23817 (diff) | |
| parent | a82fe52acc120e56e4649249bbe3adb2e60baba6 (diff) | |
| download | cpython-27dc02e8c5b8e0d8355b6a12d3121891604f2e7d.zip cpython-27dc02e8c5b8e0d8355b6a12d3121891604f2e7d.tar.gz cpython-27dc02e8c5b8e0d8355b6a12d3121891604f2e7d.tar.bz2 | |
Fix the internals of our hash functions to used unsigned values during hash
computation as the overflow behavior of signed integers is undefined.
NOTE: This change is smaller compared to 3.2 as much of this cleanup had
already been done. I added the comment that my change in 3.2 added so that the
code would match up. Otherwise this just adds or synchronizes appropriate UL
designations on some constants to be pedantic.
In practice we require compiling everything with -fwrapv which forces overflow
to be defined as twos compliment but this keeps the code cleaner for checkers
or in the case where someone has compiled it without -fwrapv or their
compiler's equivalent. We could work to get rid of the -fwrapv requirement
in 3.4 but that requires more planning.
Found by Clang trunk's Undefined Behavior Sanitizer (UBSan).
Cleanup only - no functionality or hash values change.
Diffstat (limited to 'Include')
| -rw-r--r-- | Include/pyport.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Include/pyport.h b/Include/pyport.h index bef6b3c..8d38ccb 100644 --- a/Include/pyport.h +++ b/Include/pyport.h @@ -145,7 +145,7 @@ Used in: PY_LONG_LONG #endif /* Prime multiplier used in string and various other hashes. */ -#define _PyHASH_MULTIPLIER 1000003 /* 0xf4243 */ +#define _PyHASH_MULTIPLIER 1000003UL /* 0xf4243 */ /* Parameters used for the numeric hash implementation. See notes for _Py_HashDouble in Objects/object.c. Numeric hashes are based on |