From: Jason Gunthorpe <jgg@xxxxxxxxxxxx> We free the memory a few lines later, the first assign cannot do anything useful. No sense in zeroing a stack variable before exiting. Signed-off-by: Jason Gunthorpe <jgg@xxxxxxxxxxxx> --- providers/hns/hns_roce_u.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/providers/hns/hns_roce_u.c b/providers/hns/hns_roce_u.c index 9b5de441a3fdf1..489b71614614fe 100644 --- a/providers/hns/hns_roce_u.c +++ b/providers/hns/hns_roce_u.c @@ -167,10 +167,7 @@ static void hns_roce_free_context(struct ibv_context *ibctx) if (to_hr_dev(ibctx->device)->hw_version == HNS_ROCE_HW_VER1) munmap(context->cq_tptr_base, HNS_ROCE_CQ_DB_BUF_SIZE); - context->uar = NULL; - free(context); - context = NULL; } static void hns_uninit_device(struct verbs_device *verbs_device) -- 2.15.1 -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
