> > +
> > +   rc = ib_get_cached_gid(ibqp->device, attr->ah_attr.port_num,
> > +                          attr->ah_attr.grh.sgid_index, &gid, &gid_attr);
> > +   if (!rc && !memcmp(&gid, &zgid, sizeof(gid)))
> > +           rc = -ENOENT;
> > +
> > +   if (!rc && gid_attr.ndev) {

<snip>

> > +           }
> > +   }
> > +   if (rc)
> > +           return -EINVAL;
> 
> I think it is better to check "rc" right after call to ib_get_cached_gid().

Somehow this e-mail slipped.
Thanks, will update.

Reply via email to