Re: [win-pv-devel] [PATCH] Function RangeSetPop needs to move Cursor forward

[Paul Durrant <Paul.Durrant@xxxxxxxxxx>]

> -----Original Message-----
> From: win-pv-devel [mailto:win-pv-devel-bounces@xxxxxxxxxxxxxxxxxxxx] On
> Behalf Of Lin Liu
> Sent: 18 September 2018 10:12
> To: win-pv-devel@xxxxxxxxxxxxxxxxxxxx
> Cc: Lin Liu <lin.liu@xxxxxxxxxx>
> Subject: [win-pv-devel] [PATCH] Function RangeSetPop needs to move Cursor
> forward
> 
> To traverse the RangeSet list, the Cursor needs to move forward,
> otherwise it will end with dead loop
> 
> Signed-off-by: Lin Liu <lin.liu@xxxxxxxxxx>

Acked-by: Paul Durrant <paul.durrant@xxxxxxxxxx>

> ---
>  src/xenbus/range_set.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/src/xenbus/range_set.c b/src/xenbus/range_set.c
> index f52af63..162e3c8 100644
> --- a/src/xenbus/range_set.c
> +++ b/src/xenbus/range_set.c
> @@ -213,6 +213,8 @@ RangeSetPop(
> 
>          if ((ULONGLONG)(Range->End + 1 - Range->Start) >= Count)
>              goto found;
> +
> +        Cursor = Cursor->Flink;
>      }
> 
>      goto fail3;
> --
> 2.17.0
> 
> 
> _______________________________________________
> win-pv-devel mailing list
> win-pv-devel@xxxxxxxxxxxxxxxxxxxx
> https://lists.xenproject.org/mailman/listinfo/win-pv-devel
_______________________________________________
win-pv-devel mailing list
win-pv-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/win-pv-devel