1berry/postgres
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix missing initialization of buffer_std field in _bt_newroot().

Browse Source 
This would only have any effect if the old root page needed to have
a full-page image made (ie, this was the first mod to it since a
checkpoint), *and* if the byte left uninitialized chanced to contain
zero.  In that case the WAL code would fail to remove the "hole" from
the full-page image, which would bloat the WAL log a bit but not have
any effect worse than that.  Found by buildfarm member skink, whose
valgrind run noticed the use of an uninitialized value.  Apparently
timing in the regression tests is such that the triggering condition
is rare, or valgrind testing would have seen this before.

Oversight in commit 40dae7ec537c5619fc93ad602c62f37be786d161.  This
bug affects only the 9.4 branch, since in later branches refactoring
of the WAL-log-creation APIs fixed it.

Report: <20160521203101.jp5yxquhhkabvo56@alap3.anarazel.de>
This commit is contained in:
Tom Lane 2016-05-21 17:37:16 -04:00
parent e0ca9a4ca6
commit 9561f6e977
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
src/backend/access/nbtree/nbtinsert.c
View File

@ -2118,6 +2118,7 @@ _bt_newroot(Relation rel, Buffer lbuf, Buffer rbuf)
rdata[2].data = NULL; rdata[2].data = NULL;
rdata[2].len = 0; rdata[2].len = 0;
rdata[2].buffer = lbuf; rdata[2].buffer = lbuf;
rdata[2].buffer_std = true;
rdata[2].next = NULL; rdata[2].next = NULL;
recptr = XLogInsert(RM_BTREE_ID, XLOG_BTREE_NEWROOT, rdata); recptr = XLogInsert(RM_BTREE_ID, XLOG_BTREE_NEWROOT, rdata);