NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitde5afdd

committed

Fix overflow in bsearch_arg() with more than INT_MAX elements

This was introduced in commitbfa2cee, which replaced the oldbsearch_cmp() function we had in extended_stats.c with the currentimplementation. The original discussion or commit message ofbfa2cee didn't mention where the new implementation came from, butbased on some googling, I'm guessing *BSD or libiberty, all of whichshare this same code, with or without this fix.Author: Ranier VilelaReviewed-by: Nathan BossartBackpatch-through: 14Discussion:https://www.postgresql.org/message-id/CAEudQAp34o_8u6sGSVraLwuMv9F7T9hyHpePXHmRaxR2Aboi%2Bw%40mail.gmail.com

1 parent22bb889 commitde5afddCopy full SHA for de5afdd

File tree

1 file changed

-2

lines changed

src/port
- bsearch_arg.c

1 file changed

-2

lines changed

`‎src/port/bsearch_arg.c`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -58,8 +58,8 @@ bsearch_arg(const void key, const void base0,`
`58`	`58`	`void*arg)`
`59`	`59`	`{`
`60`	`60`	`constcharbase= (constchar)base0;`
`61`		`-intlim,`
`62`		`-cmp;`
	`61`	`+size_tlim;`
	`62`	`+intcmp;`
`63`	`63`	`constvoid*p;`
`64`	`64`
`65`	`65`	`for (lim=nmemb;lim!=0;lim >>=1)`

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!

Commitde5afdd

File tree

1 file changed

1 file changed

`‎src/port/bsearch_arg.c`

0 commit comments