On Fri, 2015-05-29 at 23:12 -0700, Roland Dreier wrote: > From: Roland Dreier <roland@xxxxxxxxxxxxxxx> > > We don't assign pi_ctx to desc->pi_ctx until we're certain to succeed > in the function. That means the cleanup path should use the local > pi_ctx variable, not desc->pi_ctx. > > This was detected by Coverity (CID 1260062). > > Signed-off-by: Roland Dreier <roland@xxxxxxxxxxxxxxx> > --- > drivers/infiniband/ulp/isert/ib_isert.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/infiniband/ulp/isert/ib_isert.c b/drivers/infiniband/ulp/isert/ib_isert.c > index 327529ee85eb..3f40319a55da 100644 > --- a/drivers/infiniband/ulp/isert/ib_isert.c > +++ b/drivers/infiniband/ulp/isert/ib_isert.c > @@ -547,11 +547,11 @@ isert_create_pi_ctx(struct fast_reg_descriptor *desc, > return 0; > > err_prot_mr: > - ib_dereg_mr(desc->pi_ctx->prot_mr); > + ib_dereg_mr(pi_ctx->prot_mr); > err_prot_frpl: > - ib_free_fast_reg_page_list(desc->pi_ctx->prot_frpl); > + ib_free_fast_reg_page_list(pi_ctx->prot_frpl); > err_pi_ctx: > - kfree(desc->pi_ctx); > + kfree(pi_ctx); > > return ret; > } Applied to target-pending/master. Thanks Roland! --nab -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
