Hi,

while browsing the source I found a typo in socket.c::SOCK_Constructor()

Original code:
if (rv)
rv->buffer_size =
conn->connInfo.drivers.socket_buffersize;
else
rv->buffer_size = globals.socket_buffersize;

But since rv is allocated and checked before, the test should probably be:
if (conn)
rv->buffer_size =
conn->connInfo.drivers.socket_buffersize;
else
rv->buffer_size = globals.socket_buffersize;

Rainer

Rainer Bauer wrote:
> Hi,
>
> while browsing the source I found a typo in socket.c::SOCK_Constructor()
>
> Original code:
> if (rv)
> rv->buffer_size =
> conn->connInfo.drivers.socket_buffersize;
> else
> rv->buffer_size = globals.socket_buffersize;
>
> But since rv is allocated and checked before, the test should probably be:
> if (conn)
> rv->buffer_size =
> conn->connInfo.drivers.socket_buffersize;
> else
> rv->buffer_size = globals.socket_buffersize;

Oops you are right.
Thanks.

regards,
Hiroshi Inoue

---------------------------(end of broadcast)---------------------------
TIP 2: Don't 'kill -9' the postmaster