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

David Dai
2016-03-03T00:04:24Z

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