]> granicus.if.org Git - postgresql/commit
Fix another longstanding problem in copy_relation_data: it was blithely
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 29 Jul 2010 19:24:05 +0000 (19:24 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 29 Jul 2010 19:24:05 +0000 (19:24 +0000)
commit8e21bf52504616024618f3386dc68e8936405dc2
tree74c6f1582fbf5d956a43516b69c283208678d18b
parent3137b496985b4e2242220bdabd4de86cf5c2d36e
Fix another longstanding problem in copy_relation_data: it was blithely
assuming that a local char[] array would be aligned on at least a word
boundary.  There are architectures on which that is pretty much guaranteed to
NOT be the case ... and those arches also don't like non-aligned memory
accesses, meaning that log_newpage() would crash if it ever got invoked.
Even on Intel-ish machines there's a potential for a large performance penalty
from doing I/O to an inadequately aligned buffer.  So palloc it instead.

Backpatch to 8.0 --- 7.4 doesn't have this code.
src/backend/commands/tablecmds.c