Commit3b9c875

michaelpq

authored and

pull[bot]

committed

Fix out-of-bound read in gtsvector_picksplit()

This could lead to an imprecise choice when splitting an index page of aGiST index on a tsvector, deciding which entries should remain on theold page and which entries should move to a new page.This is wrong since tsearch2 has been moved into core with commit140d4eb, so backpatch all the way down. This error has beenspotted by valgrind.Author: Alexander LakhinDiscussion:https://postgr.es/m/17950-6c80a8d2b94ec695@postgresql.orgBackpatch-through: 11

1 parentc022f37 commit3b9c875Copy full SHA for 3b9c875

File tree

1 file changed

-2

lines changed

src/backend/utils/adt
- tsgistidx.c

1 file changed

-2

lines changed

`‎src/backend/utils/adt/tsgistidx.c‎`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -730,7 +730,7 @@ gtsvector_picksplit(PG_FUNCTION_ARGS)`
`730`	`730`	`size_alpha=SIGLENBIT(siglen)-`
`731`	`731`	`sizebitvec((cache[j].allistrue) ?`
`732`	`732`	`GETSIGN(datum_l) :`
`733`		`-GETSIGN(cache[j].sign),`
	`733`	`+cache[j].sign,`
`734`	`734`	`siglen);`
`735`	`735`	`}`
`736`	`736`	`else`
`@@ -744,7 +744,7 @@ gtsvector_picksplit(PG_FUNCTION_ARGS)`
`744`	`744`	`size_beta=SIGLENBIT(siglen)-`
`745`	`745`	`sizebitvec((cache[j].allistrue) ?`
`746`	`746`	`GETSIGN(datum_r) :`
`747`		`-GETSIGN(cache[j].sign),`
	`747`	`+cache[j].sign,`
`748`	`748`	`siglen);`
`749`	`749`	`}`
`750`	`750`	`else`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

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

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit3b9c875

File tree

1 file changed

1 file changed

`‎src/backend/utils/adt/tsgistidx.c‎`

0 commit comments