Re: [libvirt] [PATCH] cmdFreepages: initialize @tmp

2014-06-23 Thread Eric Blake
On 06/23/2014 12:34 PM, Michal Privoznik wrote: > In the 404bac14 the @tmp variable was introduced. It's purpose is to > avoid typecasting when parsing --pagesize argument. However, if the > argument is not presented, tmp may be used uninitialized resulting in > bogus virNodeGetFreePages() API call

[libvirt] [PATCH] cmdFreepages: initialize @tmp

2014-06-23 Thread Michal Privoznik
In the 404bac14 the @tmp variable was introduced. It's purpose is to avoid typecasting when parsing --pagesize argument. However, if the argument is not presented, tmp may be used uninitialized resulting in bogus virNodeGetFreePages() API call: virsh freepages --cellno 2 error: Failed to open file