aboutsummaryrefslogtreecommitdiff
path: root/contrib/postgres_fdw/postgres_fdw.c
diff options
context:
space:
mode:
authorAndres Freund <andres@anarazel.de>2016-03-07 14:24:03 -0800
committerAndres Freund <andres@anarazel.de>2016-03-07 14:24:03 -0800
commitb63bea5fd3bba4d7a61c3beaba51a06f24b38da6 (patch)
treef8bbb7871fe6f213fcec3df2c7cdba09370bb3d3 /contrib/postgres_fdw/postgres_fdw.c
parent3fc6e2d7f5b652b417fa6937c34de2438d60fa9f (diff)
downloadpostgresql-b63bea5fd3bba4d7a61c3beaba51a06f24b38da6.tar.gz
postgresql-b63bea5fd3bba4d7a61c3beaba51a06f24b38da6.zip
Further improvements to c8f621c43.
Coverity and inspection for the issue addressed in fd45d16f found some questionable code. Specifically coverity noticed that the wrong length was added in ReorderBufferSerializeChange() - without immediate negative consequences as the variable isn't used afterwards. During code-review and testing I noticed that a bit of space was wasted when allocating tuple bufs in several places. Thirdly, the debug memset()s in ReorderBufferGetTupleBuf() reduce the error checking valgrind can do. Backpatch: 9.4, like c8f621c43.
Diffstat (limited to 'contrib/postgres_fdw/postgres_fdw.c')
0 files changed, 0 insertions, 0 deletions