diff options
| author | Robert Haas <rhaas@postgresql.org> | 2019-12-17 15:53:17 -0500 |
|---|---|---|
| committer | Robert Haas <rhaas@postgresql.org> | 2019-12-17 15:53:17 -0500 |
| commit | 5184f110aa4130ec87b0b3e0834292cd8cb1fd8a (patch) | |
| tree | 22a68c0a88b09175903f096b6d986fdc064dc542 /src | |
| parent | d5406dea25b600408e7acf17d5a06e82d3ce6d0d (diff) | |
| download | postgresql-5184f110aa4130ec87b0b3e0834292cd8cb1fd8a.tar.gz postgresql-5184f110aa4130ec87b0b3e0834292cd8cb1fd8a.zip | |
Fix bad formula in previous commit.
Commit d5406dea25b600408e7acf17d5a06e82d3ce6d0d used a slightly
novel, and wrong, approach to compute the length of the last
toast chunk. It worked fine unless the last chunk happened to
have the largest possible size.
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/access/common/detoast.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/access/common/detoast.c b/src/backend/access/common/detoast.c index 61d7e64c449..6341107e882 100644 --- a/src/backend/access/common/detoast.c +++ b/src/backend/access/common/detoast.c @@ -443,7 +443,7 @@ toast_fetch_datum(struct varlena *attr) toast_pointer.va_valueid, RelationGetRelationName(toastrel)))); expected_size = curchunk < totalchunks - 1 ? TOAST_MAX_CHUNK_SIZE - : attrsize % TOAST_MAX_CHUNK_SIZE; + : attrsize - ((totalchunks - 1) * TOAST_MAX_CHUNK_SIZE); if (chunksize != expected_size) ereport(ERROR, (errcode(ERRCODE_DATA_CORRUPTED), @@ -676,7 +676,7 @@ toast_fetch_datum_slice(struct varlena *attr, int32 sliceoffset, toast_pointer.va_valueid, RelationGetRelationName(toastrel)))); expected_size = curchunk < totalchunks - 1 ? TOAST_MAX_CHUNK_SIZE - : attrsize % TOAST_MAX_CHUNK_SIZE; + : attrsize - ((totalchunks - 1) * TOAST_MAX_CHUNK_SIZE); if (chunksize != expected_size) ereport(ERROR, (errcode(ERRCODE_DATA_CORRUPTED), |