]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Fix mdtruncate() to close fd.c handle of deleted segments.
authorAndres Freund <andres@anarazel.de>
Thu, 8 Sep 2016 23:51:09 +0000 (16:51 -0700)
committerAndres Freund <andres@anarazel.de>
Thu, 8 Sep 2016 23:52:13 +0000 (16:52 -0700)
commitf5462dedb70417f57e76b4f90d68d7d099e5b046
tree7b66442ec2a46d97fe362a5bc3d4f4d6ee9d928f
parent942ad33b20b7b57290902463dea3c460f5289ea8
Fix mdtruncate() to close fd.c handle of deleted segments.

mdtruncate() forgot to FileClose() a segment's mdfd_vfd, when deleting
it. That lead to a fd.c handle to a truncated file being kept open until
backend exit.

The issue appears to have been introduced way back in 1a5c450f3024ac5,
before that the handle was closed inside FileUnlink().

The impact of this bug is limited - only VACUUM and ON COMMIT TRUNCATE
for temporary tables, truncate files in place (i.e. TRUNCATE itself is
not affected), and the relation has to be bigger than 1GB. The
consequences of a leaked fd.c handle aren't severe either.

Discussion: <20160908220748.oqh37ukwqqncbl3n@alap3.anarazel.de>
Backpatch: all supported releases
src/backend/storage/smgr/md.c