[Pgpool-hackers] Return value from SimpleForwardToBackend()

[Fujii Masao]

Hi,

Why does SimpleForwardToBackend() return POOL_CONTINUE when the message length
from the client is less than 4?

> 	if (pool_read(frontend, &sendlen, sizeof(sendlen)))
> 	{
> 		return POOL_END;
> 	}
> 
<snip>
> 
> 	len = ntohl(sendlen) - 4;
> 
> 	if (len <= 0)
> 		return POOL_CONTINUE;

In PostgreSQL, pq_getmessage() returns EOF with the error message at the same condition.
I think that SimpleForwardToBackend() should return POOL_END as follows.

> 	len = ntohl(sendlen) - 4;
>
> 	if (len == 0)
> 		return POOL_CONTINUE;
> 	else if (len < 0)
> 		return POOL_END;

Regards,