]> granicus.if.org Git - postgresql/commit
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:44 +0000 (19:52 -0500)
commit81f3c20a65e683ee235a5d3aa6146db84eb75ebf
tree676103c8bcc11e50fab3feab6aa3abc8f5dfdb26
parentebe5dc9e02a6464713bd48e266eee48351d23062
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