On 11/16/10 02:16, Michael Roth wrote:
> +/* read handler for proxied connections */
> +static void vp_conn_read(void *opaque)
> +{
> + VPConn *conn = opaque;
> + VPDriver *drv = conn->drv;
> + VPPacket pkt;
> + char buf[VP_CONN_DATA_LEN];
> + int fd, count, ret;
> + bool client;
> +
> + TRACE("called with opaque: %p, drv: %p", opaque, drv);
> +
> + if (conn->state != VP_STATE_CONNECTED) {
> + LOG("invalid connection state");
> + return;
> + }
A read handler which doesn't pass back error values in any shape or form
does seem a little weird to me?
Cheers,
Jes
