aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRobert Haas <rhaas@postgresql.org>2018-02-01 15:21:13 -0500
committerRobert Haas <rhaas@postgresql.org>2018-02-01 15:23:45 -0500
commitad25a6b1f25baf09c869c903c9c8e26d390875f5 (patch)
treeea9382f968b931e90cd6f20cf90c2a6302a9cf26
parentdf9f599bc6f14307252ac75ea1dc997310da5ba6 (diff)
downloadpostgresql-ad25a6b1f25baf09c869c903c9c8e26d390875f5.tar.gz
postgresql-ad25a6b1f25baf09c869c903c9c8e26d390875f5.zip
Fix possible failure to mark hash metapage dirty.
Report and suggested fix by Lixian Zou. Amit Kapila put it in the form of a patch and reviewed. Discussion: http://postgr.es/m/151739848647.1239.12528851873396651946@wrigleys.postgresql.org
-rw-r--r--src/backend/access/hash/hashovfl.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/backend/access/hash/hashovfl.c b/src/backend/access/hash/hashovfl.c
index c9de1283dca..2033b2f7f97 100644
--- a/src/backend/access/hash/hashovfl.c
+++ b/src/backend/access/hash/hashovfl.c
@@ -341,9 +341,10 @@ found:
metap->hashm_mapp[metap->hashm_nmaps] = BufferGetBlockNumber(newmapbuf);
metap->hashm_nmaps++;
metap->hashm_spares[splitnum]++;
- MarkBufferDirty(metabuf);
}
+ MarkBufferDirty(metabuf);
+
/*
* for new overflow page, we don't need to explicitly set the bit in
* bitmap page, as by default that will be set to "in use".