Re: [PATCH 1/2] rds-tools user space part of change for rds-stress -D option failure

Hi, Santosh:
   Pardon me, just check the mail box today.
   I used rds-tools-2.0.6-3.el6.src.rpm from RHEL 6,7 GA'ed source rpm. 

Thanks! - David

On Tue, 2016-03-01 at 11:03 -0800, santosh shilimkar wrote:

On 2/29/2016 5:47 PM, David Dai wrote:

quoted

---
  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