Fix bug in do_query.
authorTatsuo Ishii <ishii@postgresql.org>
Fri, 15 Nov 2024 08:08:26 +0000 (17:08 +0900)
committerTatsuo Ishii <ishii@postgresql.org>
Fri, 15 Nov 2024 08:58:38 +0000 (17:58 +0900)
commit4eb55054e9c413c14db151c08fad39f5cc9e1352
treed314fa936bbdbace43a832e29e33aea874876482
parent43940e96310f65be6e36ab4c63d6acd106493cd3
Fix bug in do_query.

Upon receiving DataRow packet, it converts the number of fields from
network byte order to host byte order.  Unfortunately it used htons()
for this purpose instead of ntohs(). This is simply wrong. Similarly it
used htonl() instead of htohl() while converting the data length from
network byte order to host byte order. This is wrong too. But
fortunately both ntohs()/htons() and ntohl()/htonl() swap the byte and
they bring the same result (i.e. htonl(data_len) ==
ntohl(data_len). So actually the bug does not hurt anything.
However a bug is a bug. This commit fixes them.

Backpatch-through: v4.1
src/protocol/pool_process_query.c