Fix unportable code in pgbench.

tglsfdc · tglsfdc · commit 1a75c1d0c5d9 · 2019-04-17T17:30:29.000-04:00
The buildfarm points out that UINT64_FORMAT might not work with sscanf; it's calibrated for our printf implementation, which might not agree with the platform-supplied sscanf. Fall back to just accepting an unsigned long, which is already more than the documentation promises. Oversight in e6c3ba7; back-patch to v11, as that was.
diff --git a/src/bin/pgbench/pgbench.c b/src/bin/pgbench/pgbench.c
@@ -5019,16 +5019,19 @@ set_random_seed(const char *seed)
 	}
 	else
 	{
-		/* parse seed unsigned int value */
+		/* parse unsigned-int seed value */
+		unsigned long ulseed;
 		char		garbage;
 
-		if (sscanf(seed, UINT64_FORMAT "%c", &iseed, &garbage) != 1)
+		/* Don't try to use UINT64_FORMAT here; it might not work for sscanf */
+		if (sscanf(seed, "%lu%c", &ulseed, &garbage) != 1)
 		{
 			fprintf(stderr,
 					"unrecognized random seed option \"%s\": expecting an unsigned integer, \"time\" or \"rand\"\n",
 					seed);
 			return false;
 		}
+		iseed = (uint64) ulseed;
 	}
 
 	if (seed != NULL)

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 long ulseed;`
`5023`	`5024`	`char garbage;`
`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)`