Re: [PATCH 1/2] rds-tools user space part of change for rds-stress -D option failure
From: santosh shilimkar <hidden>
Date: 2016-03-01 19:03:00
On 2/29/2016 5:47 PM, David Dai wrote:
quoted hunk
--- rds-stress.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)diff --git a/rds-stress.c b/rds-stress.c index 9f0623c..6632d58 100755 --- a/rds-stress.c +++ b/rds-stress.c@@ -1400,7 +1400,7 @@ static int recv_message(int fd, return ret; if (ret && ret < sizeof(struct header)) die("recvmsg() returned short data: %zd", ret); - if (msg.msg_namelen < sizeof(struct sockaddr_in)) + if (ret && msg.msg_namelen < sizeof(struct sockaddr_in)) die("socklen = %d < sizeof(sin) (%zu)\n", msg.msg_namelen, sizeof(struct sockaddr_in));
External rds-tools repo's are really stale. Which sources you used to create this patch against ? Regards, Santosh -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html