diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2004-08-17 23:16:07 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2004-08-17 23:16:07 +0000 |
| commit | ba9b9a6e6220b47b92a477e9dc663456ef04cd24 (patch) | |
| tree | dc5e230a726eb4c71f5eaf71e48ea9c5874aa26d /src | |
| parent | 17663e1aebf91a926a8ff2dd30d1fa1dedee2563 (diff) | |
| download | postgresql-ba9b9a6e6220b47b92a477e9dc663456ef04cd24.tar.gz postgresql-ba9b9a6e6220b47b92a477e9dc663456ef04cd24.zip | |
Fix bug introduced into _bt_getstackbuf() on 2003-Feb-21: the initial
value of 'start' could be past the end of the page, if the page was
split by some concurrent inserting process since we visited it. In
this situation the code could look at bogus entries and possibly find
a match (since after all those entries still contain what they had
before the split). This would lead to 'specified item offset is too large'
followed by 'PANIC: failed to add item to the page', as reported by Joe
Conway for scenarios involving heavy concurrent insertion activity.
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/access/nbtree/nbtinsert.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/src/backend/access/nbtree/nbtinsert.c b/src/backend/access/nbtree/nbtinsert.c index 27c833408d6..d2644b25dde 100644 --- a/src/backend/access/nbtree/nbtinsert.c +++ b/src/backend/access/nbtree/nbtinsert.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/access/nbtree/nbtinsert.c,v 1.106 2003/09/25 06:57:57 petere Exp $ + * $Header: /cvsroot/pgsql/src/backend/access/nbtree/nbtinsert.c,v 1.106.2.1 2004/08/17 23:16:07 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -1316,6 +1316,13 @@ _bt_getstackbuf(Relation rel, BTStack stack, int access) start = minoff; /* + * Need this check too, to guard against possibility that page + * split since we visited it originally. + */ + if (start > maxoff) + start = OffsetNumberNext(maxoff); + + /* * These loops will check every item on the page --- but in an * order that's attuned to the probability of where it * actually is. Scan to the right first, then to the left. |