Received: by 2002:a25:d7c1:0:0:0:0:0 with SMTP id o184csp4032787ybg; Fri, 25 Oct 2019 12:21:18 -0700 (PDT) X-Google-Smtp-Source: APXvYqzVgLhP9ZlNUxzMh0+qm/GAvPnRN2QvE7MCQnWBEQCN/Hiby91zAkJyBKcGY6t0aLebWm7t X-Received: by 2002:a05:6402:386:: with SMTP id o6mr5942622edv.148.1572031278696; Fri, 25 Oct 2019 12:21:18 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1572031278; cv=none; d=google.com; s=arc-20160816; b=bC2MEwXMn/lmy+Ash/CA/ImLeB8XwH5SrKhGmCMXqDW3mocP1qYWrWwCmStco2F3kU +nFKNrEt34/MrKay8rmfUZYhzzRahHA3RVvtYu1dJV825lBZsY2aANsG1a2xSOUsuHht d15pKOT8+UeYI+sC3vltIVWR5vbSsgtLqKhtFYpQ1Ygits3xHcloxmexAkOucz017ceL aDGMUXRTObswwnBnWvb2nOXQ4A3//SUEvsmOu18iVdJPogXeMfqM+6nu2seH9JotP6GQ CjoyCrkqFFzpP0YY+depBY1Q7PR9sMdNJUJdd1+O9e3DtSsReUq9sodd/IRuzYfbYaZa kE6Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=UO48AEJ3FLWD5Ww/KlypNXqpqE7XLnTrjj30Gu+5FZE=; b=JjBp6QtFowj60/qkE1dGyV0p0ovhSrNb/pfJw2YR3PyegZtjwB3Ece44QnsIX25QZp Ce7mXWKsNQ4CA1n605JIcYDTnX9qLwCpG1xAALlVutGAPbutcoX2V/j1OeWN3Mxl3MCf uj+2Aa6QX66YxmrQQ8UuFNTyZBBE28QvhV/u1F7paCSSFgdEQqb9PcAtcfpTqOAdbc8q DCmuhWpdr+J61y/1LMM9lKUYQaL7PmVeQVjWE3jtPl+8IENZ2UUmWTYkn8Ncl87Gm7/W 59d2o5EnC3AZKcXVh1Hh3HUQpDX5b5T7RryemlNsG2k/ZHxnUDYNPt1zDCp8oBI2BBu+ w/OA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id b3si1924846ede.118.2019.10.25.12.20.55; Fri, 25 Oct 2019 12:21:18 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2403848AbfJYDQO (ORCPT + 99 others); Thu, 24 Oct 2019 23:16:14 -0400 Received: from szxga04-in.huawei.com ([45.249.212.190]:5176 "EHLO huawei.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1732607AbfJYDQO (ORCPT ); Thu, 24 Oct 2019 23:16:14 -0400 Received: from DGGEMS411-HUB.china.huawei.com (unknown [172.30.72.59]) by Forcepoint Email with ESMTP id BE266CE46B02804BCB4D; Fri, 25 Oct 2019 11:16:11 +0800 (CST) Received: from huawei.com (10.175.104.225) by DGGEMS411-HUB.china.huawei.com (10.3.19.211) with Microsoft SMTP Server id 14.3.439.0; Fri, 25 Oct 2019 11:16:05 +0800 From: Hewenliang To: , , , , , , , , , CC: , Subject: [PATCH] perf tools: Call closedir to release the resource before we return Date: Thu, 24 Oct 2019 23:16:05 -0400 Message-ID: <20191025031605.23658-1-hewenliang4@huawei.com> X-Mailer: git-send-email 2.19.1 MIME-Version: 1.0 Content-Transfer-Encoding: 7BIT Content-Type: text/plain; charset=US-ASCII X-Originating-IP: [10.175.104.225] X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org We should close the directory on pattern failure before the return of rm_rf_depth_pat. Fixes: cdb6b0235f170 ("perf tools: Add pattern name checking to rm_rf") Signed-off-by: Hewenliang --- tools/perf/util/util.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tools/perf/util/util.c b/tools/perf/util/util.c index 5eda6e19c947..1aadca8c43f3 100644 --- a/tools/perf/util/util.c +++ b/tools/perf/util/util.c @@ -154,8 +154,10 @@ static int rm_rf_depth_pat(const char *path, int depth, const char **pat) if (!strcmp(d->d_name, ".") || !strcmp(d->d_name, "..")) continue; - if (!match_pat(d->d_name, pat)) + if (!match_pat(d->d_name, pat)) { + closedir(dir); return -2; + } scnprintf(namebuf, sizeof(namebuf), "%s/%s", path, d->d_name); -- 2.19.1