aboutsummaryrefslogtreecommitdiff
path: root/src/backend/executor/nodeWindowAgg.c
diff options
context:
space:
mode:
authorRobert Haas <rhaas@postgresql.org>2018-02-06 14:24:57 -0500
committerRobert Haas <rhaas@postgresql.org>2018-02-06 14:24:57 -0500
commit9fafa413ac602624e10f61ef44a20c17029d43d8 (patch)
treea94d9aa750deb7d1dc398b1de17d2cd8fafeede4 /src/backend/executor/nodeWindowAgg.c
parent3785f7eee3d98bc0a7ac2576aac9c0be974217d4 (diff)
downloadpostgresql-9fafa413ac602624e10f61ef44a20c17029d43d8.tar.gz
postgresql-9fafa413ac602624e10f61ef44a20c17029d43d8.zip
Avoid valgrind complaint about write() of uninitalized bytes.
LogicalTapeFreeze() may write out its first block when it is dirty but not full, and then immediately read the first block back in from its BufFile as a BLCKSZ-width block. This can only occur in rare cases where very few tuples were written out, which is currently only possible with parallel external tuplesorts. To avoid valgrind complaints, tell it to treat the tail of logtape.c's buffer as defined. Commit 9da0cc35284bdbe8d442d732963303ff0e0a40bc exposed this problem but did not create it. LogicalTapeFreeze() has always tended to write out some amount of garbage bytes, but previously never wrote less than one block of data in total, so the problem was masked. Per buildfarm members lousyjack and skink. Peter Geoghegan, based on a suggestion from Tom Lane and me. Some comment revisions by me.
Diffstat (limited to 'src/backend/executor/nodeWindowAgg.c')
0 files changed, 0 insertions, 0 deletions