]> granicus.if.org Git - postgresql/commit
Change pqformat.h's integer handling functions to take unsigned integers.
authorAndres Freund <andres@anarazel.de>
Wed, 27 Jun 2018 06:40:32 +0000 (23:40 -0700)
committerAndres Freund <andres@anarazel.de>
Wed, 27 Jun 2018 06:40:32 +0000 (23:40 -0700)
commit42121790ca9b27e01fd0471fdac28ec4a381eaee
tree24c01fc6be6377bdaf331dfd8098dfe2534e49cb
parent986070872ffaae5d9d0e39eddcc952bc261982a2
Change pqformat.h's integer handling functions to take unsigned integers.

As added in 1de09ad8eb1fa673ee7899d6dfbb2b49ba204818 the new functions
all accept signed integers as parameters. That's not great, because
it's perfectly reasonable to call them with unsigned parameters.
Unfortunately unsigned to signed conversion is not well defined, when
exceeding the range of the signed value.  That's presently not a
practical issue in postgres (among other reasons because we force
gcc's hand with -fwrapv).  But it's clearly not quite right.

Thus change the signatures to accept unsigned integers instead, signed
to unsigned conversion is always well defined. Also change the
backward compat pq_sendint() - while it's deprecated it seems better
to be consistent.

Per discussion between Andrew Gierth, Michael Paquier, Alvaro Herrera
and Tom Lane.

Reported-By: Andrew Gierth
Author: Andres Freund
Discussion: https://postgr.es/m/87r2m10zm2.fsf@news-spur.riddles.org.uk
src/include/libpq/pqformat.h