]> git.eshelyaron.com Git - emacs.git/commit
Port flexible array members to GCC + valgrind
authorPaul Eggert <eggert@cs.ucla.edu>
Thu, 8 Sep 2016 01:08:45 +0000 (18:08 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Thu, 8 Sep 2016 01:10:11 +0000 (18:10 -0700)
commitd2f1971dd570439da4198fa76603b53b072060f8
tree38b1ddbeda27b6ed6ac52205169624608cc597fd
parent12a7e0f88eaa68aabe7e32589e2d5c8f776f6346
Port flexible array members to GCC + valgrind

These changes are needed to conform to the C standard's rule for
allocating structs containing flexible array members.  C11 says
that malloc (offsetof (struct s, m) + n) does not suffice to
allocate a struct with an n-byte tail; instead, malloc’s arg
should be rounded up to the nearest multiple of alignof (struct s).
Although this is arguably a defect in C11, gcc -O2 + valgrind
sometimes complains when this rule is violated, and when debugging
it’s better to keep valgrind happy.
For details please see the thread containing the message at:
https://gcc.gnu.org/ml/gcc-patches/2016-09/msg00416.html
* lib-src/ebrowse.c, src/alloc.c, src/image.c, src/process.c:
Include flexmember.h.
* lib-src/ebrowse.c (add_sym, add_member, make_namespace)
(register_namespace_alias):
* src/alloc.c (SDATA_SIZE, allocate_string_data):
* src/image.c (xpm_cache_color, imagemagick_create_cache):
* src/process.c (Fmake_network_process):
Use FLEXSIZEOF instead of offsetof and addition.
* src/alloc.c (SDATA_SIZE, vector_alignment):
Use FLEXALIGNOF instead of sizeof (ptrdiff_t).
* src/lisp.h (ALIGNOF_STRUCT_LISP_VECTOR):
Remove, as alloc.c can now calculate this on its own.
lib-src/ebrowse.c
src/alloc.c
src/image.c
src/lisp.h
src/process.c