diff options
author | Andres Freund <andres@anarazel.de> | 2016-03-07 14:24:03 -0800 |
---|---|---|
committer | Andres Freund <andres@anarazel.de> | 2016-03-07 14:24:03 -0800 |
commit | b63bea5fd3bba4d7a61c3beaba51a06f24b38da6 (patch) | |
tree | f8bbb7871fe6f213fcec3df2c7cdba09370bb3d3 /contrib/postgres_fdw/postgres_fdw.c | |
parent | 3fc6e2d7f5b652b417fa6937c34de2438d60fa9f (diff) | |
download | postgresql-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