Thanks Dan, nice catches. For all three: Acked-by: Andy Grover <andy.grover@xxxxxxxxxx> -- Andy On Thu, May 27, 2010 at 2:11 AM, Dan Carpenter <error27@xxxxxxxxx> wrote: > The "dma_pages" variable is always NULL at this point so the kfree() > isn't needed. > > Signed-off-by: Dan Carpenter <error27@xxxxxxxxx> > > diff --git a/net/rds/iw_rdma.c b/net/rds/iw_rdma.c > index 48d5073..742ab6a 100644 > --- a/net/rds/iw_rdma.c > +++ b/net/rds/iw_rdma.c > @@ -334,7 +334,6 @@ static u64 *rds_iw_map_scatterlist(struct rds_iw_device *rds_iwdev, > out_unmap: > ib_dma_unmap_sg(rds_iwdev->dev, sg->list, sg->len, DMA_BIDIRECTIONAL); > sg->dma_len = 0; > - kfree(dma_pages); > return ERR_PTR(ret); > } > > -- > To unsubscribe from this list: send the line "unsubscribe netdev" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
