linux-kernel - Re: [PATCH] perf: close the opened directories.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4C22FFE8.5050507@cn.fujitsu.com>
Date:	Thu, 24 Jun 2010 14:49:12 +0800
From:	Gui Jianfeng <guijianfeng@...fujitsu.com>
To:	"Zhang, Yanmin" <yanmin_zhang@...ux.intel.com>
CC:	acme@...stprotocols.net, mingo@...e.hu,
	linux kernel mailing list <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] perf: close the opened directories.

Zhang, Yanmin wrote:
> On Sat, 2010-06-19 at 11:54 +0800, Gui Jianfeng wrote:
>> When I ran "perf kvm ... top", I encountered the following error output.
>>
>>   Error: perfcounter syscall returned with -1 (Too many open files)
>>
>>   Fatal: No CONFIG_PERF_EVENTS=y kernel support configured?
>>
>> Looking into perf, I found perf opens too many direcotries at initialization
>> time, but forgets to close them. Here is the fix.
> Good catch!
> 
>> Signed-off-by: Gui Jianfeng <guijianfeng@...fujitsu.com>
>> ---
>>  tools/perf/util/symbol.c |   18 +++++++++++-------
>>  1 files changed, 11 insertions(+), 7 deletions(-)
>>
>> diff --git a/tools/perf/util/symbol.c b/tools/perf/util/symbol.c
>> index 7fd6b15..a00dcad 100644
>> --- a/tools/perf/util/symbol.c
>> +++ b/tools/perf/util/symbol.c
>> @@ -1443,6 +1443,7 @@ static int map_groups__set_modules_path_dir(struct map_groups *self,
>>  {
>>  	struct dirent *dent;
>>  	DIR *dir = opendir(dir_name);
>> +	int ret = 0;
>>  
>>  	if (!dir) {
>>  		pr_debug("%s: cannot open %s dir\n", __func__, dir_name);
>> @@ -1465,8 +1466,10 @@ static int map_groups__set_modules_path_dir(struct map_groups *self,
>>  
>>  			snprintf(path, sizeof(path), "%s/%s",
>>  				 dir_name, dent->d_name);
>> -			if (map_groups__set_modules_path_dir(self, path) < 0)
>> -				goto failure;
>> +			if (map_groups__set_modules_path_dir(self, path) < 0) {
>> +				ret = -1;
>> +				goto out;
>> +			}
> How about changing above to:
> 			ret = map_groups__set_modules_path_dir(self, path);
> 			if (ret < 0)
> 				goto out;

Sure, will change.

Thanks,
Gui

> 
> 
> 
>>  		} else {
>>  			char *dot = strrchr(dent->d_name, '.'),
>>  			     dso_name[PATH_MAX];
>> @@ -1487,17 +1490,18 @@ static int map_groups__set_modules_path_dir(struct map_groups *self,
>>  				 dir_name, dent->d_name);
>>  
>>  			long_name = strdup(path);
>> -			if (long_name == NULL)
>> -				goto failure;
>> +			if (long_name == NULL) {
>> +				ret = -1;
>> +				goto out;
>> +			}
>>  			dso__set_long_name(map->dso, long_name);
>>  			dso__kernel_module_get_build_id(map->dso, "");
>>  		}
>>  	}
>>  
>> -	return 0;
>> -failure:
>> +out:
>>  	closedir(dir);
>> -	return -1;
>> +	return ret;
>>  }
>>  
>>  static char *get_kernel_version(const char *root_dir)
> 
> 
> 
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/