Michael Paquier writes:
> On Tue, Dec 9, 2014 at 11:11 AM, Mark Dilger wrote:
>> Perhaps the code really meant to say:
>> reserveFromBuffer(&buffer, VARHDRSZ)
> Good catch! The code is indeed incorrect. Attached is a one-line patch
> addressing that, I guess someone is going to pick up that soon
On Tue, Dec 9, 2014 at 11:11 AM, Mark Dilger wrote:
> The call:
>
> reserveFromBuffer(&buffer, sizeof(VARHDRSZ))
>
> is assuming that the size of varlena header is the same
> size as the type used to return that size, which happens
> to be so, but someone could easily change that macro
> t
The call:
reserveFromBuffer(&buffer, sizeof(VARHDRSZ))
is assuming that the size of varlena header is the same
size as the type used to return that size, which happens
to be so, but someone could easily change that macro
to:
#define VARHDRSZ ((int64) sizeof(int32))
And you'd wan