Alexander Lakhin <exclus...@gmail.com> writes: > Please look at the function _readA_Const() (introduced in a6bc33019), which > fails on current master under valgrind: > ... > Here _readA_Const() performs: > union ValUnion *tmp = nodeRead(NULL, 0);
> memcpy(&local_node->val, tmp, sizeof(*tmp)); > where sizeof(union ValUnion) = 16, but nodeRead()->makeInteger() produced > Integer (sizeof(Integer) = 8). Right, so we can't get away without a switch-on-value-type like the other functions for A_Const have. Will fix. regards, tom lane