[ovs-dev] [optimize 07/13] socket-util: New function get_socket_rcvbuf().
Ethan Jackson
ethan at nicira.com
Thu Oct 13 22:47:16 UTC 2011
> - socklen_t rcvbuf_len;
> - size_t rcvbuf = 0;
> + size_t rcvbuf;
>
> - rcvbuf_len = sizeof rcvbuf;
> - if (getsockopt(fd, SOL_SOCKET, SO_RCVBUF, &rcvbuf, &rcvbuf_len) < 0) {
> - static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(5, 10);
> - VLOG_ERR_RL(&rl, "getsockopt(SO_RCVBUF) failed: %s", strerror(errno));
> - return errno;
> + rcvbuf = get_socket_rcvbuf(fd);
> + if (rcvbuf < 0) {
> + return -rcvbuf;
> }
> +
I missed this in my original review. rcvbuf is a size_t and so will
never be less than zero. I think you need an int.
Ethan
More information about the dev
mailing list