Code

correct cache_entry allocation
authorJeff King <peff@peff.net>
Thu, 23 Oct 2008 04:30:58 +0000 (04:30 +0000)
committerJunio C Hamano <gitster@pobox.com>
Sun, 2 Nov 2008 06:46:34 +0000 (23:46 -0700)
commit13494ed14c3539b3e36ff47d1d8b65f5a9a3043b
treec02062778a40cb0cb3c63d5675e8e94dcc643231
parentfaf1dc7223be9ffddf775916913bb8e22762cdfb
correct cache_entry allocation

Most cache_entry structs are allocated by using the
cache_entry_size macro, which rounds the size of the struct
up to the nearest multiple of 8 bytes (presumably to avoid
memory fragmentation).

There is one exception: the special "conflict entry" is
allocated with an empty name, and so is explicitly given
just one extra byte to hold the NUL.

However, later code doesn't realize that this particular
struct has been allocated differently, and happily tries
reading and copying it based on the ce_size macro, which
assumes the 8-byte alignment.

This can lead to reading uninitalized data, though since
that data is simply padding, there shouldn't be any problem
as a result. Still, it makes sense to hold the padding
assumption so as not to surprise later maintainers.

This fixes valgrind errors in t1005, t3030, t4002, and
t4114.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
unpack-trees.c