diff options
| author | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2013-10-23 14:03:54 +0300 |
|---|---|---|
| committer | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2013-10-23 14:24:37 +0300 |
| commit | 83eb54001cb69d6ee8f0813c4e280be876823068 (patch) | |
| tree | 14fb288e289dcb9c2edb95b2a7272e89913cf22a /src/backend/utils/adt/encode.c | |
| parent | 5f1ab4610102a73c124000788585c1af2a36284b (diff) | |
| download | postgresql-83eb54001cb69d6ee8f0813c4e280be876823068.tar.gz postgresql-83eb54001cb69d6ee8f0813c4e280be876823068.zip | |
Fix two bugs in setting the vm bit of empty pages.
Use a critical section when setting the all-visible flag on an empty page,
and WAL-logging it. log_newpage_buffer() contains an assertion that it
must be called inside a critical section, and it's the right thing to do
when modifying a buffer anyway.
Also, the page should be marked dirty before calling log_newpage_buffer(),
per the comment in log_newpage_buffer() and src/backend/access/transam/README.
Patch by Andres Freund, in response to my report. Backpatch to 9.2, like
the patch that introduced these bugs (a6370fd9).
Diffstat (limited to 'src/backend/utils/adt/encode.c')
0 files changed, 0 insertions, 0 deletions