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 kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html