NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitae10dbb

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 parente70ed4b commitae10dbbCopy full SHA for ae10dbb

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!

Commitae10dbb

File tree

1 file changed

1 file changed

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

0 commit comments