NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit1a75c1d

committed

Fix unportable code in pgbench.

The buildfarm points out that UINT64_FORMAT might not work with sscanf;it's calibrated for our printf implementation, which might not agreewith the platform-supplied sscanf. Fall back to just accepting anunsigned long, which is already more than the documentation promises.Oversight ine6c3ba7; back-patch to v11, as that was.

1 parent8cde7f4 commit1a75c1dCopy full SHA for 1a75c1d

File tree

1 file changed

-2

lines changed

src/bin/pgbench
- pgbench.c

1 file changed

-2

lines changed

`‎src/bin/pgbench/pgbench.c`

Lines changed: 5 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -5019,16 +5019,19 @@ set_random_seed(const char *seed)`
`5019`	`5019`	`}`
`5020`	`5020`	`else`
`5021`	`5021`	`{`
`5022`		`-/* parse seed unsigned int value */`
	`5022`	`+/* parse unsigned-int seed value */`
	`5023`	`+unsigned longulseed;`
`5023`	`5024`	`chargarbage;`
`5024`	`5025`
`5025`		`-if (sscanf(seed,UINT64_FORMAT"%c",&iseed,&garbage)!=1)`
	`5026`	`+/* Don't try to use UINT64_FORMAT here; it might not work for sscanf */`
	`5027`	`+if (sscanf(seed,"%lu%c",&ulseed,&garbage)!=1)`
`5026`	`5028`	`{`
`5027`	`5029`	`fprintf(stderr,`
`5028`	`5030`	`"unrecognized random seed option \"%s\": expecting an unsigned integer, \"time\" or \"rand\"\n",`
`5029`	`5031`	`seed);`
`5030`	`5032`	`return false;`
`5031`	`5033`	`}`
	`5034`	`+iseed= (uint64)ulseed;`
`5032`	`5035`	`}`
`5033`	`5036`
`5034`	`5037`	`if (seed!=NULL)`

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!

Commit1a75c1d

File tree

1 file changed

1 file changed

`‎src/bin/pgbench/pgbench.c`

0 commit comments