From mboxrd@z Thu Jan 1 00:00:00 1970 From: Subject: [PATCH] there are some memory leak about Dir operator Date: Tue, 19 Aug 2014 11:51:11 +0800 Message-ID: <1408420271-11568-1-git-send-email-zhang.zhangkun@huawei.com> Mime-Version: 1.0 Content-Type: text/plain Cc: zhangkun To: Return-path: List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" From: zhangkun Signed-off-by: zhangkun --- lib/librte_eal/linuxapp/eal/eal_hugepage_info.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/librte_eal/linuxapp/eal/eal_hugepage_info.c b/lib/librte_eal/linuxapp/eal/eal_hugepage_info.c index b72a205..d784b7c 100644 --- a/lib/librte_eal/linuxapp/eal/eal_hugepage_info.c +++ b/lib/librte_eal/linuxapp/eal/eal_hugepage_info.c @@ -311,12 +311,14 @@ eal_hugepage_info_init(void) /* if blocking lock failed */ if (flock(hpi->lock_descriptor, LOCK_EX) == -1) { RTE_LOG(CRIT, EAL, "Failed to lock hugepage directory!\n"); + closedir(dir); return -1; } /* clear out the hugepages dir from unused pages */ - if (clear_hugedir(hpi->hugedir) == -1) + if (clear_hugedir(hpi->hugedir) == -1) { + closedir(dir); return -1; - + } /* for now, put all pages into socket 0, * later they will be sorted */ hpi->num_pages[0] = get_num_hugepages(dirent->d_name); -- 1.7.3.1.msysgit.0