[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] x86/hvm: Correct hvm_ioreq_server_alloc_rangesets() failure path



On 04/06/14 13:22, Jan Beulich wrote:
>
>> @@ -861,24 +873,11 @@ static int hvm_ioreq_server_alloc_rangesets(struct 
>> hvm_ioreq_server *s,
>>      return 0;
>>  
>>   fail:
>> -    while ( --i >= 0 )
> ... the one line equivalent fix would have been to use while ( i-- ).
>
> Jan

I was considering the possible case where alloc/free_rangesets() gain
further moving parts, although I guess is probably unlikely.

~Andrew

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.