On 10/21/2010 03:54 PM, Dhaval Giani wrote:
> On Thu, Oct 21, 2010 at 2:35 PM, Ivana Hutarova Varekova
> <[email protected]>  wrote:
>    
>> Because cgpconfigparser changes the permissions of all files there have to 
>> be checked the permissions of files in root group to set vhether the 
>> variable i s writable or not.
>>
>> Signed-off-by: Ivana Hutarova Varekova<[email protected]>
>> ---
>>
>>   src/tools/cgsnapshot.c |   10 ++++++----
>>   1 files changed, 6 insertions(+), 4 deletions(-)
>>
>> diff --git a/src/tools/cgsnapshot.c b/src/tools/cgsnapshot.c
>> index dcab130..caecc3a 100644
>> --- a/src/tools/cgsnapshot.c
>> +++ b/src/tools/cgsnapshot.c
>> @@ -273,7 +273,7 @@ static int display_permissions(const char *path,
>>
>>   static int display_cgroup_data(struct cgroup *group,
>>                 char controller[CG_CONTROLLER_MAX][FILENAME_MAX],
>> -               const char *group_path, int first,
>> +               const char *group_path, int root_path_len, int first,
>>                 const char *program_name)
>>      
> minor nit. can you add it to the end of the list of arguments.
>
> Dhaval
>    
Hello,
I put root_path_len to group_path because this variable have reference 
to it. Do you really want to have it at the end of variable list? If yes 
I will move it there.
Ivana

------------------------------------------------------------------------------
Nokia and AT&T present the 2010 Calling All Innovators-North America contest
Create new apps & games for the Nokia N8 for consumers in  U.S. and Canada
$10 million total in prizes - $4M cash, 500 devices, nearly $6M in marketing
Develop with Nokia Qt SDK, Web Runtime, or Java and Publish to Ovi Store 
http://p.sf.net/sfu/nokia-dev2dev
_______________________________________________
Libcg-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/libcg-devel

Reply via email to