Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit9dc2a3f

Browse files
committed
Check block number against the correct fork in get_raw_page().
get_raw_page tried to validate the supplied block number againstRelationGetNumberOfBlocks(), which of course is only right whenaccessing the main fork. In most cases, the main fork is longerthan the others, so that the check was too weak (allowing alower-level error to be reported, but no real harm to be done).However, very small tables could have an FSM larger than their heap,in which case the mistake prevented access to some FSM pages.Per report from Torsten Foertsch.In passing, make the bad-block-number error into an ereport not elog(since it's certainly not an internal error); and fix sloppilymaintained comment for RelationGetNumberOfBlocksInFork.This has been wrong since we invented relation forks, so back-patchto all supported branches.
1 parent4c6d0ab commit9dc2a3f

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

‎contrib/pageinspect/rawpage.c

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
#include"funcapi.h"
2525
#include"miscadmin.h"
2626
#include"storage/bufmgr.h"
27+
#include"storage/smgr.h"
2728
#include"utils/builtins.h"
2829

2930
PG_MODULE_MAGIC;
@@ -130,9 +131,13 @@ get_raw_page_internal(text *relname, ForkNumber forknum, BlockNumber blkno)
130131
(errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
131132
errmsg("cannot access temporary tables of other sessions")));
132133

133-
if (blkno >=RelationGetNumberOfBlocks(rel))
134-
elog(ERROR,"block number %u is out of range for relation \"%s\"",
135-
blkno,RelationGetRelationName(rel));
134+
RelationOpenSmgr(rel);
135+
136+
if (blkno >=smgrnblocks(rel->rd_smgr,forknum))
137+
ereport(ERROR,
138+
(errcode(ERRCODE_INVALID_PARAMETER_VALUE),
139+
errmsg("block number %u is out of range for relation \"%s\"",
140+
blkno,RelationGetRelationName(rel))));
136141

137142
/* Initialize buffer to copy to */
138143
raw_page= (bytea*)palloc(BLCKSZ+VARHDRSZ);

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp