On Mon, Mar 12, 2018 at 04:52:13PM +0000, Hefty, Sean wrote: > > --- a/drivers/infiniband/core/ucma.c > > +++ b/drivers/infiniband/core/ucma.c > > @@ -1363,6 +1363,9 @@ static ssize_t ucma_process_join(struct > > ucma_file *file, > > if (IS_ERR(ctx)) > > return PTR_ERR(ctx); > > > > + if (!ctx->cm_id->device) > > + return -EINVAL; > > rdma_join_multicast() includes state checks which are intended to validate that the cm_id is in a valid state. It may make more sense to expand those checks with a device check, since the state checks alone are insufficient. Do you know why these checks not enough? Why did the change state without cm_id->device? Thanks > > - Sean >
Attachment:
signature.asc
Description: PGP signature
