NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitf0ee42d

committed

Fix unportable coding in DetermineSleepTime().

We should not assume that struct timeval.tv_sec is a long, becauseit ain't necessarily. (POSIX says that it's a time_t, which mightwell be 64 bits now or in the future; or for that matter might be32 bits on machines with 64-bit longs.) Per buildfarm member panther.Back-patch to 9.3 where the dubious coding was introduced.

1 parent60ff2fd commitf0ee42dCopy full SHA for f0ee42d

File tree

1 file changed

-1

lines changed

src/backend/postmaster
- postmaster.c

1 file changed

-1

lines changed

`‎src/backend/postmaster/postmaster.c`

Lines changed: 3 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -1462,10 +1462,12 @@ DetermineSleepTime(struct timeval * timeout)`
`1462`	`1462`
`1463`	`1463`	`if (next_wakeup!=0)`
`1464`	`1464`	`{`
	`1465`	`+longsecs;`
`1465`	`1466`	`intmicrosecs;`
`1466`	`1467`
`1467`	`1468`	`TimestampDifference(GetCurrentTimestamp(),next_wakeup,`
`1468`		`-&timeout->tv_sec,&microsecs);`
	`1469`	`+&secs,&microsecs);`
	`1470`	`+timeout->tv_sec=secs;`
`1469`	`1471`	`timeout->tv_usec=microsecs;`
`1470`	`1472`
`1471`	`1473`	`/* Ensure we don't exceed one minute */`

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!

Commitf0ee42d

File tree

1 file changed

1 file changed

`‎src/backend/postmaster/postmaster.c`

0 commit comments