]> granicus.if.org Git - postgresql/commit
Adjust SP-GiST WAL record formats to reduce alignment padding.
authorHeikki Linnakangas <heikki.linnakangas@iki.fi>
Thu, 5 Jun 2014 09:55:35 +0000 (12:55 +0300)
committerHeikki Linnakangas <heikki.linnakangas@iki.fi>
Thu, 5 Jun 2014 09:55:35 +0000 (12:55 +0300)
commit8776faa81cb651322b8993422bdd4633f1f6a487
tree6640167704ea17b559704c800bcf36a9cfd05a22
parentd4d48a5edd9eb28a7f2ee2e4cbe20d984274982e
Adjust SP-GiST WAL record formats to reduce alignment padding.

The way the code was written, the padding was copied from uninitialized
memory areas.. Because the structs are local variables in the code where
the WAL records are constructed, making them larger and zeroing the padding
bytes would not make the code very pretty, so rather than fixing this
directly by zeroing out the padding bytes, it seems more clear to not try to
align the tuples in the WAL records. The redo functions are taught to copy
the tuple header to a local variable to avoid unaligned access.

Stable-branches have the same problem, but we can't change the WAL format
there, so fix in master only. Reading a few random extra bytes at the stack
is harmless in practice, so it's not worth crafting a different
back-patchable fix.

Per reports from Kevin Grittner and Andres Freund, using clang static
analyzer and Valgrind, respectively.
src/backend/access/spgist/spgdoinsert.c
src/backend/access/spgist/spgvacuum.c
src/backend/access/spgist/spgxlog.c
src/include/access/spgist_private.h
src/include/access/xlog_internal.h