Fix unsafe assumption that struct timeval.tv_sec is a "long".
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 7 Dec 2016 00:52:34 +0000 (19:52 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 7 Dec 2016 00:52:34 +0000 (19:52 -0500)
commit0645dacc371da6169b06934e3bd238c5f770fe25
tree2c8f1ab6c3daf911cd80d3f68e710f5800d2f6b2
parentc648f058319a59ad591dd9d1b0c48dfd655d063a
Fix unsafe assumption that struct timeval.tv_sec is a "long".

It typically is a "long", but it seems possible that on some platforms
it wouldn't be.  In any case, this silences a compiler warning on
OpenBSD (cf buildfarm member curculio).

While at it, use snprintf not sprintf.  This format string couldn't
possibly overrun the supplied buffer, but that doesn't seem like
a good reason not to use the safer style.

Oversight in commit f828654e1.  Back-patch to 9.6 where that came in.
src/backend/utils/error/elog.c