On 10/15/2014 05:15 AM, Sergey Senozhatsky wrote:
> On (10/14/14 11:09), Joe Konno wrote:
>> diff --git a/src/lib.cpp b/src/lib.cpp
>> index 437803b..9cbf78d 100644
>> --- a/src/lib.cpp
>> +++ b/src/lib.cpp
>> @@ -474,10 +474,10 @@ int read_msr(int cpu, uint64_t offset, uint64_t *value)
>>  	int fd;
>>  	char msr_path[256];
>>  
>> -	sprintf(msr_path, "/dev/cpu/%d/msr", cpu);
>> +	fd = snprintf(msr_path, 256, "/dev/cpu/%d/msr", cpu);
> 
> iirc, this `fd =' has been removed. it's useless per se.
> 
> 	-ss

Will fix.