Staging
v0.8.1
https://github.com/python/cpython
Revision 392520bd78cd18639a27e5d2803c2e1c2bd593a8 authored by Miss Islington (bot) on 20 April 2018, 17:06:21 UTC, committed by Gregory P. Smith on 20 April 2018, 17:06:21 UTC
Fix clang ubsan (undefined behavior sanitizer) warnings in dictobject.c by
adjusting how the internal struct _dictkeysobject shared keys structure is
declared.

This remains ABI compatible.  We get rid of the union at the end of the
struct being used for conveinence to avoid typecasting in favor of char[]
variable length array at the end of a struct. This is known to clang to be
used for variable sized objects and will not cause an undefined behavior
problem.  Similarly, char arrays do not have strict aliasing undefined
behavior when cast.

PEP-007 does not currently list variable length arrays (VLAs) as allowed
in our subset of C99.  If this turns out to be a problem, the fix to this is
to change the char `dk_indices[]` into `dk_indices[1]` and restore the
three size computation subtractions this change removes:
  `- Py_MEMBER_SIZE(PyDictKeysObject, dk_indices)`

If this works as is I'll make a separate PR to update PEP-007.
(cherry picked from commit 397f1b28c4a12e3b3ed59a89599eabc457412649)
1 parent 18cd828
History
Tip revision: 392520bd78cd18639a27e5d2803c2e1c2bd593a8 authored by Miss Islington (bot) on 20 April 2018, 17:06:21 UTC
bpo-33312: Fix clang ubsan out of bounds warnings in dict. (GH-6537) (GH-6543)
Tip revision: 392520b
File Mode Size
clinic
Python-ast.c -rw-r--r-- 263.0 KB
README -rw-r--r-- 62 bytes
_warnings.c -rw-r--r-- 37.1 KB
asdl.c -rw-r--r-- 1.4 KB
ast.c -rw-r--r-- 161.5 KB
ast_opt.c -rw-r--r-- 22.9 KB
ast_unparse.c -rw-r--r-- 28.8 KB
bltinmodule.c -rw-r--r-- 83.5 KB
bootstrap_hash.c -rw-r--r-- 17.9 KB
ceval.c -rw-r--r-- 158.7 KB
ceval_gil.h -rw-r--r-- 9.0 KB
codecs.c -rw-r--r-- 44.7 KB
compile.c -rw-r--r-- 156.8 KB
condvar.h -rw-r--r-- 8.9 KB
context.c -rw-r--r-- 26.7 KB
dtoa.c -rw-r--r-- 78.5 KB
dup2.c -rw-r--r-- 759 bytes
dynamic_annotations.c -rw-r--r-- 6.6 KB
dynload_aix.c -rw-r--r-- 5.7 KB
dynload_dl.c -rw-r--r-- 581 bytes
dynload_hpux.c -rw-r--r-- 1.9 KB
dynload_shlib.c -rw-r--r-- 3.3 KB
dynload_stub.c -rw-r--r-- 186 bytes
dynload_win.c -rw-r--r-- 10.0 KB
errors.c -rw-r--r-- 31.9 KB
fileutils.c -rw-r--r-- 49.2 KB
formatter_unicode.c -rw-r--r-- 49.6 KB
frozen.c -rw-r--r-- 1.7 KB
frozenmain.c -rw-r--r-- 2.9 KB
future.c -rw-r--r-- 4.6 KB
getargs.c -rw-r--r-- 75.5 KB
getcompiler.c -rw-r--r-- 538 bytes
getcopyright.c -rw-r--r-- 511 bytes
getopt.c -rw-r--r-- 4.4 KB
getplatform.c -rw-r--r-- 133 bytes
getversion.c -rw-r--r-- 308 bytes
graminit.c -rw-r--r-- 44.0 KB
hamt.c -rw-r--r-- 78.8 KB
import.c -rw-r--r-- 63.9 KB
importdl.c -rw-r--r-- 7.2 KB
importdl.h -rw-r--r-- 487 bytes
importlib.h -rw-r--r-- 96.3 KB
importlib_external.h -rw-r--r-- 139.5 KB
makeopcodetargets.py -rwxr-xr-x 1.6 KB
marshal.c -rw-r--r-- 49.8 KB
modsupport.c -rw-r--r-- 16.9 KB
mysnprintf.c -rw-r--r-- 3.3 KB
mystrtoul.c -rw-r--r-- 8.1 KB
opcode_targets.h -rw-r--r-- 6.3 KB
pathconfig.c -rw-r--r-- 10.2 KB
peephole.c -rw-r--r-- 17.2 KB
pyarena.c -rw-r--r-- 5.5 KB
pyctype.c -rw-r--r-- 7.7 KB
pyfpe.c -rw-r--r-- 345 bytes
pyhash.c -rw-r--r-- 13.2 KB
pylifecycle.c -rw-r--r-- 63.6 KB
pymath.c -rw-r--r-- 1.5 KB
pystate.c -rw-r--r-- 39.7 KB
pystrcmp.c -rw-r--r-- 613 bytes
pystrhex.c -rw-r--r-- 1.7 KB
pystrtod.c -rw-r--r-- 40.1 KB
pythonrun.c -rw-r--r-- 43.5 KB
pytime.c -rw-r--r-- 26.3 KB
strdup.c -rw-r--r-- 277 bytes
structmember.c -rw-r--r-- 8.6 KB
symtable.c -rw-r--r-- 57.5 KB
sysmodule.c -rw-r--r-- 80.4 KB
thread.c -rw-r--r-- 5.3 KB
thread_nt.h -rw-r--r-- 11.2 KB
thread_pthread.h -rw-r--r-- 20.7 KB
traceback.c -rw-r--r-- 24.2 KB
wordcode_helpers.h -rw-r--r-- 1.2 KB

README

back to top