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

[Andrew Cooper <andrew.cooper3@xxxxxxxxxx>]

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